-
Notifications
You must be signed in to change notification settings - Fork 0
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
Merge to master for v0.7.4 #43
Merged
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Playlists were treated as platforms due to typos. Likely broke AM taglists, but had no sideffects for LB platforms as modifiedPlaylists() was being used before playlists were added, and seemingly didn't affect LB playlists either as the null playlist ID would cause the check functions to think the bogus playlists had already been handled due to the platform entries in Parents.xml
Previous max of -1 was a leftover from the previous implementation of Qx::FreeIndexTracker who's index type was templated, so -1 represented a reasonable max (likely 32-bit integer limit); however, now that the tracker uses quint64 for indexing, -1 was interpreted as the maximum value for 64-bit integers and caused allocaiton of its internal bit- array to fail due to absurd size requirements. This was undiscovered due to the previous playlist bugs.
Accidentally was calling release() instead of reserve().
No need to store as a QHash with the playlist name as the key since the name is not needed. This also avoids the awkwardness that is the conversion between the original playlist name and the translated name, specific to LB. Before, playlists with names that differed between their original and translated names were not being added to the Flashpoint platform category as the keys in the hash were the pre-translated names, while the list checked against at the end were the translated names.
Fix some playlists not being included under the Flashpoint Platform Category in LaunchBox
Frontends often use modifiedPlatforms()/modifiedPlaylists() to check which Platforms/Playlists were affected by the important and need to be taken into account by other parts of the frontend; however, this obviously only applies if the doc was stricly modified and not outright removed, which can happen if the commited modified doc is empty. This change ensures that only modified docs that remain will be part of the modified list.
Needed for some ImportWorker tasks
LB, the only platform that supports them currently, already has its own. Will need to revisit this if other platforms that support them are added.
…egories LB: Nest platforms and playlists under their own platform categories
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.