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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

remove deprecated extra artwork settings from AS.xml #22748

Merged

Conversation

rmrector
Copy link
Contributor

@rmrector rmrector commented Feb 11, 2023

Description

Remove the advancedsettings.xml configuration for extra artwork. This was added in v18, then we moved it to GUI settings in v19, with one-time migration functionality. v21 seems a good time to remove the deprecated settings and migration code.

Motivation and context

Simplify. Remove deprecated stuff. Prove that advancedsettings.xml can be moved / removed later. Just gotta remember to do it. 馃槃

How has this been tested?

Manual testing. It doesn't do anything.

What is the effect on users?

Simplifies configuration.

馃摪 Extraart configurations in advancedsettings.xml for Kodi 18 will no longer be migrated to the new GUI settings when upgrading. If you had meaningful configuration there, check that the artwork settings in the GUI match your preferences.

Screenshots (if appropriate):

Types of change

  • Bug fix (non-breaking change which fixes an issue)
  • Clean up (non-breaking change which removes non-working, unmaintained functionality)
  • Improvement (non-breaking change which improves existing functionality)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that will cause existing functionality to change)
  • Cosmetic change (non-breaking change that doesn't touch code)
  • None of the above (please explain below)

Checklist:

  • My code follows the Code Guidelines of this project
  • My change requires a change to the documentation, either Doxygen or wiki
  • I have updated the documentation accordingly
  • I have read the Contributing document
  • I have added tests to cover my change
  • All new and existing tests passed

@rmrector rmrector added Type: Cleanup non-breaking change which removes non-working or unmaintained functionality Component: Music Component: Video Component: Settings v21 Omega labels Feb 11, 2023
@rmrector rmrector added this to the Omega 21.0 Alpha 1 milestone Feb 11, 2023
@DaveTBlake
Copy link
Member

Doesn't this mean we are assuming that users will always be incremetally updating Kodi, that none of them will be on v18 and leap to v21 without installing the versions inbetween. Historically we have users that change version infrequently.

Maybe we don't care about that, but if that is the case then can we document (both for users and devs) that such step changes in version can have consequnces for as.xml and art.

@rmrector
Copy link
Contributor Author

"We don't care about that" is a strong way to put it, but I guess it's closer. It's just old obscure artwork config from a single version of Kodi that has been out of date longer than it was in use.

@jenkins4kodi jenkins4kodi added the Rebase needed PR that does not apply/merge cleanly to current base branch label Apr 18, 2023
@rmrector rmrector force-pushed the remove-deprecated-artwork-advancedsettingsxml branch from 8b1196d to b58ab95 Compare April 20, 2023 03:04
@jenkins4kodi jenkins4kodi removed the Rebase needed PR that does not apply/merge cleanly to current base branch label Apr 20, 2023
@rmrector
Copy link
Contributor Author

Added a blurb for Alpha 2, if I remember when it comes.

Extraart configurations in advancedsettings.xml for Kodi 18 will no longer be migrated to the new GUI settings when upgrading. If you had meaningful configuration there, check that the artwork settings in the GUI match your preferences.

@rmrector rmrector merged commit 4078054 into xbmc:master Apr 20, 2023
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Component: Music Component: Settings Component: Video Type: Cleanup non-breaking change which removes non-working or unmaintained functionality v21 Omega Wiki: Done
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

7 participants