Wrong GP values after a few undos including decay

[GoogleCodeExporter]

What steps will reproduce the problem?
1. I made about 100 undos including some decays to fix the "too high pr"
issue we had (gave ep but no gp in random raid)
2. Now some chars (even lev 40 who get no ep or gp) have wrong GP values
(some have 110 not 100 as set for base value)

What version of the product are you using? What locale is your client (en,
ru, kr, ch, es, tw)? What other addons do you have installed and what
versions?
5.4.1

Paste the text surrounded by -EPGP- in Guild Information here:
-EPGP-
@BASE_GP:100
@DECAY_P:10
@MIN_EP:5000
-EPGP-

Original issue reported on code.google.com by the.bl4c...@gmail.com on 5 Apr 2009 at 3:32

[GoogleCodeExporter]

Did you by any change get any error message like:

        string.format("Ignoring attempt to set note before flushing pending "..
                      "note for %s! "..
                      "current=[%s] pending=[%s] new[%s]. "..
                      "Please report this bug along with the actions that "..
                      "lead to this on http://epgp.googlecode.com",

To your chat frame?

Also can you give some clarification:  the lvl 40 had 0 EP and 100 GP before 
and after the undos they ended up 
with 110?

Original comment by evlogimenos on 5 Apr 2009 at 4:43

• Changed state: Accepted
• Added labels: Module-Log, Version-5

[GoogleCodeExporter]

hm I could not see a error message in the chat.
And right, the lvl40 had 0 ep , 100 base gp and now has 0 ep and 110 gp (like 
many
other members (but not all) he got 10 gp)

Original comment by the.bl4c...@gmail.com on 5 Apr 2009 at 5:58

[GoogleCodeExporter]

Duh, I found the issue. The log has the actual amount attempted to be decayed 
but not the actual amount 
decayed.

Put more simple: Member has 100GP. Decay of 10% is -10GP. 100GP - 10GP = 100GP 
since BaseGP is 100GP. 
Entry in the log says -10GP so if you undo that, you get +10GP.

Original comment by evlogimenos on 11 Apr 2009 at 12:49

[GoogleCodeExporter]

This is fixed in r1187.

Original comment by evlogimenos on 16 Apr 2009 at 8:55

• Changed state: Fixed