Triggering Variables in other Weapons

[greggiles]

I am making a custom attack system and I swear I tested this before and it worked.

On my Movement 'weapon' I set a variable --- canmove = 1;
Pretty much this checks to see if the player is allowed to move

On my Attack 'weapon' the "canmove" is set to 0. Works

THEN I have a ScheduleEvent function that sets the 'canmove' to 1 again...but it doesn't set.

PHP Code:

//MOVEMENT SNIPPET
//#CLIENTSIDE
function onCreated()
{
 canatk = 1;
 canmove = 1;
 canjump = 1;
 disabledefmovement();
 onTimeOut();
} 


PHP Code:

//TEST WEAPON TO SET VARIABLES
//#CLIENTSIDE
function onKeyPressed(code, key)
{
 if (key == "v")
 {
  setAni("sword", NULL);
  canmove = 0;
  scheduleEvent(1, "onUnfreeze");
  }
  }
  
function onUnfreeze(){
player.chat = "unfreeze";
client.canmove = 1;
} 


Like I said, it will set it to 0, but wont return it to 1

[fowlplay4]

Tip: Don't use variables in multiple scripts. Use public functions instead:

Weapon: -Movement

PHP Code:

//#CLIENTSIDE

public function enableMovement() {
  this.canmove = true;
}

public function disableMovement() {
  this.canmove = false;
}

// use this.canmove in your system 


then in your weapons:

PHP Code:

//#CLIENTSIDE
function onKeyPressed(code, key) {
  if (key == "v") {
    ("-Movement").disableMovement();
    setani("sword", "");
    sleep(1);
    ("-Movement").enableMovement();
  }
} 


[cbk1994]

EDIT: Jer's response is right—I didn't pay attention to the variables—but if you want to support v5, keep the below in mind too.

Are you using v6? The only thing I see is that the scheduleEvent needs another parameter to support v5 and below. Functions like trigger, scheduleEvent, etc. all require an extra parameter for passing data, even if you don't want to use it. So just add another parameter (null will work fine) to the scheduleevent:

PHP Code:

scheduleEvent(1, "onUnFreeze", null); 


edit 2: on second reading it's clear the variables are at least part of the problem. You're using "canmove" in some cases and "client.canmove" in others. If you change them all to "canmove" you should be fine, but the real solution is to organize your code better (see above).

[greggiles]

edit 2: on second reading it's clear the variables are at least part of the problem. You're using "canmove" in some cases and "client.canmove" in others. If you change them all to "canmove" you should be fine, but the real solution is to organize your code better (see above).

The client.canmove was a mistype. sorry, that was never there in the original script

[Stowen]

Thanks Jer, I've been wondering how to use public functions. When I can give rep again, I'll make sure to hit you.