| Summary: | Date for Videos should be sourced from Video metadata instead of filesystem file modify date | ||
|---|---|---|---|
| Product: | [Applications] kphotoalbum | Reporter: | Victor Lobo <victor.ip.lobo> |
| Component: | Backend | Assignee: | KPhotoAlbum Bugs <kphotoalbum-bugs-null> |
| Status: | RESOLVED FIXED | ||
| Severity: | normal | CC: | johannes, victor.ip.lobo |
| Priority: | NOR | ||
| Version First Reported In: | GIT master | ||
| Target Milestone: | --- | ||
| Platform: | openSUSE | ||
| OS: | Linux | ||
| Latest Commit: | https://invent.kde.org/graphics/kphotoalbum/-/commit/328641be8027d54d80c7dcb9101fc8c074aaa778 | Version Fixed/Implemented In: | |
| Sentry Crash Report: | |||
|
Description
Victor Lobo
2023-07-23 06:06:35 UTC
Just some notes for a potential fix: A good location to read creation_date would be in VideoLengthExtractor (although that needs some change at least in its name then). The following command can be used to read the creation_date in addition to the video length: `ffprobe -v 0 -show_entries format_tags=creation_time:format=duration -of default=noprint_wrappers=1:nokey=0` One thing that needs working out if we use ffprobe, though: some video files have local time, some have UTC. I.e. some (most?) (phone-)cameras store the time as UTC, but no offset. A possibly relevant merge request was started @ https://invent.kde.org/graphics/kphotoalbum/-/merge_requests/55 Git commit 328641be8027d54d80c7dcb9101fc8c074aaa778 by Johannes Zarl-Zierl. Committed on 23/11/2025 at 21:54. Pushed by johanneszarl into branch 'master'. Update Changelog Add Changelog for mr/55 (commits related to 159be3502310a69724594a899e52574e31e4aab6) M +1 -0 CHANGELOG.md M +1 -1 MainWindow/Window.cpp https://invent.kde.org/graphics/kphotoalbum/-/commit/328641be8027d54d80c7dcb9101fc8c074aaa778 |