Bug 447545 - strange Automatic Date decision in "Move to..." + New Album
Summary: strange Automatic Date decision in "Move to..." + New Album
Status: RESOLVED FIXED
Alias: None
Product: digikam
Classification: Applications
Component: Albums-IconView (other bugs)
Version First Reported In: 7.3.0
Platform: Microsoft Windows Microsoft Windows
: NOR normal
Target Milestone: ---
Assignee: Digikam Developers
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2021-12-26 15:10 UTC by Alexandre Belz
Modified: 2021-12-27 09:51 UTC (History)
2 users (show)

See Also:
Latest Commit:
Version Fixed/Implemented In: 7.5.0
Sentry Crash Report:


Attachments
Screen Capture (590.36 KB, image/png)
2021-12-26 15:10 UTC, Alexandre Belz
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Alexandre Belz 2021-12-26 15:10:56 UTC
Created attachment 144880 [details]
Screen Capture

SUMMARY
***
When I want to move a bunch of selected pictures to a new album, I often used the automatic date proposal.
(Oldest/Average/Newest). But recently It does not work as expected. 
The suggest dates are not relevant to the "Photograph properties" of the group of pictures I selected, but instead to the file creation date of the whole surrounding Album.

***
STEPS TO REPRODUCE
0. From a Album where a lot of pictures reside, with many different dates.
1. Select the enclosed sample jpg picture, taken on 12th Feb 2020.
2. Right click "Move to"
3. Select "New Album"
4. Click Average

OBSERVED RESULT
on my computer, 1st July 2020 is proposed

EXPECTED RESULT
12th Feb 2020 , as this is the dates of the (only) selected picture (file and photograph taken).

SOFTWARE/OS VERSIONS
Windows: 10

ADDITIONAL INFORMATION
I believe that there could be 2 reasons of this "issue".
* analyzed dates seems to be the full/original Album content from where the selection was made. I would expect the analysis would better scope the selected pictures.  I almost consider this sicrepency as a bug.
* analyzed dates seems to be OS File creation dates, not the Photograph taken dates (I would prefer that 2nd option ; this approach could be discussed, I must admit).
Comment 1 Alexandre Belz 2021-12-26 15:12:56 UTC
Sample JPG can be found here :
https://1drv.ms/u/s!Ahp2yXCYf8WmmWkOUmTMI41SRaig?e=dP13QB
Comment 2 Maik Qualmann 2021-12-26 17:42:51 UTC
There was a patch here from an external developer. The album date is set to the newest date of a image in the album. We also had bug reports on this request. He sorted the albums by date so that the sorting is correct after adding new images to the albums. I was also a bit critical of this patch. We may need to make it configurable.

Maik
Comment 3 Maik Qualmann 2021-12-26 17:46:38 UTC
In principle, if you don't add any more images to the album, you can set the average, which should then be retained.

Maik
Comment 4 Alexandre Belz 2021-12-26 17:56:23 UTC
Hi Maik, thank you for your feedback.
On my side, the bug i report is just at Album creation step (from a selection of pictures). At that creation stage, an Album date can be decided (Oldest Average Newest) but none of the 3 are consistent to the picture(s) i've selected.
Comment 5 Maik Qualmann 2021-12-26 18:13:38 UTC
I hope I understand you correctly, in the New Album Dialog, I think the selection of images is being ignored. Since the album date is set to the newest when moving and scanning, you should only edit the date after moving.

Maik
Comment 6 Maik Qualmann 2021-12-26 18:51:29 UTC
The album has to be definitely created, otherwise the date cannot be calculated. I'll hide the buttons in New Album mode.

Maik
Comment 7 Maik Qualmann 2021-12-26 18:55:57 UTC
Git commit bdca73993d6b2f14c95701ba78c7a63bc71e6ad0 by Maik Qualmann.
Committed on 26/12/2021 at 18:54.
Pushed by mqualmann into branch 'master'.

hide date calculation button in new album mode
FIXED-IN: 7.5.0

M  +1    -1    NEWS
M  +3    -0    core/libs/album/widgets/albumpropsedit.cpp

https://invent.kde.org/graphics/digikam/commit/bdca73993d6b2f14c95701ba78c7a63bc71e6ad0
Comment 8 Alexandre Belz 2021-12-27 09:51:43 UTC
Hi Maik,
thank you for your message.

I agree with you, hiding the Date calculation buttons is maybe the best solution if Album date is anyway not meant to be managed at this initial step.

Thank you.

Season's greetings !