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.
 

The King is dead, long live the new King! (API switched ownerships!)

First post
Author
Chaotic Mind
Wet Corpses
#101 - 2014-01-19 17:59:18 UTC
CCP FoxFour wrote:
I hate you all very much... as in very very much... as in KAJ GHSBF&P*(UI#WOSDHBUI&*(SEFVDD&*UIFGHIKSDg

http://i.imgur.com/jTWAhjd.png

Please keep in mind this is again just me smashing my face on the keyboard hoping for things to work.

For now it mostly works, I have some more stuff to do on the backend but basically it's a new endpoint: eve/CharacterAffiliation.xml.aspx?ids=

You will notice thats ids, as in plural... as in batch that **** up.

No promises on anything aside from I will continue to work on it and hopefully get it code reviewed and fixed enough for release at some point in the future.


AWESOMEEEEEEEE.....

and now add an indicator on what I've found in the "CharacterID.xml.aspx" API... like don't do characterID when the ID belongs to a corporation...

just my 12.5 cents
CCP FoxFour
C C P
C C P Alliance
#102 - 2014-01-21 14:49:49 UTC
Chaotic Mind wrote:
CCP FoxFour wrote:
I hate you all very much... as in very very much... as in KAJ GHSBF&P*(UI#WOSDHBUI&*(SEFVDD&*UIFGHIKSDg

http://i.imgur.com/jTWAhjd.png

Please keep in mind this is again just me smashing my face on the keyboard hoping for things to work.

For now it mostly works, I have some more stuff to do on the backend but basically it's a new endpoint: eve/CharacterAffiliation.xml.aspx?ids=

You will notice thats ids, as in plural... as in batch that **** up.

No promises on anything aside from I will continue to work on it and hopefully get it code reviewed and fixed enough for release at some point in the future.


AWESOMEEEEEEEE.....

and now add an indicator on what I've found in the "CharacterID.xml.aspx" API... like don't do characterID when the ID belongs to a corporation...

just my 12.5 cents


That really bothers me as well... however changing it would break a lot of other apps. On top of that it is called character. I was thinking a new endpoint might be in order though that does the same thing, but with a proper return. Dunno. Need to poke some people when I am back in the office.

@CCP_FoxFour // Technical Designer // Team Tech Co

Third-party developer? Check out the official developers site for dev blogs, resources, and more.

Querns
Science and Trade Institute
Caldari State
#103 - 2014-01-21 15:14:58 UTC
Very nice -- thanks for your work. That should simplify quite a few things of mine, as well.

This post was crafted by the wormhole expert of the Goonswarm Economic Warfare Cabal, the foremost authority on Eve: Online economics and gameplay.

CCP FoxFour
C C P
C C P Alliance
#104 - 2014-01-22 13:44:26 UTC
As for the CharacterID endpoint, I don't really want to change it as it would break a lot of existing applications as I said. What I was thinking was a new endpoint along the lines of eve/OwnerID with the following columns: name, id, group

Group is straight from the SDE with 1 being character, 2 being corporation, 19 being faction, and 32 being alliance.

Thats my current thought... I mean I could just add group to the existing one but then you have characterName and characterID attributes for alliances and such.. and thats just messy... but it's already messy... so **** I don't know. Will think some more.

@CCP_FoxFour // Technical Designer // Team Tech Co

Third-party developer? Check out the official developers site for dev blogs, resources, and more.

Querns
Science and Trade Institute
Caldari State
#105 - 2014-01-22 15:04:08 UTC
Unless you run into an issue where caching the results for what's essentially the same information twice taking up too many resources (note: I have no idea how the eve api's caching layer works! :V ) I don't see a real downside to having a new endpoint for it. Heck, didn't that happen for the killmails api endpoint at some point? I can't remember. As long as you don't remove the old endpoint, it probably won't change anything for previously existing code, and anyone who wants to use the new information was gonna have to touch their application anyways.

This post was crafted by the wormhole expert of the Goonswarm Economic Warfare Cabal, the foremost authority on Eve: Online economics and gameplay.

CCP FoxFour
C C P
C C P Alliance
#106 - 2014-01-23 08:47:24 UTC
So, came in early this morning to get some time in on this little project. I just really want to stress that neither this or the other API change I have made are approved for release right now. Once I have them in a better state I will do that. For now, enjoy:

http://i.imgur.com/z2OVtaF.png

@CCP_FoxFour // Technical Designer // Team Tech Co

Third-party developer? Check out the official developers site for dev blogs, resources, and more.

CCP FoxFour
C C P
C C P Alliance
#107 - 2014-01-23 14:13:13 UTC
Since someone else asked and others might wonder, here is the URL I used to produce the result seen in the above screenshot: eve/OwnerID.xml.aspx?names=wtfbbq,Regner,Federal%20Marines,Gallente%20Federation,something%20invalid

@CCP_FoxFour // Technical Designer // Team Tech Co

Third-party developer? Check out the official developers site for dev blogs, resources, and more.

CCP FoxFour
C C P
C C P Alliance
#108 - 2014-01-24 14:51:52 UTC
I hate myself so much... I went and added the DUST wallet to the corp wallet balance/transaction/journal history API endpoint. Now I gotta go test and I hate testing... damnit... /rage

@CCP_FoxFour // Technical Designer // Team Tech Co

Third-party developer? Check out the official developers site for dev blogs, resources, and more.

Querns
Science and Trade Institute
Caldari State
#109 - 2014-01-24 17:46:34 UTC
CCP FoxFour wrote:
I hate myself so much... I went and added the DUST wallet to the corp wallet balance/transaction/journal history API endpoint. Now I gotta go test and I hate testing... damnit... /rage

don't hate testing

testing is your friend

I am writing a massive test suite for an api endpoint at work right now

kill me

This post was crafted by the wormhole expert of the Goonswarm Economic Warfare Cabal, the foremost authority on Eve: Online economics and gameplay.

Y U Dont
Viziam
Amarr Empire
#110 - 2014-01-25 22:45:49 UTC
CCP FoxFour wrote:
I hate myself so much...


While you are in guilt phase, could you add container names to existing assets API endpoints? Please do it!
Drailen
Doomheim
#111 - 2014-01-25 23:06:28 UTC
Y U Dont wrote:
While you are in guilt phase, could you add container names to existing assets API endpoints? Please do it!


This would be duplicating what the Locations API already does.
CCP FoxFour
C C P
C C P Alliance
#112 - 2014-01-27 17:25:27 UTC  |  Edited by: CCP FoxFour
Just to get an actual list of changes out since I have been rambling a fair bit (and a dev blog will come later):
* New OwnerID endpoint for looking up names
* DUST wallet division added
* Character affiliations endpoint added that contains slim character info about what corp/alli/faction they are in.
* Added factionName to the corp/CorporationSheet when factionID > 0
* Added contestedAmount to map/FacWarSystem that contains a float from 0 to 1 of the contested percentage

I am also now looking at adding factionID to other missing locations.

Just to be clear, this is all stuff I have done locally but am still waiting on proper code reviews and such before it gets deployed. There will be a dev blog before it's all done so just keep watching for that.

@CCP_FoxFour // Technical Designer // Team Tech Co

Third-party developer? Check out the official developers site for dev blogs, resources, and more.

Crasniya
Aliastra
Gallente Federation
#113 - 2014-01-27 20:17:23 UTC
CCP FoxFour wrote:

* DUST wallet division added


Is this live now? Or Soon(TM)?

Soraya Xel - Council of Planetary Management 1 - soraya@biomassed.net

CCP FoxFour
C C P
C C P Alliance
#114 - 2014-01-27 20:20:18 UTC
Crasniya wrote:
CCP FoxFour wrote:

* DUST wallet division added


Is this live now? Or Soon(TM)?


Soon(TM)

@CCP_FoxFour // Technical Designer // Team Tech Co

Third-party developer? Check out the official developers site for dev blogs, resources, and more.

Crasniya
Aliastra
Gallente Federation
#115 - 2014-01-27 20:29:41 UTC
CCP FoxFour wrote:
I hate myself so much... I went and added the DUST wallet to the corp wallet balance/transaction/journal history API endpoint. Now I gotta go test and I hate testing... damnit... /rage


FoxFour for president. I know a lot of people who will be extremely happy about this.

Soraya Xel - Council of Planetary Management 1 - soraya@biomassed.net

dent308
The Scope
Gallente Federation
#116 - 2014-01-28 15:48:47 UTC
CCP FoxFour wrote:
Just to get an actual list of changes out since I have been rambling a fair bit (and a dev blog will come later):
* New OwnerID endpoint for looking up names
* DUST wallet division added
* Character affiliations endpoint added that contains slim character info about what corp/alli/faction they are in.
* Added factionName to the corp/CorporationSheet when factionID > 0
* Added contestedAmount to map/FacWarSystem that contains a float from 0 to 1 of the contested percentage

I am also now looking at adding factionID to other missing locations.

Just to be clear, this is all stuff I have done locally but am still waiting on proper code reviews and such before it gets deployed. There will be a dev blog before it's all done so just keep watching for that.


Parsers and unit tests primed and waiting.
Skutter Midumulf
Proioxis Assault Force
Rogue Caldari Union
#117 - 2014-01-31 15:58:43 UTC
Katrinya wrote:
Where the bloody heck is CREST?

Why even bother this dead horse of an API?


This.
Tahnil
Gunboat Commando
#118 - 2014-01-31 16:04:35 UTC
Skutter Midumulf wrote:
Katrinya wrote:
Where the bloody heck is CREST?

Why even bother this dead horse of an API?


This.


Not this.
Draleth
Aliastra
Gallente Federation
#119 - 2014-02-04 17:49:49 UTC  |  Edited by: Draleth
The following is the primary inconsistency that is turning my own code into spaghetti:


  • APIKeyInfo returns almost everything I need except Alliance information.
  • CharacterInfo is better, but is missing character gender, ancestry, and publicly visible titles and roles, and is inconsistent (alliance vs. allianceName from the other calls) thus requiring edge case code. Titles and roles are public "get info" items in-game…
  • CharacterSheet is even better, but some of our members may be reluctant to give away private information to our Auth service and is missing security status.


This makes progressive enhancement of the imported data quite difficult to implement, as with a zero-mask key (which is perfectly acceptable to verify control over a character) there is no alliance information. For auth services the titles and roles are important, and requiring a more expansive key from people with titles is not unreasonable, but the API as it stands now is killing me.

Adding the missing alliance information to the APIKeyInfo call, as I notice some screenshots in prior replies indicating, is a great step. Adding the missing public "get info" information would be even better.

I'm glad to see attention being paid to the API. Ensuring compatibility while making additions should be fairly reasonable.

My blue-sky hope is for the serialization format to be selectable. JSON FTW.

Edited to add: camel case keys (tags/attributes) are used everywhere… except CharacterInfo, which returns 'bloodline' instead of 'bloodLine'. The more I use the API the more I want to flip my desk.
Steve Ronuken
Fuzzwork Enterprises
Vote Steve Ronuken for CSM
#120 - 2014-02-04 18:29:07 UTC
Tahnil wrote:
Skutter Midumulf wrote:
Katrinya wrote:
Where the bloody heck is CREST?

Why even bother this dead horse of an API?


This.


Not this.



With a delayed answer:

There's no reason to get rid of this API, when CREST eventually becomes available. If /all/ you are doing is providing a regular content download, then it's appropriate and probably lighter weight.

CREST will be somewhat harder to implement, than the 'grab a page. stick it in an xml parser and play with the results'.

They both have their place.

Of course, I still want CREST to come out. But it's a very different beast of a thing.

Woo! CSM XI!

Fuzzwork Enterprises

Twitter: @fuzzysteve on Twitter