TNT script not working

[PrinceOfKenshin]

Ok so the tnt will only work if you equip it then unequip it and say /tnt

PHP Code:

function onActionServerside()
{
if (params[1] == "use"){

  if (clientr.toolcount == NULL) clientr.toolcount = 0;
  this.ok = 0;

  for (i = 0; i < clientr.toolcount; i ++) {
    if (makevar("clientr.tool" @ i)[1] == "Tools/TNT") this.ok = 1;
  }
  if (this.ok == 0) return;
  if (!player.level.name.starts("at_mine")) return;
  serverr.tntlayer = player.account;
  putnpc2(player.x, player.y, "").join("tnt");

  this.temp = { 1, "Tools/TNT", "tnt","at_tnt-icon.png", "", "", 1,};
  if (clientr.toolcount == "") clientr.toolcount = 0;
  if (this.temp[5] == 1) {
    for (i = 0; i < clientr.toolcount; i ++) {
      if (makevar("clientr.tool" @ i)[1] == this.temp[1]) {
        if (makevar("clientr.tool" @ i)[4] == this.temp[4]) {
          makevar("clientr.tool" @ i)[0] -= 1;
          if (makevar("clientr.tool" @ i)[0] <= 0) {
            makevar("clientr.tool" @ i) = "";
            for (j = i; j <= clientr.toolcount; j++) {
              makevar("clientr.tool" @ j) = makevar("clientr.tool" @ j + 1);
            }
            clientr.toolcount--;
          }
        }
      }
    }
  }
}
}

//#CLIENTSIDE
function onCreated() setTimer(.05);

function onPlayerChats()
{
  if (player.chat == "/tnt") {
    if (this.timer <= 0) {
      triggeraction(0,0,"serverside",Tools/TNT,"use");
      this.timer = 2;
    } else player.chat = "You must wait " @ this.timer @ "seconds!";
  }
}


function onTimeout()
{
  if (this.timer > 0) this.timer -=.05;
  else if (this.timer < 0) this.timer = 0;
  
  setTimer(.05);
} 


[Googi]

When it doesn't work, does it not execute at all or does it do something like say that the player still has to wait when it shouldn't? From the looks of your script the this.temp[5] in the if statement should be this.temp[6].

[Chompy]

Originally Posted by Googi

When it doesn't work, does it not execute at all or does it do something like say that the player still has to wait when it shouldn't? From the looks of your script the this.temp[5] in the if statement should be this.temp[6].

Googi is right, change

PHP Code:

if (this.temp[5] == 1) { 


into

PHP Code:

if (this.temp[6] == 1) { 


[PrinceOfKenshin]

It still isn't working. It will work if you equip it then unequip it and reconnect. And even after you log off. I don't know it is confusing to me and this is the only script i need to fix for Atrius to get hosted and no one else on the staff team is helping

[Googi]

Originally Posted by PrinceOfKenshin

It still isn't working. It will work if you equip it then unequip it and reconnect. And even after you log off. I don't know it is confusing to me and this is the only script i need to fix for Atrius to get hosted and no one else on the staff team is helping

As I said before, tell us what you mean by "not working". Does it execute partially? Does it not execute at all? Which parts don't execute when they should? Debug by having the script produce something visual after each if statement.

[DustyPorViva]

Ya, the best way to fix scripts is to debug the crap out of them. Start by assuming what's the problem, where the script isn't working. Do this by echo'ing variables and other important information. Repeat this process until you figure out what isn't working correctly, what isn't doing what you want it to do, and change it.
Also:
triggeraction(0,0,"serverside",Tools/TNT,"use");
should be:
triggeraction(0,0,"serverside","Tools/TNT","use");
though it's preferred you do:
triggerserver("gui","Tools/TNT","use");

Also, it might be "/tnt", for some reason I have problems getting stuff like player.chat=="/..." to work, but if I remove the /, it works perfectly fine(I doubt this is the problem, but could be... that's what debugging is for). And going along with what everyone else has said, parameters, like array, start their index at 0. so basically subtract 1 from every params[x].