-
-
Notifications
You must be signed in to change notification settings - Fork 6.3k
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
[xbmc] Addon settings fixes #12536
[xbmc] Addon settings fixes #12536
Conversation
…he element name instead of the id Fix a conversion issue where settings without category was skipped over.
settingId += setting->ValueStr(); | ||
auto id = setting->ToElement()->Attribute("id"); | ||
if (id) | ||
settingId += id; |
This comment was marked as spam.
This comment was marked as spam.
Sorry, something went wrong.
This comment was marked as spam.
This comment was marked as spam.
Sorry, something went wrong.
This comment was marked as spam.
This comment was marked as spam.
Sorry, something went wrong.
// build a vector of settings from the same category | ||
std::vector<std::shared_ptr<const CSetting>> categorySettings; | ||
std::set<std::string> settingIds; |
This comment was marked as spam.
This comment was marked as spam.
Sorry, something went wrong.
This comment was marked as spam.
This comment was marked as spam.
Sorry, something went wrong.
This comment was marked as spam.
This comment was marked as spam.
Sorry, something went wrong.
This comment was marked as spam.
This comment was marked as spam.
Sorry, something went wrong.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good exept the one minor thing
@Paxxi anything left or need updating? |
Good to go @martijn. Was waiting on me looking into peaks comment but no changes are warranted there |
Fix a parsing issue for new setting format where settingId would be the element name instead of the id
Fix a conversion issue where settings without category was skipped over.
This should solve log spam of creating settings on the fly for valid addons and properly save/load addons after converted to the new setting format.
@peak3d for testing.
Description
Motivation and Context
How Has This Been Tested?
Screenshots (if appropriate):
Types of change
Checklist: