Pages: 1 [2] :: one page |
|
Author |
Thread Statistics | Show CCP posts - 0 post(s) |
Fiat Money
Perkone Caldari State
6
|
Posted - 2012.08.27 21:21:00 -
[31] - Quote
Hi,
I'm about to combine different API classes, such as the Alliance class with the CharacterSheet and noticed you are using different datatypes for allianceID. The CharacterSheet class provides UInt64, but the Alliance itself Int64. Datatypes shall be standardized to ease comparison by Id. |
Fiat Money
Perkone Caldari State
6
|
Posted - 2012.08.28 20:34:00 -
[32] - Quote
Another thing.
Found a bug with the CorporationSheet that is not loading at all. Loading from API results into "Illegal page request", but I have all priviliges set. I've tried with your EVE.Net client. |
Desmont McCallock
216
|
Posted - 2012.08.28 20:40:00 -
[33] - Quote
ItsmeHcK1 wrote:Will do a check as soon as I can. ^^ Thanks in advance!
Edit: Seems to be fixed indeed.
Ninja-edit: The accessmask in APIKeyInfo can no longer be an int, as CCP have just added another flag, making it larger than an int can handle. (check the API key generator thingy, a flag for Locations has just been added, bringing the total to 268435455) Locations API call has been added a long time ago from CCP PrismX. Just FYI. Edit: It's advised to check with http://wiki.eve-id.net/APIv2_Page_Index |
ItsmeHcK1
Kicked. Shadow Cartel
22
|
Posted - 2012.09.04 21:48:00 -
[34] - Quote
Hmm, good point... Well, Visual Studio threw an error saying it was too big, so... vOv I was too lazy to check. :P
Anyhoo, here's another one... For the CharacterInfo API, the keyid and vcode are optional, but there is no constructor that only asks for the character id, nor a query override that deals with that.
I simply changed the Query override into this: if (String.IsNullOrEmpty(keyID)) reader.Query(Uri, this, "characterID={0}", actorID);
And added a constructor: public CharacterInfo(string actorid) : base("", "", actorid) { } |
ItsmeHcK1
Burning Napalm Northern Coalition.
22
|
Posted - 2012.09.06 14:33:00 -
[35] - Quote
Same goes for the corporationsheet one; an API key is not required, but the class is not set up for that eventuality. |
ItsmeHcK1
Burning Napalm Northern Coalition.
24
|
Posted - 2012.10.10 15:14:00 -
[36] - Quote
Are you still actively developing on this? Or do you consider it completed?
Trying to decide whether I should modify it a bit. :P (error handling, for example) |
Fenian
Ancient Artifacts
1
|
Posted - 2012.10.18 11:53:00 -
[37] - Quote
I notice in this code you dont handle CorpwalletsJournals Correct. I have no way to pass what WalletID i want to get results back for. You put it in with the Chracter Wallet journal and thats fine but thing you could fix it up to handle the corp wallet it. |
|
|
|
Pages: 1 [2] :: one page |
First page | Previous page | Next page | Last page |