[Digikam-users] More questions about sort order in folders

[Arnd Baecker]

On Wed, 9 Jul 2008, Chris G wrote:

> On Wed, Jul 09, 2008 at 08:07:09PM +0100, Chris G wrote:
> > On Tue, Jul 08, 2008 at 07:38:32PM +0200, Arnd Baecker wrote:
> > >
> > > It seems that
> > > int AlbumFolderViewItem::compare(QListViewItem *i, int col, bool
> > > ascending)
> > > does already something like this.
> > > So maybe this, together with the corresponding QListViewItem
> > > needs to be extended?
> > >
> > OK, thanks, that looks more like the right[ish] place, I'll do some
> > digging around and see if I can understand it.
> >
> ... and after a bit of looking around and fiddling I have got things
> to work the way I want!  :-)
>
> Currently all I have done is to replace the alphabetical sort in "Sort
> Albums - By Folder" with a sort by the date set in the album properties.
> So now I can set the dates in album properties and I get the albums in
> folder view in chronological order - whoopee!  :-)

That's excellent!

> Would you like this developed a little further so that one can switch
> from alphabetical to chronological sorting?  Presumably the way to do
> this would be to add an extra menu option so there is:-
>
>     Sort Albums - By Folder Alphabetical
>                   By Folder Chronological
>                   By Collection
>                   By Date

Sounds good to me.
Gilles, Marcel, what do you think?

> Another thing I would like to add is the ability to force a resort,
> currently one has to switch to a different "Sort Albums" option and
> back again to force a resort.

In which situation do you want to enforce a resort?

> Is this change of general interest or shall I just keep it as a
> private patch which I apply for my own use only?

As far as I can say this is clearly of general interest.

Could you attach your current patch at
https://bugs.kde.org/show_bug.cgi?id=156186
as it addresses the first part of the wish there?
At this point the patch need not be polished etc.,
but I (and others) can have a look at it.

Thanks a lot!

Best, Arnd

P.S. I think your patch would also solve
https://bugs.kde.org/show_bug.cgi?id=158868

Also we should keep in mind that there are some issues
related to computing the right date for an album:
- "Change date of album to exif date of first image"
  https://bugs.kde.org/show_bug.cgi?id=89364
- "Album date average incorrect"
  https://bugs.kde.org/show_bug.cgi?id=121460
  (We need to check whether this one is still happening in current svn...)