Amplify Magic Power not showing information in the correct moment

Originally posted by emong
I've also experienced this problem..

attached in this reply is the screenshots of the said bug/glitch..

Originally posted by ngek202
Made a video., notice this before:

Originally posted by xRaisen
Its working though its not displaying.

Originally posted by Vali

Its working though its not displaying.

That is what i tried to say in my first post, yes.

Originally posted by Wildcard
True, I neglected to test this particular aspect when rewriting the skill. It turns out official servers don't display a matk bonus at all, which is a bit ugly to implement with the current code. Will most likely be done when someone gets around to revisiting mATK.

Originally posted by karazu
confirmed it in my server

Originally posted by malufett
I got a question...

since the status window is change therefore we can't see anymore min and max matk only the status atk that is why we can't see the changes..

so now if it change on the status window therefore it boost the status matk???

we need confirmation/more information for this....

Originally posted by RaGERO
I have this problem on my server too.

High Wiz uses Amplify skill, checks status window and it doesn't display the correct additional matk, High wiz casted storm gust, now the status window displays the correct matk with amplify.

Using client 2010-07-30.

Originally posted by malufett
I check in official it doesn't display in the status window since it only modify matk and not status matk..

Originally posted by MarkZD

I check in official it doesn't display in the status window since it only modify matk and not status matk..

Wildcard, already said it just some posts above.

I think you didn't get this report, it's saying it's beeing displayed after you cast some magic, after AMP.

I tested in 16355 and it's still displaying after you casta magical attack after AMP.

Edited by MarkZD, 29 June 2012 - 09:51 PM.

Originally posted by malufett
oppps...sorry my mistake..=p

ok I found out where is the problem...because ATM the status matk is share with min matk variable...
but once I'm done to recode the RE matk system this will be fixed soon...

but here is a temporary fix however this is not the right way to do it..

if (sc && sc->count && sc->data[SC_MAGICPOWER])
{
	 if (sc->data[SC_MAGICPOWER]->val4)
	 {
		  status_change_end(bl, SC_MAGICPOWER, INVALID_TIMER);
	 }
	 else
	 {
		  sc->data[SC_MAGICPOWER]->val4 = 1;
         -status_calc_bl(bl, status_sc2scb_flag(SC_MAGICPOWER));
	 }
     + status_calc_bl(bl, status_sc2scb_flag(SC_MAGICPOWER))
}

Originally posted by Vali
2 months bump.

Originally posted by cataclysmro
any news?

Originally posted by Lighta
if you do that malufet the bonus won't be show anyway as calc_matk expecting val4 to be true to make the change :
if(sc->data[SC_MAGICPOWER] && sc->data[SC_MAGICPOWER]->val4)

So basically you asking to reparse calc_matk without making anychange in it, plus if it was true you'll have the bonus right away (like just after amp), instead right now you have the status after AMP but get the bonus just after caster a valid skill.
In short that workarround wont work and would break that fonction purpose if so :

For the display info well there directly link to status point so there nothing you can do about it or you may want to add some exception in clif_updatestatus but that would be ugly.

Originally posted by CairoLee
I have this problem on my server too.

any news?

Originally posted by Lighta
ok I'll reexplain that in a easier way.
-right now status windlay display real matk (with status and such)
-AMP matk bonus is only active while val4 is true. (like when casting skill)
-Once skill is ended amp status is ended (so amp bonus disapear)

Now you have 2 choice either you do a small hack to display amp bonus even if it's not active yet. ->clif.c
or you add the bonus right after the status is casted->2

for the choice 2 and it's more or less what malufett was proposing,
(go in status.c and found "if(sc->data[SC_MAGICPOWER] && sc->data[SC_MAGICPOWER]->val4)", and replace with "if(sc->data[SC_MAGICPOWER]" )
now your bonus will be apply once the status is finished, you have the good display but you kinda broke hte purpose of the function..

Originally posted by malufett
or better to recode the matk system..ATM I already had the RE MATK(including the fix w/ this) 100% working with PVP however matk with mobs is still unknown and I'm still in a test to determine the formula..

Originally posted by CairoLee

go in status.c and found "if(sc->data[SC_MAGICPOWER] && sc->data[SC_MAGICPOWER]->val4)", and replace with "if(sc->data[SC_MAGICPOWER]"

I try to use this solution,but the when i do the test,the mapserver was carsh. the msg is :

[Error]: Server received crash signal! Attempting to save all online characters!

===============================================

Then I try the another solution, by myself (i am a super newbie)

in status.c , the status_calc_bl_ function:

• found "if(b_status.matk_max != status->matk_max)" and replace whit "if((b_status.matk_max != status->matk_max) && !(status_get_sc(bl)->data[SC_MAGICPOWER]))"
• found "if(b_status.matk_min != status->matk_min)" and replace whit "if((b_status.matk_max != status->matk_max) && !(status_get_sc(bl)->data[SC_MAGICPOWER]))"

i do the test, it's worked.

but !! when you in the SC_MAGICPOWER ‘s Duration,try to uses something will change MATK in the status panel,then the status panel will have a error display..

Originally posted by Arcenciel
Confirmed. Just noticed this today.

Originally posted by malufett
Fixed @ [rev='16981']