anti-freekill script troubles

[Hawt]

i'm currently making an anti-freekill script (hasn't been tested) and i am stuck on one part: how do I revive the freekilled player and ban the freekiller? code:


function servercmdExplain(%client, %Chat, %Chat2, %Chat3, %Chat4, %Chat5, %Chat6, %Chat7, %Chat8, %Chat9, %Chat10, %Chat11, %Chat12, %Chat13, %Chat14, %Chat15, %Chat16)
{
   if(%client.isFreekillSuspect == 0)
   {
       messageClient(%client,'MsgError',"\c6You are not a freekill suspect.");
       return;
   }

   if(%client.isAdmin == 1)
   {
      messageClient(%client,'',"\c6Admins cannot do /explain.");
      return;
   }

    %clientB = ClientGroup.getObject(%cl);
   if(%clientB.isAdmin)
   {
      messageClient(%clientb,'MsgAdminForce', "\c3" @ %client.name @ "\c6 explained: " @ %chat SPC %chat2 SPC %chat3 SPC %chat4 SPC %chat5 SPC %chat6 SPC %chat7 SPC %chat8 SPC %chat9 SPC %chat10 SPC %chat11 SPC %chat12 SPC %chat13 SPC %chat14 SPC %chat15 SPC %chat16 @ "");
   }

   messageClient(%client,'',"\c6You explained to the staff: " @ %chat SPC %chat2 SPC %chat3 SPC %chat4 SPC %chat5 SPC %chat6 SPC %chat7 SPC %chat8 SPC %chat9 SPC %chat10 SPC %chat11 SPC %chat12 SPC %chat13 SPC %chat14 SPC %chat15 SPC %chat16 @ "");
    %client.isFreekillSuspect == 0;
    cancel($freekillschedule);
}



function serverCmdRevive(%client, %target)
{
   if(!%client.isAdmin == 1)
   {
      messageClient(%client,'MsgError',"\c6You aren't an admin!");
      return;
   }

   if(isObject(%target))
   {
      messageClient(%client,'MsgError',"\c6Target is alive.");
       return;
   }
   
    %target = findClientByName(%target);

    %target.addLives(1);
    messageAll('',"\c6" @ %client.name @ " has revived " @ %target.name @ "\c6.");
}

function revive(%client, %target)
{
   %target = findClientByName

   if(isObject(%target)) return;

   %target.addLives(1);
}

function GameConnection::onDeath(%client, %killerObject, %killerClient, %damageType, %position)
{
   if(%killerClient.isAdmin == 1)
   {
      messageClient(%killerClient,'MsgAdminForce',"\c6You are an admin, so you do not need to do /explain. However, please do not freekill.");
      return;
   }

   if(%client.isWanted == 0)
   {
      messageClient(%killerClient,'MsgAdminForce',"<font:impact:35>PLEASE USE /EXPLAIN [REASON] TO EXPLAIN WHY YOU FREEKILLED<font:impact:20>You will be kicked in 30 seconds.");
      %client.isFreekillSuspect == 1;
      return;
   }

    messageClient(%client,'',"Please wait to be resurrected if you have been freekilled for a valid reason.");
    $freekillschedule = schedule(30000, 0, freeKillBan(%client));
    return parent::GameConnection::onDeath(%client, %killerobject, %killerClient, %damageType, %position);
}

function freekillBan(%client, %victimblid, %freekilledguy)
{
   %victimblid = findClientByBL_ID(%victimblid);

   commandtoserver('ban',0,%victimblid,10,"You have been banned for freekilling.");
   

}

function wantedStatusLoop(%client)
{
   cancel($WantedStatusGV);
    if(!%client.tdmTeam == 0) return;
    if(%client.isWanted == 0) return;
    centerPrintAll("                                                                                 \c6Status: <color:ff0000>Wanted");
    $WantedStatusGV = schedule(10,0,wantedStatusLoop);
}

package AFKD
{
   function Slayer_MinigameSO::onReset(%mini, %client)  //stolen from Tetro Block's freekill detecter script, credit to him.
   {
          for (%i = 0; %i < clientGroup.getCount(); %i++)
       {
          %client = clientGroup.getObject(%i);
          %client.isWanted = 0;
       }

       return parent::onReset(%mini, %client);
    }
};
activatePackage(AFKD);



am i able to do something like schedule(30000, 0, freekillBan(%arg, %arg2));

[Ad Bot]

[Greek2me]

Instead of reinventing the wheel with your /revive command, just use Slayer's existing /addLives command:

/addlives 1 Greek2me

It will automatically revive them if they're dead.


Also, there's a problem in this code:

Code: [Select]

package AFKD
{
   function Slayer_MinigameSO::onReset(%mini, %client)  //stolen from Tetro Block's freekill detecter script, credit to him.
   {
          for (%i = 0; %i < clientGroup.getCount(); %i++)
       {
          %client = clientGroup.getObject(%i);
          %client.isWanted = 0;
       }

       return parent::onReset(%mini, %client);
    }
};
activatePackage(AFKD);

In your loop, you should use a different variable than %client so that the variable passed to the parent isn't overwritten.

edit: Another problem: you're using commandToServer in server-sided code. Use serverCmdCommandName instead.

[Hawt]

done, now i need an answer to the real problem

[Greek2me]

What's the "real problem"? I covered everything in the OP.

[Hawt]

am i able to do something like schedule(30000, 0, freekillBan(%arg, %arg2));

[Amade]

Do this instead
schedule(30000, 0, freekillBan, %arg, %arg2);

[Hawt]

thx amde and greek, locking