Skip to content
This repository has been archived by the owner on Jan 28, 2022. It is now read-only.

POGOProtos.Data.PlayerData.tutorialState invalid #35

Closed
andrewmunro opened this issue Jul 22, 2016 · 8 comments
Closed

POGOProtos.Data.PlayerData.tutorialState invalid #35

andrewmunro opened this issue Jul 22, 2016 · 8 comments

Comments

@andrewmunro
Copy link

Using the latest Protobufs, I get ERROR: Illegal wire type for field Message.Field .POGOProtos.Data.PlayerData.tutorialState: 2 (0 expected) when trying to convert them using protobufjs.

This occurs when doing a RequestType.GET_PLAYER request and I decode the response using the Networking.Responses.GetPlayerResponse protobuf.

I guess the TutorialState enum is incorrect. I can convert the response correctly if I set https://github.com/AeonLucid/POGOProtos/blob/master/src/POGOProtos/Data/PlayerData.proto#L15 to bytes

@AeonLucid
Copy link
Owner

Can you try to set it to
.POGOProtos.Enums.TutorialState tutorial_state = 7;
and see if it works correctly?

@andrewmunro
Copy link
Author

Nope same error :(

@AeonLucid
Copy link
Owner

Woops uhm, how about:
repeated .POGOProtos.Enums.TutorialState tutorial_state = 7 [packed=true];

This should work.

@andrewmunro
Copy link
Author

Works!

  username: 'dave976',
  team: 1,
  tutorialState: [ 0, 1, 3, 4, 7 ],
  ...

Now I'm having issues with https://github.com/AeonLucid/POGOProtos/blob/master/src/POGOProtos/Data/PokemonData.proto#L34 ... it works when I set it to float.

@AeonLucid
Copy link
Owner

Fixed the tutorial state. (966250b)

I think that the multiplier should be a float indeed.

@rastapasta
Copy link
Contributor

protobuf.js doesn't handle syntax3 repeated fields in a correct way yet
protobufjs/protobuf.js#432

@rubenvereecken
Copy link
Contributor

Any objections to closing the issue?

@andrewmunro
Copy link
Author

Thanks :)

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

No branches or pull requests

4 participants