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

/v2/characters list of bags no null entries after last bag #46

Closed
zeeZ opened this issue Jun 26, 2015 · 2 comments
Closed

/v2/characters list of bags no null entries after last bag #46

zeeZ opened this issue Jun 26, 2015 · 2 comments

Comments

@zeeZ
Copy link

zeeZ commented Jun 26, 2015

My interpretation of characters.js is that the ''bags'' list will always have the size of that character's count of unlocked slots. The example shows a null entry after the last bag, however the API currently does not contain null objects after the last bag object in the list.

Example for a character with 5 bag slots unlocked:

Bags in slot 0 and 1:

{
  ...
  "bags": [
    {
      "id": 8941,
      "size": 4
      "inventory": [ null, null, null, null ]
    },
    {
      "id": 8941,
      "size": 4
      "inventory": [ null, null, null, null ]
    }
  ]
}

Bags in slot 0 and 2:

{
  ...
  "bags": [
    {
      "id": 8941,
      "size": 4
      "inventory": [ null, null, null, null ]
    },
    null,
    {
      "id": 8941,
      "size": 4
      "inventory": [ null, null, null, null ]
    }
  ]
}

Bags in slot 0 and 4:

{
  ...
  "bags": [
    {
      "id": 8941,
      "size": 4
      "inventory": [ null, null, null, null ]
    },
    null,
    null,
    null,
    {
      "id": 8941,
      "size": 4
      "inventory": [ null, null, null, null ]
    }
  ]
}

For fun, bags in slot 1 and 3:

{
  ...
  "bags": [
    null,
    {
      "id": 8941,
      "size": 4
      "inventory": [ null, null, null, null ]
    },
    null,
    {
      "id": 8941,
      "size": 4
      "inventory": [ null, null, null, null ]
    }
  ]
}
@tivac tivac added the Bug label Jun 26, 2015
@lye
Copy link
Contributor

lye commented Jun 26, 2015

Oof, will have to look into this one next week.

@tivac tivac added the Community label Jul 6, 2015
@lye lye closed this as completed Jul 28, 2015
@lye lye removed the needs deploy label Jul 28, 2015
@lye
Copy link
Contributor

lye commented Jul 28, 2015

This should be fixed on live.

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

No branches or pull requests

3 participants