[solved] autospell mammonite should not casted while 0 zeny

[jamesandrew]

This script is not working. I'm still getting auto spell mammo even though my zeny is 0
Is there an item script to check player zeny?

Edited September 4, 2023 by jamesandrew
solved

[Winterfox]

You should write a git issue to make the rathena team aware of that. https://github.com/rathena/rathena/issues

Edited August 28, 2023 by Winterfox

[xJohn]

 

14 hours ago, jamesandrew said:

This script is not working. I'm still getting auto spell mammo even though my zeny is 0
Is there an item script to check player zeny?

5609,Chung_Hairband,Chung Hairband,4,0,,500,,5,,1,0xFFFFFFFF,63,2,256,,60,0,583,{ bonus bLuk,3; bonus bMdef,4; /* bonus3 bAutoSpellWhenHit,"MC_MAMMONITE",5,5; */ if(zeny >=500) { bonus3 bAutoSpellWhenHit,"MC_MAMMONITE",5,5; } },{},{}

5573,Bogy_Horn,Dokebi Horn,4,20,,100,,1,,1,0xFFFFFFFF,63,2,256,,75,1,554,{ bonus2 bAddMonsterDropItemGroup,IG_Jewel,100; /* bonus3 bAutoSpell,"MC_MAMMONITE",5,70; */ if(zeny >=500) { bonus3 bAutoSpell,"MC_MAMMONITE",5,70; } },{},{}

Edited August 28, 2023 by xJohn

[Winterfox]

4 hours ago, xJohn said:

 

5609,Chung_Hairband,Chung Hairband,4,0,,500,,5,,1,0xFFFFFFFF,63,2,256,,60,0,583,{ bonus bLuk,3; bonus bMdef,4; /* bonus3 bAutoSpellWhenHit,"MC_MAMMONITE",5,5; */ if(zeny >=500) { bonus3 bAutoSpellWhenHit,"MC_MAMMONITE",5,5; } },{},{}

5573,Bogy_Horn,Dokebi Horn,4,20,,100,,1,,1,0xFFFFFFFF,63,2,256,,75,1,554,{ bonus2 bAddMonsterDropItemGroup,IG_Jewel,100; /* bonus3 bAutoSpell,"MC_MAMMONITE",5,70; */ if(zeny >=500) { bonus3 bAutoSpell,"MC_MAMMONITE",5,70; } },{},{}

This won't work as expected. The script is only run on onequip. So, the code you provide leads to the bonus3 only to be applied when the player has 500 zeny or more at the time he equips the item, but it doesn't check it on every autospell trigger.

That means if mammonite consumes the last 500 zeny the following casts will still be done, since the check doesn't happen at the time of the casting.

The solution would be either to instead of just providing a skill name, you could provide a script on each trigger like autobonus works or a parameter to tell the function it should check skill requirements before casting it.
But those are things that need a source change.

What could work but is kinda messy would be this:

autobonus2("{ if(zeny >=500) bonus3(bAutoSpellWhenHit,\"MC_MAMMONITE\",5,5); }", 1000, 1000);

I didn't test it but what it should do in theory is to check on each hit if the player has at least 500 zeny and if he does, apply the bonus for 1 second to check if it should trigger mammonite according to its chance.
But this is very speculative. It could easily be that the bonus3 doesn't work on the attack that triggered the attachment via autobonus, but instead only on the successive attacks because of the way the state flow is organized.

Edited August 28, 2023 by Winterfox

[jamesandrew]

Hey thanks for the input guys. I appreciate it.
@Winterfox awesome, your script is working. I've just tested it.