MySQL AND operator - Printable Version

MySQL AND operator - Printable Version

+- SA-MP Forums Archive (https://sampforum.blast.hk)
+-- Forum: SA-MP Scripting and Plugins (https://sampforum.blast.hk/forumdisplay.php?fid=8)
+--- Forum: Scripting Help (https://sampforum.blast.hk/forumdisplay.php?fid=12)
+--- Thread: MySQL AND operator (/showthread.php?tid=618889)

MySQL AND operator - NeXoR - 10.10.2016

I inserted a row to my user database, in order to test my IP banning system
That's the pawn code

PHP Code:



    new query[248];

    mysql_format(mysql, query, sizeof(query), "SELECT `BannedBy`, `BanReason`, `BannedDate`, `Username` WHERE `IP`='%e' AND `Banned`='1'", RPIP(playerid));

    mysql_tquery(mysql, query, "CheckBan", "i", playerid);

PHP Code:



forward CheckBan(playerid);

public CheckBan(playerid)

{

    new rows, fields;

    cache_get_data(rows, fields, mysql);

    if(rows)

    {

        new banreason[128], bannedby[24], bandate[10], username[24], string[128];

        cache_get_field_content(0, "BannedBy", bannedby, mysql, sizeof(bannedby));

        cache_get_field_content(0, "BanReason", banreason, mysql, sizeof(banreason));

        cache_get_field_content(0, "BannedDate", bandate, mysql, sizeof(bandate));

        cache_get_field_content(0, "Username", username, mysql, sizeof(username));

        SendClientMessage(playerid, COLOR_GREEN, "-------[Banned]-------");

        format(string, sizeof(string), "Username: %s || Banning Date: %s", username, bandate);

        SendClientMessage(playerid, COLOR_RED, string);

        format(string, sizeof(string), "IP Address: %s || Banned By: %s", RPIP(playerid), bannedby);

        SendClientMessage(playerid, COLOR_RED, string);

        format(string, sizeof(string), "Reason: %s", banreason);

        SendClientMessage(playerid, COLOR_RED, string);

        SendClientMessage(playerid, COLOR_YELLOW, "If you think that you have been wrongfully banned, visit our forums and apply for an unban.");

        SendClientMessage(playerid, COLOR_GREEN, "-------[Banned]-------");

        SetTimerEx("DelayedKick", 1000, 0, "i", playerid);

    }

}

That's the error:

Code:

[01:35:09] [ERROR] CMySQLQuery::Execute[CheckBan] - (error #1064) You have an error in your SQL syntax; check the manual that corresponds to your MariaDB server version for the right syntax to use near 'WHERE `IP`='127.0.0.1' AND `Banned`='1'' at line 1

Re: MySQL AND operator - WhiteKinG - 10.10.2016

PHP Code:



mysql_format(mysql, query, sizeof(query), "SELECT `BannedBy`, `BanReason`, `BannedDate`, `Username` FROM table_ban WHERE `IP`='%e' AND `Banned`='1'", RPIP(playerid));

you forget the table name

Re: MySQL AND operator - NeXoR - 10.10.2016

Quote:

Originally Posted by WhiteKinG

PHP Code:



mysql_format(mysql, query, sizeof(query), "SELECT `BannedBy`, `BanReason`, `BannedDate`, `Username` FROM table_ban WHERE `IP`='%e' AND `Banned`='1'", RPIP(playerid));

you forget the table name

Thanks

Re: MySQL AND operator - Isolated - 10.10.2016

You also don't need to wrap integer values with quotations:

Code:

`Banned`='1'
// Becomes
`Banned`=1

Re: MySQL AND operator - NeXoR - 10.10.2016

Quote:

Originally Posted by Isolated

You also don't need to wrap integer values with quotations:

Code:

`Banned`='1'
// Becomes
`Banned`=1

Thanks, still doesn't work tho, no errors

PHP Code:



    mysql_format(mysql, query, sizeof(query), "SELECT `BannedBy`, `BanReason`, `BannedDate`, `Username` FROM `Users` WHERE `IP`='%e' AND `Banned`=1", RPIP(playerid));

    mysql_tquery(mysql, query, "CheckBan", "i", playerid);

PHP Code:



forward CheckBan(playerid);

public CheckBan(playerid)

{

    new rows, fields;

    cache_get_data(rows, fields, mysql);

    if(rows)

    {

        new banreason[128], bannedby[24], bandate[10], username[24], string[128];

        cache_get_field_content(0, "BannedBy", bannedby, mysql, sizeof(bannedby));

        cache_get_field_content(0, "BanReason", banreason, mysql, sizeof(banreason));

        cache_get_field_content(0, "BannedDate", bandate, mysql, sizeof(bandate));

        cache_get_field_content(0, "Username", username, mysql, sizeof(username));

        SendClientMessage(playerid, COLOR_GREEN, "-------[Banned]-------");

        format(string, sizeof(string), "Username: %s || Banning Date: %s", username, bandate);

        SendClientMessage(playerid, COLOR_RED, string);

        format(string, sizeof(string), "IP Address: %s || Banned By: %s", RPIP(playerid), bannedby);

        SendClientMessage(playerid, COLOR_RED, string);

        format(string, sizeof(string), "Reason: %s", banreason);

        SendClientMessage(playerid, COLOR_RED, string);

        SendClientMessage(playerid, COLOR_YELLOW, "If you think that you have been wrongfully banned, visit our forums and apply for an unban.");

        SendClientMessage(playerid, COLOR_GREEN, "-------[Banned]-------");

        SetTimerEx("DelayedKick", 1000, 0, "i", playerid);

    }

}

Code:

[02:47:54] [DEBUG] mysql_format - connection: 1, len: 248, format: "SELECT `BannedBy`, `BanReason`, `BannedDate`, `Username` FROM `Users` WHERE `IP`='%e' AND `Banned`=1"
[02:47:54] [DEBUG] mysql_tquery - connection: 1, query: "SELECT `BannedBy`, `BanReason`, `BannedDate`, `Username` FROM `U", callback: "CheckBan", format: "i"
[02:47:54] [DEBUG] CMySQLQuery::Execute[CheckBan] - starting query execution
[02:47:54] [DEBUG] CMySQLQuery::Execute[CheckBan] - query was successfully executed within 0.311 milliseconds
[02:47:54] [DEBUG] CMySQLResult::CMySQLResult() - constructor called
[02:47:54] [DEBUG] Calling callback "CheckBan"..
[02:47:54] [DEBUG] cache_get_data - connection: 1
[02:47:54] [DEBUG] CMySQLResult::~CMySQLResult() - deconstructor called

Re: MySQL AND operator - Alcatrik - 11.10.2016

Can you throw in a print(query)? I'm wondering whether mysql_format isn't actually formatting it properly - your query is executing successfully, which suggests the IP might be being fucked? I had a couple of issues a year or two back with mysql_format glitching out and not replacing %e because of something weird like array sizes.

Re: MySQL AND operator - NeXoR - 11.10.2016

Quote:

Originally Posted by Alcatrik

Can you throw in a print(query)? I'm wondering whether mysql_format isn't actually formatting it properly - your query is executing successfully, which suggests the IP might be being fucked? I had a couple of issues a year or two back with mysql_format glitching out and not replacing %e because of something weird like array sizes.

There you go

Code:

SELECT `BannedBy`, `BanReason`, `BannedDate`, `Username` FROM `Users` WHERE `IP`='127.0.0.1' AND `Banned`=1

Re: MySQL AND operator - iLearner - 11.10.2016

How about you select Banned value as well? And check if its 1 then send banned message otherwise ignre

Re: MySQL AND operator - NeXoR - 11.10.2016

Quote:

Originally Posted by iLearner

How about you select Banned value as well? And check if its 1 then send banned message otherwise ignre

The whole point is to check if the player is banned before I select everything, but I will try it.

Re: MySQL AND operator - NeXoR - 11.10.2016

Quote:

Originally Posted by iLearner

How about you select Banned value as well? And check if its 1 then send banned message otherwise ignre

Dumb me, I forgot to change "Banned" value to 1 for testing XD