datasucker / netrunner-datasucker

MIT License
20 stars 7 forks source link

Ordering of /sets #5

Closed DoubleAitch closed 9 years ago

DoubleAitch commented 9 years ago

The set.number objects do not 100% match the chronological ordering of the expansions due to being computed from the ordering of the setcode values in the source data (which are not in chronological order).

Specifically Upstalk (number:15, setid:333) should be after Honor & Profit (number:16, setid:342).

Adding a secondary sets.sort function based on the cyclenumber before assigning the set.number values should resolve the issue (~line 154 of main.js).

datasucker commented 9 years ago

Resolved. Good catch!