ntt / eveapi

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

Certain keys should probably never be converted to int/float/date #5

Closed nuxi closed 12 years ago

nuxi commented 12 years ago

PL's -10.0 ticker is interpreted as a float. I'm not aware of any others that actually cause a problem, but especially with floats there is always the chance it will be represented as something like -10.0000000000000001 and then not cast back to a string properly. If one is accidently interpreted as a date (which has special handling code) that might be a problem too.

ntt commented 12 years ago

Yep, I'm perfectly aware of that. it's not the only case; characters or transaction reasons only containing numbers are another problem, etc. Feel free to substitute the default _autocast function with something ever so slightly smarter. It's the reason I included the attribute name in the parameters of it.