eve-val / evelink

Python bindings for the EVE API.

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

char/industry_jobs seems to be returning status = failed prematurely.

flexd opened this issue · comments

As per http://wiki.eve-id.net/APIv2_Char_IndustryJobs_XML

Invention/completedStatus:
0 = failed
1 = delivered
2 = aborted
3 = GM aborted
4 = inflight unanchored
5 = destroyed
If it's not ready yet, completed = 0 and completedStatus is irrelevant/uninitialized/probably 0
If complete = 1 and status = 0, then it failed

Currently evelink returns failed on a job in-progress when it should be reporting something like 'in-progess' or whatever else what might be suitable.

The problem is here https://github.com/eve-val/evelink/blob/master/evelink/constants.py#L31 , and in the parser.

Do you agree? It should be a fairly easy fix.

Yeah, that looks like a bug.

5ef53e0 adds the missing "aborted" value; the parsing logic still needs fixing.

This is not working correctly. Completed and delivered jobs are now still showing as in-progress here.

I've sort of fixed it locally by making the changes I commented on the commit, but successful is still False for jobs I know are successful (just pulled a bunch of modules out from a job now)

The in-progress bug should be sorted now. I'm not sure what's up with completedSuccessfully.