GANI scripts in attributes; updating

[cbk1994]

I'm having trouble with GANI scripts used in player attributes to show an HP bar. It works for the first display, but then doesn't always update.

This is the code (serverside):

PHP Code:

this.attr[3] = "era_hpbar.gani," @ (this.health / this.initialHealth); 


and this is the GANI script:

PHP Code:

SCRIPT
function onCreated() {
  showBar(params[0]);
}

function onPlayerEnters() {
  onCreated();
}

function showBar(percent) {
  with (findimg(200)) {
    image = "erai_hpbar.png";
    
    x = this.x + 0.25;
    y = this.y - 1.5;
    layer = 1;
    
    attachtoowner = true;
  }
  
  changeImgPart(200, 0, 0, 41, 11);
  
  with (findimg(201)) {
    image = "erai_hpbar.png";
    
    x = this.x + 0.25;
    y = this.y - 1.5;
    
    layer = 2;
    
    attachtoowner = true;
  }
  
  changeImgPart(201, 0, 12, max(1, 41 * percent), 11);
}
SCRIPTEND 


The bar will always show correctly if I try to display it when the bar isn't already showing, but if it is already showing, it won't update.

[Fulg0reSama]

I was thinking about how to fix this because i face a similar problem.

i was thinking like a timeout to readd while a onplayerhurt function would remove the attribute setting for the duration than add back afterwords Though im not sure if that could even work. Because i have a smaller project that requires a image to stay up at all times. Hope this gives you a lead onto a fix perhaps. If so give me the method and maybe i can work my own way around my problem :P

But of course im talking about how mine works... Good luck to your problem

[cbk1994]

What I ended up doing is just rotating between two attributes (2 and 3) instead of sticking with one, which lets it always update, but it's still a pain in the ass, and I think there's a way to make it work anyway

[Fulg0reSama]

not a bad loophole for this problem. good job. Hopefully someone has an answer to fixing this without having to use 2 attributes.

[DustyPorViva]

Sadly I've tried to force it to update, even by clearing the attr and setting it again, but to no avail. So I end up having to resort to using another attr to store the data, which sucks.

I've made a bug report on this update problem, but no response...

[Crow]

Originally Posted by DustyPorViva

Sadly I've tried to force it to update, even by clearing the attr and setting it again, but to no avail. So I end up having to resort to using another attr to store the data, which sucks.

I've made a bug report on this update problem, but no response...

Did you put a short sleep after clearing the attribute and before setting it again? As far as I remember, that did the trick before for me.

[DustyPorViva]

Originally Posted by Crow

Did you put a short sleep after clearing the attribute and before setting it again? As far as I remember, that did the trick before for me.

Can't put a sleep in timeouts

[Crow]

Originally Posted by DustyPorViva

Can't put a sleep in timeouts

scheduleEvent()

[DustyPorViva]

Originally Posted by Crow

scheduleEvent()

Keep forgetting about that! I really should utilize it more often... however I wish this bug was just crushed instead. Having a gani that's supposed to update constantly always running behind because we have to delay it before it updates isn't very ideal.

[Chompy]

Originally Posted by DustyPorViva

Can't put a sleep in timeouts

You can use sleep() as long as you restart the event (timeout = #; || setTimer()) (Which you should usually do at the end of the timeout event)

[DustyPorViva]

Originally Posted by Chompy

You can use sleep() as long as you restart the event (timeout = #; || setTimer()) (Which you should usually do at the end of the timeout event)

Not in a way that's healthy for the script, though. From my experience the sleep will function like a return, breaking everything after it, even if you reset the timeout.

[Fulg0reSama]

Interesting!

btw crow +rep just for your whole signature when I can do it again

[xXziroXx]

http://forums.graalonline.com/forums...ad.php?t=72413

[DustyPorViva]

Originally Posted by xXziroXx

http://forums.graalonline.com/forums...ad.php?t=72413

Indeed, but it's a bit silly to have one piece of functionality, like a health bar, rely on two attr's to work. I mean it's really the only way for it to function right now, but this needs to be addressed because it makes attr gani parameters pretty useless.

[xXziroXx]

Originally Posted by DustyPorViva

Indeed, but it's a bit silly to have one piece of functionality, like a health bar, rely on two attr's to work. I mean it's really the only way for it to function right now, but this needs to be addressed because it makes attr gani parameters pretty useless.

I think I linked the wrong thing, meant to link a reply in the thread:

http://forums.graalonline.com/forums...66&postcount=5