-
-
Notifications
You must be signed in to change notification settings - Fork 247
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
Fix: correctly read state of map info contributors from profile XML #5976
Fix: correctly read state of map info contributors from profile XML #5976
Conversation
Hey there! Thanks for helping Mudlet improve. 🌟 Test versionsYou can directly test the changes here:
No need to install anything - just unzip and run. |
clang-tidy review says "All clean, LGTM! 👍" |
Fix: fix reading map info contributors
4e31dd1
to
824f530
Compare
@vadi2 can you rerun appveyor build please? |
…tributors-bugfixes
…tributors-bugfixes
All good now |
Not sure why clangtidy suddenly failing. |
Checked the build log? Looks like an action issue, should be reported |
I guess it complains about complexity of |
Seems fine to me to merge this time |
Brief overview of PR changes/additions
Just as title says... there were two bugs.
After flattening XML it was not longer easy to say, that user selected none of map info contributors or came from old save file.
While fixing that, I've noticed that I used wrong attribute name for reading map info contributors.
Motivation for adding to Mudlet
Other info (issues closed, discussion etc)