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

Crash due to PUT content response update #205

Closed
axel3rd opened this issue Oct 5, 2021 · 2 comments
Closed

Crash due to PUT content response update #205

axel3rd opened this issue Oct 5, 2021 · 2 comments
Assignees
Labels
bug Something isn't working
Milestone

Comments

@axel3rd
Copy link
Owner

axel3rd commented Oct 5, 2021

Stack:

Exception in thread "main" java.lang.UnsupportedOperationException: The team has not been updated, result message: {"id":"mpg_match_team_formation_NL931FU9_1_1_6_3_4","type":"mpg_match_team_formation","composition":"343","playersOnPitch":{"1":"mpg_championship_player_138039","2":"mpg_championship_player_219717","3":"mpg_championship_player_170116","4":"mpg_championship_player_472148","5":"mpg_championship_player_61256","6":"mpg_championship_player_111232","7":"mpg_championship_player_51952","8":null,"9":"mpg_championship_player_40270","10":"mpg_championship_player_61597","11":"mpg_championship_player_61316","12":"mpg_championship_player_220325","13":null,"14":null,"15":null,"16":null,"17":null,"18":"mpg_championship_player_175204"},"tacticalSubs":[{"rating":5,"subId":"mpg_championship_player_220325","starterId":"mpg_championship_player_472148"}],"captain":"mpg_championship_player_61256"}
        at org.blondin.mpg.root.MpgClient.updateCoach(MpgClient.java:126)
        at org.blondin.mpg.Main.updateTeamWithRetry(Main.java:282)
        at org.blondin.mpg.Main.processGames(Main.java:194)
        at org.blondin.mpg.Main.processLeague(Main.java:114)
        at org.blondin.mpg.Main.process(Main.java:82)
        at org.blondin.mpg.Main.main(Main.java:71)

"null" is perhaps not the good value when no player at some position.

@axel3rd axel3rd added the bug Something isn't working label Oct 5, 2021
@axel3rd axel3rd added this to the 1.8.1 milestone Oct 5, 2021
@axel3rd axel3rd self-assigned this Oct 5, 2021
@axel3rd axel3rd changed the title Crash when not enough players to fill pitch Crash due to PUT content update Oct 5, 2021
@axel3rd axel3rd changed the title Crash due to PUT content update Crash due to PUT content response update Oct 5, 2021
@axel3rd
Copy link
Owner Author

axel3rd commented Oct 5, 2021

The response seems have changed (before {"success":true})

@axel3rd
Copy link
Owner Author

axel3rd commented Oct 5, 2021

Fixed in mpg-coach-bot-1.8.1-20211005.174015-1.jar. Waiting a little before release, to verify if there are no other major changes on API during the week.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

1 participant