Ban

[FarisDon]

Code Select Expand

function CheckBan( p )
{
           local q = QuerySQL( db, "SELECT Name,IP,UID FROM Bans WHERE Name='" + p + "'" );
   local  Name = GetSQLColumnData( q, 0 ), IP = GetSQLColumnData( q, 1 ) , UID = GetSQLColumnData( q, 2);
   if ( ( p.Name ) || ( p.IP)  || ( p.UID ))  return 1;
   else return 0;}
   

Code Select Expand

if ( CheckBan(player)=="1")Ban(player); // on server joins function
    Well whenever ._. i join the server it kicks me even if my name is not in the database i had made this specially for vpn ban , and ip ban , but it always kick me.

[DizzasTeR]

Why are you comparing a bool value with string? it will never work, and secondly why are you re-banning a banned player? Try

Code Select Expand

if ( CheckBan( player ) ) KickPlayer( player );


[FarisDon]

Why are you comparing a bool value with string? it will never work, and secondly why are you re-banning a banned player? Try

Code Select Expand

if ( CheckBan( player ) ) KickPlayer( player );


OH Dam i seriously forgot that ; Thank you Doom , and Eid mubarak.
Edit:Nope still it is banning me even there is no record in the Database.

[[VSS]Shawn]

try remove from db bam and try and eid mubarak FarisDon

[FarisDon]

try remove from db bam and try and eid mubarak FarisDon

I can't understand what you mean?

[.]

try remove from db bam and try and eid mubarak FarisDon

Damn dude. You must be truly retarded Talking with your self and making no sense overall.

[Thijn]

Well no fucking shit it's banning you. You check if p.Name exists, if it does you get banned. Of course it exists >.<
You gotta rewrite that if statement.

[FarisDon]

Well no fucking shit it's banning you. You check if p.Name exists, if it does you get banned. Of course it exists >.<
You gotta rewrite that if statement.

._. Eh thijn Which If statement there is only one that you mean checkban statement , but to re-write it like ?

[Thijn]

Well no fucking shit it's banning you. You check if p.Name exists, if it does you get banned. Of course it exists >.<
You gotta rewrite that if statement.

._. Eh thijn Which If statement there is only one that you mean checkban statement , but to change it like what?

You wanna check if someone is banned, not if the player has a name. You got the query, now do something with it

[DizzasTeR]

Ah yes of-course nice catch @Thijn.

So yeah Axel, I just noticed the mistake after reading Thijn's reply, you have to change this line.

Code Select Expand

if ( ( p.Name ) || ( p.IP)  || ( p.UID ))  return 1;
to

Code Select Expand

if ( ( Name ) || ( IP)  || ( UID ))  return 1;

Because you don't need to check if the plaer's name, IP or UID exists, ofcourse they exist, you have to check the result of the query you are getting ( Name, IP, UID ) in that case, if the queries finds any of those data from the given name ( p ) then it will kick him otherwise it won't.

[Thijn]

Removed 2 useless replies. The solution has been posted.

[FarisDon]

Removed 2 useless replies. The solution has been posted.

Thijn is a mastermind guy he should be the one to create 0.5

Ah yes of-course nice catch @Thijn.

So yeah Axel, I just noticed the mistake after reading Thijn's reply, you have to change this line.

Code Select Expand

if ( ( p.Name ) || ( p.IP)  || ( p.UID ))  return 1;
to

Code Select Expand

if ( ( Name ) || ( IP)  || ( UID ))  return 1;

Because you don't need to check if the plaer's name, IP or UID exists, ofcourse they exist, you have to check the result of the query you are getting ( Name, IP, UID ) in that case, if the queries finds any of those data from the given name ( p ) then it will kick him otherwise it won't.

Thanks i got it .