-
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
Commits on Nov 12, 2023
-
Configuration menu - View commit details
-
Copy full SHA for 1a22e37 - Browse repository at this point
Copy the full SHA 1a22e37View commit details -
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
Configuration menu - View commit details
-
Copy full SHA for 73c6c42 - Browse repository at this point
Copy the full SHA 73c6c42View commit details -
Set reasonable max for LB free index tracker
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.
Configuration menu - View commit details
-
Copy full SHA for 91b6068 - Browse repository at this point
Copy the full SHA 91b6068View commit details -
Fix LB DB index reservation for existing playlist games
Accidentally was calling release() instead of reserve().
Configuration menu - View commit details
-
Copy full SHA for e6ad5f6 - Browse repository at this point
Copy the full SHA e6ad5f6View commit details
Commits on Nov 13, 2023
-
LB: Use modified playlist ID cache directly
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.
Configuration menu - View commit details
-
Copy full SHA for c81ba21 - Browse repository at this point
Copy the full SHA c81ba21View commit details -
Merge pull request #40 from oblivioncth/bugfix/lb_playlist_category
Fix some playlists not being included under the Flashpoint Platform Category in LaunchBox
Configuration menu - View commit details
-
Copy full SHA for 340d962 - Browse repository at this point
Copy the full SHA 340d962View commit details -
Distinguish between modified docs and deleted docs
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.
Configuration menu - View commit details
-
Copy full SHA for c0e4d25 - Browse repository at this point
Copy the full SHA c0e4d25View commit details -
Configuration menu - View commit details
-
Copy full SHA for 518315a - Browse repository at this point
Copy the full SHA 518315aView commit details -
Make translateDocName() public
Needed for some ImportWorker tasks
Configuration menu - View commit details
-
Copy full SHA for e2456d4 - Browse repository at this point
Copy the full SHA e2456d4View commit details -
Don't use FP's favorities icon
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.
Configuration menu - View commit details
-
Copy full SHA for 56518be - Browse repository at this point
Copy the full SHA 56518beView commit details -
Configuration menu - View commit details
-
Copy full SHA for e26f858 - Browse repository at this point
Copy the full SHA e26f858View commit details -
Configuration menu - View commit details
-
Copy full SHA for 7a2122e - Browse repository at this point
Copy the full SHA 7a2122eView commit details -
Configuration menu - View commit details
-
Copy full SHA for 88e74a3 - Browse repository at this point
Copy the full SHA 88e74a3View commit details -
Merge pull request #41 from oblivioncth/feature/platform_playlist_cat…
…egories LB: Nest platforms and playlists under their own platform categories
Configuration menu - View commit details
-
Copy full SHA for c6a76a0 - Browse repository at this point
Copy the full SHA c6a76a0View commit details
Commits on Nov 27, 2023
-
Configuration menu - View commit details
-
Copy full SHA for ac077d7 - Browse repository at this point
Copy the full SHA ac077d7View commit details