mirror of
https://github.com/BookStackApp/BookStack.git
synced 2026-02-08 03:09:39 +03:00
Bug - Sorting functionality #5227
Reference in New Issue
Block a user
Delete Branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Originally created by @Phi2ca on GitHub (Mar 18, 2025).
Describe the Bug
Hi
I created a sorting rule by taking “Chapters First” from “Available Operations” and putting it in “Configured Operations”.
I've put “Chapters First” in the first position in “Configured Operations”.
Bug
When I create a new book with chapters and pages everything goes fine.
However, if I apply this rule to an existing book (since I have the option in the sorting section), the chapters are always and automatically placed at the bottom of the sorting order and not in first position.
Best regards
Steps to Reproduce
Creating a sorting rule by taking “Chapters First” from “Available Operations” and putting it in “Configured Operations”.
Put “Chapters First” in the first position in “Configured Operations”.
Apply this rule to existing books
Expected Behaviour
Place chapters in first position as specified by the feature
Screenshots or Additional Context
No response
Browser Details
Firefox 136.0.1 (64 bits)
Exact BookStack Version
v25.02.1
@manikandan-1704 commented on GitHub (Mar 18, 2025):
I would like to fix this issue
@ssddanbrown commented on GitHub (Mar 18, 2025):
Hi @Phi2ca,
The "Chapters First" sorting seems to be working from my testing.
Do you have any other operations configured as part of your sort rule? Or does it only contain the "Chapters first" rule?
@Phi2ca commented on GitHub (Mar 18, 2025):
Yes, I have other settings behind “Chapters First”.
So in that order:
Chapters first
Name Numerics (Asc)
Name alphabetical (Asc)
That's all
@ssddanbrown commented on GitHub (Mar 18, 2025):
@Phi2ca Reverse the order of those operations, so
Chapters firstis last in the order.Sort operations run from top-to-bottom, so right now the chapters will be sorted by name after being moved first. Moving the operation to the bottom, means that chatpers will be moved first after being sorted by name.
@Phi2ca commented on GitHub (Mar 18, 2025):
Ok, but now we're not really in a human logic anymore:
I'll put the chapters first, then the pages in alphabetical order, thus following the very logic of what they want to achieve.
I don't know many people who'd say to themselves: I'll sort the pages alphabetically and put the chapters first...but that's just my point of view.
Or you need a clearer tutorial in the section itself, otherwise people will think that it doesn't work and shouldn't be used.
@ssddanbrown commented on GitHub (Mar 18, 2025):
Okay. The order (especially with the supporting text) makes sense to me, but this is probably a matter of opinion/expectations. I'll keep a watch to see if this is a common misunderstand and look to make UX improvements here if so, otherwise I'll go ahead and close this off since there's not a bug to address.