ntt / eveapi

Python library for accessing the EVE Online API
Other
148 stars 57 forks source link

a bug from Serenity server API #21

Closed copyliu closed 9 years ago

copyliu commented 9 years ago

from Serenity server , Corporation ticker field not parse as a string.

https://api.eve-online.com.cn/corp/CorporationSheet.xml.aspx?corporationID=98000263

>>> api = eveapi.EVEAPIConnection("api.eve-online.com.cn")
>>> q=api.corp.CorporationSheet(corporationID=98000263)
>>> q.ticker
<eveapi.Element object at 0x039C8A90>`
>>> print q.ticker
<Element 'ticker'>

xml is

<?xml version='1.0' encoding='UTF-8'?>
<eveapi version="2">
  <currentTime>2014-11-02 19:41:23</currentTime>
  <result>
    <corporationID>98000263</corporationID>
    <corporationName>军情四处</corporationName>
    <ticker>Live </ticker>
    <ceoID>90000068</ceoID>
    <ceoName>rova3</ceoName>
    <stationID>60014659</stationID>
    <stationName>Kisogo VII - State War Academy</stationName>
    <description>在这里输入你军团的描述。</description>
    <url>http://eve.tiancity.com</url>
    <allianceID>0</allianceID>
    <factionID>0</factionID>
    <taxRate>1</taxRate>
    <memberCount>18</memberCount>
    <shares>1000</shares>
    <logo>
      <graphicID>0</graphicID>
      <shape1>415</shape1>
      <shape2>574</shape2>
      <shape3>415</shape3>
      <color1>0</color1>
      <color2>676</color2>
      <color3>0</color3>
    </logo>
  </result>
  <cachedUntil>2014-11-02 19:41:23</cachedUntil>
</eveapi>```
ntt commented 9 years ago

I've pushed a workaround but it might break other things.

Note that this ticker is actually an illegal name (space at end), so if this fix breaks anything else I will revert it.