Bug 393728 - Reread metadata from Video uses sidecar only
Summary: Reread metadata from Video uses sidecar only
Status: RESOLVED FIXED
Alias: None
Product: digikam
Classification: Applications
Component: Metadata-Video (show other bugs)
Version: 6.0.0
Platform: Appimage Linux
: NOR normal
Target Milestone: ---
Assignee: Digikam Developers
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2018-05-01 12:46 UTC by meku
Modified: 2018-05-10 06:02 UTC (History)
1 user (show)

See Also:
Latest Commit:
Version Fixed In: 6.0.0


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description meku 2018-05-01 12:46:51 UTC
The metadata available for video files has improved significantly in v6.0.0 however the videos saved in older versions of Digikam do not have this metadata in their sidecar files.

The action of "Reread metadata" does not read Video metadata if there is sidecar present and "Read from sidecar" setting is enabled.

Disabling sidecars will read in the Video metadata but all captions, tags, GPS, or other user data assigned to the video is lost.

Ideally "Reread metadata" could read both the Video and Sidecar metadata and combine together. For example: GPS location info can exist in either the Video file or Sidecar and whichever is present will be imported.
Comment 1 Maik Qualmann 2018-05-01 13:08:24 UTC
DigiKam-6.0.0 cleans up the database on a re-read metadata when all 8 metadata write options are enabled. As a workaround, you could disable a write option. Reading from image and sidecar I see not as a good option. We will come up with something.

Maik
Comment 2 meku 2018-05-02 14:21:08 UTC
(In reply to Maik Qualmann from comment #1)
> DigiKam-6.0.0 cleans up the database on a re-read metadata when all 8
> metadata write options are enabled. As a workaround, you could disable a
> write option. Reading from image and sidecar I see not as a good option. We
> will come up with something.
> 
> Maik

Thanks that workaround appears to work for the purpose of updating the sidecar files.

It seems if "Read from sidecar" is enabled then the Metadata viewer still does not show certain metadata such as dimensions or GPS, even when this is in the sidecar.
Comment 3 Maik Qualmann 2018-05-02 18:58:51 UTC
Git commit a45a706a0f9b97c2ad1a57c93bf2a0d3d54bd7bc by Maik Qualmann.
Committed on 02/05/2018 at 18:57.
Pushed by mqualmann into branch 'master'.

add delete the metadata from the database as option in the setup

M  +0    -15   core/libs/database/coredb/coredb.cpp
M  +2    -1    core/libs/database/item/imagescanner.cpp
M  +3    -0    core/libs/dmetadata/metadatasettingscontainer.cpp
M  +1    -0    core/libs/dmetadata/metadatasettingscontainer.h
M  +78   -42   core/utilities/setup/metadata/setupmetadata.cpp
M  +1    -0    core/utilities/setup/metadata/setupmetadata.h

https://commits.kde.org/digikam/a45a706a0f9b97c2ad1a57c93bf2a0d3d54bd7bc
Comment 4 Maik Qualmann 2018-05-02 19:27:44 UTC
(In reply to meku from comment #2)
> It seems if "Read from sidecar" is enabled then the Metadata viewer still
> does not show certain metadata such as dimensions or GPS, even when this is
> in the sidecar.

GPS information from the sidecar are only available in the XMP tab from metadata viewer. I think we can now close the bug, it is now possible to re-scan the files and keep the old information in the database.

Maik
Comment 5 Maik Qualmann 2018-05-10 06:02:57 UTC
The new FFmpeg video EXIF parser must always read from the image. With the new option in the setup to clean up metadata, this behavior has also become clearer. I close the bug now.

Maik