[Bug 276050] New: Setting "string comprison type" is not evaluated in thumbnail-view

[Peter Albrecht-2]

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

           Summary: Setting "string comprison type" is not evaluated in
                    thumbnail-view
           Product: digikam
           Version: 2.0.0
          Platform: Gentoo Packages
        OS/Version: Linux
            Status: UNCONFIRMED
          Severity: normal
          Priority: NOR
         Component: Albums GUI
        AssignedTo: [hidden email]
        ReportedBy: [hidden email]


Version:           2.0.0 (using Devel)
OS:                Linux

Setting "settings" -> "configure digikam" -> "miscellaneous" -> "string
comprison type" to "normal" is ignored in picture thumbnail-view.

I'm using digikam-2.0.0_beta6 with KDE-4.6.80-r1.

Reproducible: Always

Steps to Reproduce:
<maybe optional>
1. start digikam with new profile
     Setting:
       - RAW: open directly
       - Metadata: add info to file
       - preview: load reduced
       - rightclick-action: open a preview
       - use tooltips
</maybe optional>
2. add three photos:
       - 12img.jpg
       - 3img.jpg
       - 4img.jpg
3. add three albums:
       - 1a
       - 23b
       - 4c
4. verify "settings" -> "configure digikam" -> "miscellaneous" -> "string
comprison type" to "natural" (should be default)
5. albums get sorted:
       - 1a
       - 4c
       - 23b
    => OK
6. pictures get sorted:
       - 3img.jpg
       - 4img.jpg
       - 12img.jpg
    => OK
7. change "settings" -> "configure digikam" -> "miscellaneous" -> "string
comprison type" to "normal"
8.  albums get sorted:
       - 1a
       - 23b
       - 4c
    => OK


Actual Results:  
pictures get sorted:
       - 3img.jpg
       - 4img.jpg
       - 12img.jpg

Expected Results:  
pictures get sorted:
       - 12img.jpg
       - 3img.jpg
       - 4img.jpg


The following bugs seem to be the "historical background" of "natural sorting"
and have been found during "is the bug already filed?":
 - Bug #158868
 - Bug #227905

--
Configure bugmail: https://bugs.kde.org/userprefs.cgi?tab=email
------- 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

[Marcel Wiesweg]

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





--- Comment #1 from Marcel Wiesweg <marcel wiesweg gmx de>  2011-06-19 15:50:53 ---
As you observed corretly, natural ordering is always used. I didnt know there
is something to adjust in the settings, and it seems to me like an option which
we really dont need?

--
Configure bugmail: https://bugs.kde.org/userprefs.cgi?tab=email
------- 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

[Peter Albrecht-2]

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





--- Comment #2 from Peter Albrecht <px79 crazymonkeys de>  2011-06-19 16:35:57 ---
(In reply to comment #1)
> As you observed corretly, natural ordering is always used. I didnt know there
> is something to adjust in the settings, and it seems to me like an option which
> we really dont need?

The discussion in bug #227905 ended with the agreement to create this setting.
So I would say: thumbnail view should respect this setting.

I personally don't like "natural string comparison". My picture numbers are
prefixed with zeros (e.g. MyPic00051.jpg). But that is a personal opinion.

--
Configure bugmail: https://bugs.kde.org/userprefs.cgi?tab=email
------- 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

[Gilles Caulier-4]

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


Gilles Caulier <[hidden email]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |[hidden email]




--- Comment #3 from Gilles Caulier <caulier gilles gmail com>  2011-12-14 09:17:23 ---
Peter,

It still valid with digiKam 2.4 ?

Gilles Caulier

--
Configure bugmail: https://bugs.kde.org/userprefs.cgi?tab=email
------- 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

[Peter Albrecht-2]

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





--- Comment #4 from Peter Albrecht <px79 crazymonkeys de>  2011-12-18 08:15:53 ---
Hi Gilles,

yes, this bug is still valid with digiKam 2.4.1 and KDE 4.7.3.

Peter Albrecht

--
Configure bugmail: https://bugs.kde.org/userprefs.cgi?tab=email
------- 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

[Gilles Caulier-4]

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

Gilles Caulier <[hidden email]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
            Summary|Setting "string comprison   |Setting "string comparison
                   |type" is not evaluated in   |type" is not evaluated in
                   |thumbnail-view              |thumbnail-view

--- Comment #5 from Gilles Caulier <[hidden email]> ---
Peter,

What's about this file with digiKam 3.5.0 ?

Gilles Caulier

--
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

[Peter Albrecht-2]

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

--- Comment #6 from Peter Albrecht <[hidden email]> ---
Gilles,

at the moment, I still use digikam 3.1.0. But I plan to switch to 3.5.0 at mid
december.
Than I will test this bug again.

Regards,
    Peter

--
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

[Gilles Caulier-4]

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

--- Comment #7 from Gilles Caulier <[hidden email]> ---
And with 3.1.0, problem still here ?

Gilles Caulier

--
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

[Peter Albrecht-2]

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

--- Comment #8 from Peter Albrecht <[hidden email]> ---
Sorry, but bug is still present in digikam 3.5.0 and 4.10.5:

"string comprison type = normal" is ignored in picture thumbnail-view.

Behaviour still exactly as described in comment #0. (Step 1 can be omitted.)

--
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

[Gilles Caulier-4]

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

Gilles Caulier <[hidden email]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
            Version|2.0.0                       |3.5.0

--
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

[Gilles Caulier-4]

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

Gilles Caulier <[hidden email]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
            Summary|Setting "string comparison  |Setting "string comparison
                   |type" is not evaluated in   |type" is not evaluated in
                   |thumbnail-view              |icon-view

--
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

[Gilles Caulier-4]

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

--- Comment #9 from Gilles Caulier <[hidden email]> ---
Maik,

Another entry where we can study a patch. This touch Image Lister if i
remember.

Gilles

--
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

[bugzilla_noreply]

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

Maik Qualmann <[hidden email]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |RESOLVED
         Resolution|---                         |FIXED
      Latest Commit|                            |http://commits.kde.org/digi
                   |                            |kam/b8566dc4928aa686822c73e
                   |                            |3d5fedac5c91a80f5
   Version Fixed In|                            |5.0.0

--- Comment #10 from Maik Qualmann <[hidden email]> ---
Git commit b8566dc4928aa686822c73e3d5fedac5c91a80f5 by Maik Qualmann.
Committed on 20/02/2016 at 23:47.
Pushed by mqualmann into branch 'master'.

fix string comparison type in the icon views
FIXED-IN: 5.0.0

M  +2    -1    NEWS
M  +2    -0    app/items/digikamimageview.cpp
M  +2    -1    libs/models/imagealbumfiltermodel.cpp
M  +7    -0    libs/models/imagefiltermodel.cpp
M  +1    -0    libs/models/imagefiltermodel.h
M  +10   -4    libs/models/imagesortsettings.cpp
M  +4    -2    libs/models/imagesortsettings.h
M  +2    -0    utilities/importui/main/importui.cpp
M  +11   -5    utilities/importui/models/camitemsortsettings.cpp
M  +4    -2    utilities/importui/models/camitemsortsettings.h
M  +7    -0    utilities/importui/models/importfiltermodel.cpp
M  +1    -0    utilities/importui/models/importfiltermodel.h

http://commits.kde.org/digikam/b8566dc4928aa686822c73e3d5fedac5c91a80f5

--
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

[bugzilla_noreply]

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

[hidden email] changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
          Component|AlbumsView                  |AlbumsView-Icons

--
You are receiving this mail because:
You are the assignee for the bug.