digiKam › digikam-devel

[digikam] [Bug 325925] New: Importer: Sorting order is not remembered

Classic

List

Threaded

4 messages Options

Simon-2

[digikam] [Bug 325925] New: Importer: Sorting order is not remembered

https://bugs.kde.org/show_bug.cgi?id=325925

Bug ID: 325925
Summary: Importer: Sorting order is not remembered
Classification: Unclassified
Product: digikam
Version: 3.4.0
Platform: Debian testing
OS: Linux
Status: UNCONFIRMED
Severity: normal
Priority: NOR
Component: Import
Assignee: [hidden email]
Reporter: [hidden email]

I have set my importer to sort by date, flat list. Whenever I restart digikam,
I have to change some of the sorting options (e.g. switch from ascending to
descending or back) until images are sorted by date again and not displayed
randomly (or perhaps randomly?).

This bug is present since around version 4.

Reproducible: Always

--
You are receiving this mail because:
You are the assignee for the bug.
_______________________________________________
Digikam-devel mailing list
[hidden email]
https://mail.kde.org/mailman/listinfo/digikam-devel

Simon-2

[digikam] [Bug 325925] Importer: Sorting order is not remembered

https://bugs.kde.org/show_bug.cgi?id=325925

--- Comment #1 from Simon <[hidden email]> ---
edit – since version 3. Digikam 2.x did not have this problem.

--
You are receiving this mail because:
You are the assignee for the bug.
_______________________________________________
Digikam-devel mailing list
[hidden email]
https://mail.kde.org/mailman/listinfo/digikam-devel

Teemu Rytilahti

[digikam] [Bug 325925] Importer: Sorting order is not remembered

In reply to this post by Simon-2

https://bugs.kde.org/show_bug.cgi?id=325925

Teemu Rytilahti <[hidden email]> changed:

What |Removed |Added
----------------------------------------------------------------------------
Status|UNCONFIRMED |CONFIRMED
CC| |[hidden email]
Ever confirmed|0 |1

--- Comment #2 from Teemu Rytilahti <[hidden email]> ---
I can confirm this. The settings are saved properly, but on load the change is
not being propagated properly. The relevant code is in
digikam/utilities/importui/main/importui.cpp's setupActions().

The naming of sorting functions/variables is a bit odd here, "Sorty by x" calls
view's slotSortImages(), but the current item is read from
ImportSettings::instance()->getImageSortOrder(). For ascending/descending the
call goes to slotSortImagesOrder() and the value is read from
getImageSorting(). Perhaps those are mixed or should there be separate calls
for setting the values on load? I'd suggest renaming those variables
nevertheless if possible. SortBy and SortOrder would be better names :-)

--
You are receiving this mail because:
You are the assignee for the bug.
_______________________________________________
Digikam-devel mailing list
[hidden email]
https://mail.kde.org/mailman/listinfo/digikam-devel

Teemu Rytilahti

[digikam] [Bug 325925] Importer: Sorting order is not remembered

In reply to this post by Simon-2