collection: #20718 breaks filmroll sort by folder

[MStraeten]

Is there an existing issue for this?

• I checked and did not find my issue in the already reported ones

Describe the bug

#20718 breaks old filmroll sort by folder.
instead of the folder now just the shown subfolders are taken into account for sorting.

example: if 2 levels are selected and order by folder
/volumes/aa/2010 and /users/bb/2012 are sorted as
aa/2010_x
bb/2012_y
aa/2013_z

instead of former
bb/2012_y
aa/2010_x
aa/2013_z

which took the whole folder path into account

Steps to reproduce

check with a multilevel folder structure

Expected behavior

old filmroll order should be kept

Logfile | Screenshot | Screencast

No response

Commit

185af71

Where did you obtain darktable from?

self compiled

darktable version

darktable 5.5.0+893~ge75b3d1356-dirty

What OS are you using?

Mac

What is the version of your OS?

macos 26

Describe your system

No response

Are you using OpenCL GPU in darktable?

None

If yes, what is the GPU card and driver?

No response

Please provide additional context if applicable. You can attach files too, but might need to rename to .txt or .zip

No response

[da-phil]

Potential fix for the sorting: #20697 (comment)

[rtmongold]

I just checked with the updates from last night and my results were:

Here is the fullpath names for reference.

When I have time I will look over the comments from your work maybe we are tripping over each other?

[rtmongold]

Just had to time to reread everything #20718 was in response to issue #20703 which requested sorting by the displayed name when using folder names.

[MStraeten]

the mentioned "Issue 2" in rtmongold@014e7b6 isn't an issue but the standard solution used for a long time.
Especially useful if you work a lot with external drives.
That needs to be reverted

[rtmongold]

My intent is not stir the pot or ruffle any feathers here. What if the sort by folder name is reverted back to the original to avoid confusion. Then a new sort option something like "display name" is added to allow end users to chose to sort based on what is being displayed in front of them.

[MStraeten]

sound like a viable solution …