EVE-SECURE / eve-net

Automatically exported from code.google.com/p/eve-net
0 stars 0 forks source link

Dual purpose WalletJournal is not that #3

Open GoogleCodeExporter opened 9 years ago

GoogleCodeExporter commented 9 years ago
The constructor for WalletJournal does not allow for one to set an accountKey 
which is required for the corporation version to work.
http://wiki.eve-id.net/APIv2_Char_JournalEntries_XML
specifies an optional accountKey which for a character would always be 1000 but 
corporations have 7 accounts thus an option to set this is required.
It seems to be an easy fix. Just add an extra .Query(accountKey)

Otherwise a very nice API :)

Original issue reported on code.google.com by pvskaa...@gmail.com on 25 Jun 2013 at 10:29

GoogleCodeExporter commented 9 years ago
public void Query(string accountKey)
       {
           APIReader reader = new APIReader();
           reader.Query(Uri, this, "keyID={0}&vCode={1}&characterID={2}&accountKey={3}", keyID, vCode, actorID, accountKey);
       }

works and is an easy dirty fix.

Original comment by pvskaa...@gmail.com on 25 Jun 2013 at 10:33