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.
 

Crucible and API changes

First post
Author
Previous Topic Next Topic
Lutz Major
Austriae Est Imperare Orbi Universo
#1 - 2011-11-30 08:37:37 UTC
Hello devs / community!

Just checked my logs and saw, that there are some changes beside the patch notes:
(I use the ( as tag opener! Stupid forum ... can we have a [code] block again?!? .. or at least a forum that displays > < ?)

The CharacterSheet does not print (allianceID>0(/allianceID> anymore, instead (allianceID /> is used.

Intended?
Desmont McCallock
#2 - 2011-11-30 16:44:46 UTC  |  Edited by: Desmont McCallock
CCPStillman, can you please fix this cause the change is breaking our apps?

After all you should return an 'integral' value and not a 'string' value for an element marked as 'allianceID', meaning that while the character corp is not in an alliance the value should be 0.
darius mclever
#3 - 2011-11-30 17:03:58 UTC
Desmont McCallock wrote:
CCPStillman, can you please fix this cause the change is breaking our apps?

After all you should return an 'integral' value and not a 'string' value for an element marked as 'allianceID', meaning that while the character corp is not in an alliance the value should be 0.


actually it isnt a string value but an empty element in the short notation. though the question might be instead of passing an empty element, if it wouldnt be better to leave out the element completely.
Desmont McCallock
#4 - 2011-11-30 17:08:02 UTC  |  Edited by: Desmont McCallock
darius mclever wrote:
Desmont McCallock wrote:
CCPStillman, can you please fix this cause the change is breaking our apps?

After all you should return an 'integral' value and not a 'string' value for an element marked as 'allianceID', meaning that while the character corp is not in an alliance the value should be 0.


actually it isnt a string value but an empty element in the short notation. though the question might be instead of passing an empty element, if it wouldnt be better to leave out the element completely.


Whatever, but .NET XML deserializer is handling it as possible string value.

True, not having the element at all makes the handling easier (.NET XML deserializer assigns a missing element value to the default of the assigned type.)
Etil DeLaFuente
Aliastra
Gallente Federation
#5 - 2011-11-30 19:22:08 UTC
And we're back to the point where XSD should be provided with those xml files. Would make life easier for everyone.
Desmont McCallock
#6 - 2011-11-30 19:42:10 UTC
I wish they had given some "love" to the API as they did with the so many staff for the Crucible expansion, but...
Dragonaire
Here there be Dragons
#7 - 2011-12-01 07:19:10 UTC
I try to keep up with making my own XSDs for most of the more popular APIs that I use with Yapeal and make available on sourceforge.

http://sourceforge.net/projects/yapeal/files/schemas/xsd-latest.tar.bz2/download

I haven't had a chance to check everything out yet to see what might have broke but I'll try to get to it this week or might be weekend before I can.

Finds camping stations from the inside much easier. Designer of Yapeal for the Eve API. Check out the Yapeal PHP API Library thread.
Lutz Major
Austriae Est Imperare Orbi Universo
#8 - 2011-12-01 07:33:08 UTC
Desmont McCallock wrote:

Whatever, but .NET XML deserializer is handling it as possible string value.

True, not having the element at all makes the handling easier (.NET XML deserializer assigns a missing element value to the default of the assigned type.)


Same in Java. I'm using JAXB to de-marshal the XML into an Object. I had the allianceID set as native 'long'. Now I have to use the object Long.

Checking against zero I have to refactor everything checking against null now. Not a big deal, but annoying.
CCP Prism X
C C P
C C P Alliance
#9 - 2011-12-01 08:47:59 UTC
No backwards incompatible changes should happen in the API (Except in all the cases where I decided to make them. I regret every such instance. This is not one of them).

This will get fixed back to the way it was and pushed out with other API fixes as soon as possible.

CCP Prism X

Programmer on Team Gridlock & Expert Ranter