Ban System geht nicht

Saphire · 28. Januar 2015

Jeffry:
Mir war nicht bewusst das es so groß wäre, dachte das geht schnell.
Egal dann machen wir das halt so :>

Hallo, irgendwie geht mein Ban befehl nicht.

Das ist der Befehl:

Wenn er sich einloggt soll er gekickt werden :

Bei OnPlayerLogin:

if(PlayerInfo[playerid][pGesperrt] == 1)
{
format(string2,sizeof(string2)," %s versucht mit seinem gebannten Account online zu kommen und wurde gekickt!",playername2);
ABroadCast(COLOR_LIGHTRED,string2,1);
SendClientMessage(playerid,COLOR_ROT," Du bist hier gebannt. Schau im Forum für mehr Informationen");
Kick(playerid);
return 1;
}

So lese ich das aus bei OnPlayerLogin:
PlayerInfo[playerid][pGesperrt] = dini_Int(string2, "Gesperrt");

Aber ich kann mich trotzdem wieder einloggen.
Warum das ?

whitetiiger · 28. Januar 2015

Hast du auch die Abfrage unterm Auslesen?

MoD · 28. Januar 2015

Zeig mal deinen KickTimer, wo du den Spieler dann nachdem er die Ban-Nachricht erhalten hat bannst bzw. kickst

Saphire · 28. Januar 2015

MoD

forward KickTimer(playerid);
new oKick = 0;

public KickTimer(playerid)
{
if(oKick == 0)
{
SetTimerEx("KickTimer",200,false,"i",playerid);
oKick = 1;
}
else if(oKick == 1)
{
Kick(playerid);
oKick = 0;
}
return 1;
}

Das ist der

#edit ich glaub ich habs gefunden :>
Wenn ich mich banne, steht in der .ini nicht Gesperrt = 1
Hab unter OnPlayerUpdate das gefunden.

format(var, 32, "Gesperrt=%d\n",PlayerInfo[playerid][pGesperrt]);fwrite(hFile, var);
Jeffry: hat mir das mal gesagt.

Jeffry · 28. Januar 2015

Das steht bei OnPlayerUpdate, steht aber nicht in der Datei?
Schreibe mal nach
PlayerInfo[giveplayerid][pGesperrt] = 1;
Im Ban Befehl
OnPlayerUpdate(playerid);
hin.

Saphire · 28. Januar 2015

Jeffry:

Hab das so im Ban befehl

SendClientMessageToAll(COLOR_LIGHTRED, string);
format(string, sizeof(string), "AdmCmd: Du wurdest von %s gebannt, Grund: %s", sendername, (result));
SendClientMessage(giveplayerid,COLOR_LIGHTRED, string);
PlayerInfo[giveplayerid][pGesperrt] = 1;
OnPlayerUpdate(playerid);
return 1;

Hat aber nichts gebracht.

mfg

Jeffry · 28. Januar 2015

Dann poste bitte dein OnPlayerUpdate und dein OnPlayerLogin. Beide Codes.

Saphire · 28. Januar 2015

Jeffry:

Spoiler anzeigen

und das hier :

Spoiler anzeigen

Jeffry · 29. Januar 2015

Das ist soweit alles richtig, das gesperrt muss auch in der Datei stehen, du schreibst es ja rein. Ob es da mit 0 oder 1 steht ist was anderes, aber es muss drin stehen, schau da mal nochmal nach.

Steht die Abfrage, ob der Spieler gebannt ist direkt nach dem Laden der Spieler Stats, oder wo im Code steht das? Poste das bitte mal zusammenhängend, weil wenn da zuvor zum Beispiel ein return ist, dann würde es nichts bringen.

Saphire · 29. Januar 2015

Jeffry:
Ja Gesperrt ist in der Datei, ich meinte oben aber das da "1" nicht steht sondern 0.

Und hier OnPlayerLogin:

Spoiler anzeigen

Jeffry · 29. Januar 2015

Schreibe es mal so:
SendClientMessageToAll(COLOR_LIGHTRED, string);
format(string, sizeof(string), "AdmCmd: Du wurdest von %s gebannt, Grund: %s", sendername, (result));
SendClientMessage(giveplayerid,COLOR_LIGHTRED, string);
PlayerInfo[giveplayerid][pGesperrt] = 1;
OnPlayerUpdate(giveplayerid); //Hier hatte ich dir die falsche Variable gesagt, sorry.
return 1;

Und das musst du mit einem Array machen, sonst bringt das nichts.
Du könntest aber auch einfach die Kick/Ban-Fix Include nutzen.
forward KickTimer(playerid);
new oKick[MAX_PLAYERS];

public KickTimer(playerid)
{
if(oKick[playerid] == 0)
{
SetTimerEx("KickTimer",200,false,"i",playerid);
oKick[playerid] = 1;
}
else if(oKick[playerid] == 1)
{
Kick(playerid);
oKick[playerid] = 0;
}
return 1;
}

Saphire · 29. Januar 2015

Jeffry:
Also bringt er Kick Timer nichts oder wie ?

und ich schau mir mal das Include an

Mfg

#edit
Hab da noch was komisches.
Wenn ich /ban und meine ID dann steht da in der Nachricht an alle "1 wurde von ... gebannt" Also da steht meine Player ID und nicht mein Name

Jeffry · 29. Januar 2015

Doch, so wie ich es gemacht habe bringt er was, sofern du das wieder zu 0 setzt bei OnPlayerDisconnect. Die Include ist trotzdem besser.

Poste den Befehl bitte.

Saphire · 29. Januar 2015

Jeffry:

if(strcmp(cmd, "/ban", true) == 0)
{
if(IsPlayerConnected(playerid))
{
tmp = strtok(cmdtext, idx);
if(!strlen(tmp))
{
SendClientMessage(playerid, COLOR_GRAD2, "FEHLER: /ban [playerid] [grund]");
return 1;
}
giveplayerid = ReturnUser(tmp);
if (PlayerInfo[playerid][pAdmin] >= 1)
{
if(IsPlayerConnected(giveplayerid))
{
if(giveplayerid != INVALID_PLAYER_ID)
{
GetPlayerName(giveplayerid, giveplayer, sizeof(giveplayer));
GetPlayerName(playerid, sendername, sizeof(sendername));
new length = strlen(cmdtext);
while ((idx < length) && (cmdtext[idx] <= ' '))
{
idx++;
}
new offset = idx;
new result[64];
while ((idx < length) && ((idx - offset) < (sizeof(result) - 1)))
{
result[idx - offset] = cmdtext[idx];
idx++;
}
result[idx - offset] = EOS;
if(!strlen(result))
{
SendClientMessage(playerid, COLOR_GRAD2, "FEHLER: /ban [playerid] [grund]");
return 1;
}
if(PlayerInfo[giveplayerid][pAdmin] > PlayerInfo[playerid][pAdmin] > PlayerInfo[playerid][pSupporter])
{
SendClientMessage(playerid, COLOR_GRAD2, "Du kannst diesen Spieler nicht bannen, da er ein höheres Adminlevel hat als du!");
return 1;
}
new year, month,day;
getdate(year, month, day);
KickTimer(giveplayerid);
format(string, sizeof(string), "AdmCmd: %s wurde von %s gebannt, Grund: %s (%d-%d-%d)", giveplayerid, sendername, (result),month,day,year);
BanLog(string);
format(string, sizeof(string), "AdmCmd: %s wurde von %s gebannt, Grund: %s", giveplayerid, sendername, (result));
SendClientMessageToAll(COLOR_LIGHTRED, string);
format(string, sizeof(string), "AdmCmd: Du wurdest von %s gebannt, Grund: %s", sendername, (result));
SendClientMessage(giveplayerid,COLOR_LIGHTRED, string);
PlayerInfo[giveplayerid][pGesperrt] = 1;
OnPlayerUpdate(giveplayerid); //Hier hatte ich dir die falsche Variable gesagt, sorry.
return 1;
}
}//not connected
}
else
{
format(string, sizeof(string), " %d ist kein aktiver Spieler.", giveplayerid);
SendClientMessage(playerid, COLOR_GRAD1, string);
}
}
return 1;
}

Ich habe mich ebend selber gebannt, jedoch war gesperrt trotzdem auf 0..

Jeffry · 29. Januar 2015

format(string, sizeof(string), "AdmCmd: %s wurde von %s gebannt, Grund: %s (%d-%d-%d)", giveplayerid, sendername, (result),month,day,year);
zu:
format(string, sizeof(string), "AdmCmd: %s wurde von %s gebannt, Grund: %s (%d-%d-%d)", giveplayer, sendername, (result),month,day,year);

Ebenso bei der anderen Nachricht.

Ändere bei OnPlayerUpdate mal:
format(var, 32, "Gesperrt=%d\n",PlayerInfo[playerid][pGesperrt]);fwrite(hFile, var);
zu:
format(var, 32, "Gesperrt=%d\n",PlayerInfo[playerid][pGesperrt]);fwrite(hFile, var);printf("var: %s", var);

Was wird geprintet, wenn du den Spieler bannst?

Saphire · 29. Januar 2015

Jeffry:
Schau das bleibt auf 0..

[20:42:52] [join] Diamond has joined the server (1:127.0.0.1)
[20:42:59] Diamond has logged in.
[20:43:27] [part] Diamond has left the server (1:2)
[20:43:34] var: Gesperrt=0

Jeffry · 29. Januar 2015

Wieso kommt der print 7 Sekunden nachdem du den Server verlassen hast??

Setze dann auch mal hier noch einen print:
SendClientMessageToAll(COLOR_LIGHTRED, string);
format(string, sizeof(string), "AdmCmd: Du wurdest von %s gebannt, Grund: %s", sendername, (result));
SendClientMessage(giveplayerid,COLOR_LIGHTRED, string);
printf("Banned: %d", giveplayerid);
PlayerInfo[giveplayerid][pGesperrt] = 1;
printf("gesperrt: %d",PlayerInfo[giveplayerid][pGesperrt]);
OnPlayerUpdate(giveplayerid);
return 1;

Saphire · 29. Januar 2015

Jeffry:

Hier :>
Ich habe das bisschen länger offen gelassen und der schreibt das öfter rein.
Ich weis nicht ob das gewollt ist aber das ist ja egal.

"[21:07:40] [join] Diamond has joined the server (1:127.0.0.1)
[21:07:47] Diamond has logged in.
[21:08:05] Banned: 1
[21:08:05] gesperrt: 1
[21:08:05] [part] Diamond has left the server (1:2)
[21:08:22] var: Gesperrt=0

[21:09:26] var: Gesperrt=0"

Jeffry · 29. Januar 2015

Das liegt daran, dass ID 0 ja noch im Server ist.
Schreibe den print so:
format(var, 32, "Gesperrt=%d\n",PlayerInfo[playerid][pGesperrt]);fwrite(hFile, var);printf("var: %s | playerid: %d", var, playerid);

Dann poste nur die Zeilen entsprechend der playerid die du gebannt hast. Das dürften nach dem Ban nicht all zu viele sein.

Beitrag von MoD (28. Januar 2015)