Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Messed columns in api.eve.CharacterAffiliation() #20

Closed
brammator opened this issue Jul 25, 2014 · 3 comments
Closed

Messed columns in api.eve.CharacterAffiliation() #20

brammator opened this issue Jul 25, 2014 · 3 comments

Comments

@brammator
Copy link

Looks like eveapi lib take columns by position, not by xml attribute name:

>>> for i in api.eve.CharacterAffiliation(ids='735566805,91989271,233838523,1427330454').characters:
...   print i
...
Row(characterName:735566805,characterID:brammator,corporationName:98104433,corporationID:SUB ZERO.,allianceName:1411711376,allianceID:Legion of xXDEATHXx,factionName:0,factionID:)
Row(characterName:91989271,characterID:Y U NO,corporationName:98104433,corporationID:SUB ZERO.,allianceName:1411711376,allianceID:Legion of xXDEATHXx,factionName:0,factionID:)
Row(characterName:233838523,characterID:Karer II,corporationName:1139652088,corporationID:Legion of xXDEATHXx Support,allianceName:1411711376,allianceID:Legion of xXDEATHXx,factionName:0,factionID:)
Row(characterName:1427330454,characterID:Taemine,corporationName:98104433,corporationID:SUB ZERO.,allianceName:1411711376,allianceID:Legion of xXDEATHXx,factionName:0,factionID:)

Actual XML:

<?xml version='1.0' encoding='UTF-8'?>
<eveapi version="2">
  <currentTime>2014-07-25 20:11:49</currentTime>
  <result>
    <rowset name="characters" key="characterID" columns="characterName,characterID,corporationName,corporationID,allianceName,allianceID,factionName,factionID">
      <row characterID="735566805" characterName="brammator" corporationID="98104433" corporationName="SUB ZERO." allianceID="1411711376" allianceName="Legion of xXDEATHXx" factionID="0" factionName="" />
      <row characterID="91989271" characterName="Y U NO" corporationID="98104433" corporationName="SUB ZERO." allianceID="1411711376" allianceName="Legion of xXDEATHXx" factionID="0" factionName="" />
      <row characterID="233838523" characterName="Karer II" corporationID="1139652088" corporationName="Legion of xXDEATHXx Support" allianceID="1411711376" allianceName="Legion of xXDEATHXx" factionID="0" factionName="" />
      <row characterID="1427330454" characterName="Taemine" corporationID="98104433" corporationName="SUB ZERO." allianceID="1411711376" allianceName="Legion of xXDEATHXx" factionID="0" factionName="" />
    </rowset>
  </result>
  <cachedUntil>2014-07-25 21:11:49</cachedUntil>
</eveapi>```
@ntt
Copy link
Owner

ntt commented Jul 25, 2014

Known issue and is a CCP bug. The columns attribute and the actual row data
must be in proper order as per the design for the API.
Processing columns out of order would incur a performance hit which why it
was done positionally.

On Fri, Jul 25, 2014 at 10:21 PM, brammator notifications@github.com
wrote:

Looks like eveapi lib take columns by position, not by xml attribute name:

for i in api.eve.CharacterAffiliation(ids='735566805,91989271,233838523,1427330454').characters:... print i...Row(characterName:735566805,characterID:brammator,corporationName:98104433,corporationID:SUB ZERO.,allianceName:1411711376,allianceID:Legion of xXDEATHXx,factionName:0,factionID:)Row(characterName:91989271,characterID:Y U NO,corporationName:98104433,corporationID:SUB ZERO.,allianceName:1411711376,allianceID:Legion of xXDEATHXx,factionName:0,<
/span>factionID:)Row(characterName:233838523,characterID:Karer II,corporationName:1139652088,corporationID:Legion of xXDEATHXx Support,allianceName:1411711376,allianceID:Legion of xXDEATHXx,factionName:0,factionID:)Row(characterName:1427330454,characterID:Taemine,corporationName:98104433,corporationID:SUB ZERO.,allianceName:1411711376,allianceID:Legion of xXDEATHXx,factionName:0,factionID:)

Actual XML:

2014-07-25 20:11:49








2014-07-25 21:11:49```


Reply to this email directly or view it on GitHub
#20.

@ntt
Copy link
Owner

ntt commented Sep 14, 2014

Fixed by CCP

@ntt ntt closed this as completed Sep 14, 2014
@regner
Copy link

regner commented Sep 16, 2014

YAY! Maybe one day I will be less terrible and just not screw this up to start with. :/

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants