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’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Bug Report] SQL Error upon deletion of all items in active tab, with sorting order which is non compliant to the order that the first populated content tab, after deletion can accept #4209

Closed
elkorol opened this issue Oct 16, 2023 · 0 comments · Fixed by #4538
Labels
bug report Bug reports that are not yet verified

Comments

@elkorol
Copy link
Contributor

elkorol commented Oct 16, 2023

Describe the bug
PR: 4032 in the newest version of Stash has introduced a bug when on the Details pages, while on the active tab, upon deleting all items, if the current sort order is set to a setting that does not conform to what the sort settings of the first populated content tab that will be active upon deletion of all items, then Stash throws an sql error. Somehow the current sort order of the active pane isn't being reset to what the first populated content tab can handle, or the sort order is being passed to the new content tab after deletion.

To Reproduce
Steps to reproduce the behavior:

  1. Create a test performer.
  2. Add some scenes, and a gallery with some images
  3. Activate the scenes tab, change the sorting order to anything that Gallery tab sorting cannot accept. I.e. Duration Ascending.
  4. Select all scenes & delete.
  5. See SQL error on screen.

Stash Version: (from Settings -> About):
v0.23.1
Build hash:
479e716

Desktop (please complete the following information):

  • OS: Windows 11
  • Browser Chrome
  • Version 118.0.5993.70
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug report Bug reports that are not yet verified
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants