Bug 458082 - mp4 video failed to update metadata timestamp
Summary: mp4 video failed to update metadata timestamp
Status: RESOLVED FIXED
Alias: None
Product: digikam
Classification: Applications
Component: Plugin-Generic-TimeAdjust (show other bugs)
Version: 7.7.0
Platform: Ubuntu Linux
: NOR normal
Target Milestone: ---
Assignee: Digikam Developers
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2022-08-20 03:26 UTC by bert
Modified: 2022-10-03 09:11 UTC (History)
2 users (show)

See Also:
Latest Commit:
Version Fixed In: 8.0.0
Sentry Crash Report:


Attachments
Screenshot of error in Update time and date dialog (95.36 KB, image/png)
2022-08-20 03:26 UTC, bert
Details

Note You need to log in before you can comment on or make changes to this bug.
Description bert 2022-08-20 03:26:26 UTC
Created attachment 151438 [details]
Screenshot of error in Update time and date dialog

SUMMARY
I try to update the timestamp of an mp4 video. The works correct on jpg photos. But for the video it fails with "Failed to update metadata timestamp" 


STEPS TO REPRODUCE
1. Select video
2. Chose from menu Item-> Adjust Time and Date
3. Set time change and press apply

OBSERVED RESULT
The dialog (see screenshot) sets an access forbidden sign (circle with diagonal stripe) in front of the filename and adds " Failed to update timestamp" as status

EXPECTED RESULT
Time stamp updated



Linux/KDE Plasma: Ubuntu 22.04
(available in About System)
KDE Plasma Version: 
KDE Frameworks Version: Version 5.95.0
Qt Version: Version 5.15.3 (built against 5.15.3)

ADDITIONAL INFORMATION
Comment 1 Maik Qualmann 2022-08-20 05:57:28 UTC
With Exiv2 it is not possible to change the metadata in video files or it is very unstable and can lead to data loss or crashes. Therefore digiKam-7.x.x is only possible to work with the help of XMP sidecar. In digiKam-8.0.0 it is possible to use ExifTool to write metadata, then supported video files can also be modified.

Maik
Comment 2 Maik Qualmann 2022-08-20 06:37:28 UTC
Git commit ce329622f602bdb80d65bffe3cc64869950bd0bd by Maik Qualmann.
Committed on 20/08/2022 at 06:36.
Pushed by mqualmann into branch 'master'.

add ExifTool writing check for the Time Adjust Tool

M  +6    -5    core/dplugins/generic/metadata/timeadjust/timeadjusttask.cpp

https://invent.kde.org/graphics/digikam/commit/ce329622f602bdb80d65bffe3cc64869950bd0bd
Comment 3 Maik Qualmann 2022-08-21 09:31:08 UTC
Not all time metadata can be set without problems. It needs more tests.

Maik
Comment 4 Maik Qualmann 2022-08-21 11:55:09 UTC
Git commit 07c05ab5bbfac380357c051f54b33c942c3038d3 by Maik Qualmann.
Committed on 21/08/2022 at 11:54.
Pushed by mqualmann into branch 'master'.

refactoring Time Adjust Tool

M  +85   -159  core/dplugins/generic/metadata/timeadjust/timeadjusttask.cpp

https://invent.kde.org/graphics/digikam/commit/07c05ab5bbfac380357c051f54b33c942c3038d3
Comment 5 bert 2022-08-21 13:30:08 UTC
Hi Maik,

Thanks for picking this up.
As for testing, let me know if I can assist.
Meanwhile  (as a datapoint) I found a sort of work around:
If I tick only the "File Last Updated" timestamp to be updated, it works. This timestamp is correctly updated and the video moves to the right position in my timeline.

Bert
Comment 6 Maik Qualmann 2022-10-03 09:11:12 UTC
Git commit b7280f263a7a349ca381e9921a75271f67d39389 by Maik Qualmann.
Committed on 03/10/2022 at 09:09.
Pushed by mqualmann into branch 'master'.

add Xmp.xmpDM.shotDate for the Time Adjust Tool
FIXED-IN: 8.0.0

M  +2    -0    core/libs/timeadjust/timeadjustcontainer.cpp

https://invent.kde.org/graphics/digikam/commit/b7280f263a7a349ca381e9921a75271f67d39389