These forums have been archived and are now read-only.

The new forums are live and can be found at https://forums.eveonline.com/

EVE Technology Lab

 
  • Topic is locked indefinitely.
 

API bug introduced in Crucible - Wallet journal shows wrong party's event

First post
Author
Previous Topic Next Topic
Ydnari
Estrale Frontiers
#1 - 2011-11-30 20:26:24 UTC
Nasty wallet journal API bug has been introduced in Crucible, I have bug reported as bug report ID 120370.

I'm seeing intermittent entries (5 or 6 out of several hundred) in both corp and personal WalletJournal data where an entry is actually the other party's wallet journal entry.

For example:


  • Corp wallet, taxed mission bounty - instead of the 1% taxed amount, it is the 99% remainder that went to the player, and the balance is the balance of the player at the time, not the resulting corp wallet balance.

  • Player trading on personal wallet journal - amount is zero as the trade did not include ISK, but the balance field is the balance of the player traded to.

  • Corp wallet to wallet transfer - amount in one wallet's API download shows a negative amount for the transfer, but the destination wallet journal entry also shows negative, instead of positive as it received the ISK.



In all cases the ISK amounts show in Journal in game are correct, but the API data is not.

I haven't seen this before yesterday - in particular if it had happened with bounties it would be very obvious as I run a "leader board" for the mission runners in our corp based on the corp wallet taxed bounty data and these few rogue entries are showing up as massive spikes.

--
CCP Prism X
C C P
C C P Alliance
#2 - 2011-12-01 11:49:39 UTC
Looking into this as this sounds pretty serious.
Seems that the wallet API saw a lot of changes during my absence from the project so it's very likely that some very elusive bug was introduced.

CCP Prism X

Programmer on Team Gridlock & Expert Ranter
Desmont McCallock
#3 - 2011-12-01 11:55:42 UTC  |  Edited by: Desmont McCallock
Without wanting to undermine CCP Stillman or CCP Elerhino, I think the API needs a "rasta" touch. Smile
CCP Prism X
C C P
C C P Alliance
#4 - 2011-12-01 12:13:10 UTC
Not to rain on my own parade but Elerhino would probably have this solved by now with time to spare for a hole or nine (except it's snowing here... not sure Extreme Golfing has made it to Iceland yet). I'm not nearly enough as experienced with the .NET framework as he is. Blink

Also, The Rasta touch doesn't always improve things. Lol

CCP Prism X

Programmer on Team Gridlock & Expert Ranter
Ydnari
Estrale Frontiers
#5 - 2011-12-01 21:19:59 UTC
Nice one, thanks. Bonus points if it's fixed in time so that I can just wipe out the bad data in my database and reload it from API Big smile

--
Nagapito
The Scope
Gallente Federation
#6 - 2011-12-02 12:57:44 UTC  |  Edited by: Nagapito
Sorry Prism but problem is not yet solved... At least for personal wallet journal.
I keep seeing very different balance values in my wallet. I'm not sure but I would say that one of the balance values is actually the balance of my friend that transfered money to me!

I have several journal entries while ratting with my correct balance and suddenly my balance drop a couple of bill, then raises a couple of hundreds and its never my correct balance!

Edit: Also, I only have the entries from the bounties paid by Concord but I dont have the entries for the taxes paid to the corp
CCP Prism X
C C P
C C P Alliance
#7 - 2011-12-02 14:05:28 UTC
I never said it was solved. I'm currently trying to break all the proper procedures to have the fix ported out to the testAPI for verification upon which I'll try to break all the proper procedures again and have it pushed to TQ. Pirate

CCP Prism X

Programmer on Team Gridlock & Expert Ranter
Nagapito
The Scope
Gallente Federation
#8 - 2011-12-02 14:22:00 UTC  |  Edited by: Nagapito
CCP Prism X wrote:
Not to rain on my own parade but Elerhino would probably have this solved by now(....)

If anything in this post was informative or could be considered as 'good news' to you - chances are you've misread it.


Only now I noticed your signature....P
CCP Prism X
C C P
C C P Alliance
#9 - 2011-12-02 15:37:24 UTC
Cowboyed to the test server, cowboyed from there over to Tranquility. Should be fixed now!
Off to hunt me some Indians!

CCP Prism X

Programmer on Team Gridlock & Expert Ranter
Ydnari
Estrale Frontiers
#10 - 2011-12-02 18:46:16 UTC
Awesome Big smile

Data reloaded, looks good! Bonus points awarded!

--