Bug 456749 - Cross-platform filename support
Summary: Cross-platform filename support
Status: RESOLVED FIXED
Alias: None
Product: digikam
Classification: Applications
Component: Database-Mysql (show other bugs)
Version: 7.7.0
Platform: Other Linux
: NOR normal
Target Milestone: ---
Assignee: Digikam Developers
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2022-07-15 10:50 UTC by spam-receiver
Modified: 2022-07-16 15:16 UTC (History)
2 users (show)

See Also:
Latest Commit:
Version Fixed In: 8.0.0


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description spam-receiver 2022-07-15 10:50:15 UTC
SUMMARY
I use Digikam with MySQL backend, and the photos stored on a WebDAV share.   
Until now, I was solely on Windows, hence I was able to mount the WebDAV share always on the same path.

Now, I additionally want to use a Ubuntu desktop.
The WebDAV share is mounted somewhere (auto-location determined by Ubuntu).
I pointed Digikam on Ubuntu to the same MySQL database and to the WebDAV mount point on Ubuntu, and voila, my photos were there.

Then I realized that this is not true: 
* Digikam thinks the The photos are at ``N:\path\to\photos``
* But on Ubuntu they are at ``/path/to/mountpoint/dav:host=server,ssl=true/path/to/phtotos``

Three topics need to be considered:
1. Photo path in the database should always be relative to the base path.
2. Photo path in the database should be platform-unaware.
3. There should be a way to select a base path which is already in the database, and redirect it to a path in the local installation.


STEPS TO REPRODUCE
1. Photo storage on network location
2. Digikam on Windows with MySQL backend 
3. Digikam on Linux with same MySQL backend
4. Try to manage the same photos (on network location) using the same database

OBSERVED RESULT
Not possible, because base paths to photo storage on network location don't match between Windows and Linux.

EXPECTED RESULT
Same photos and same database should be used on Windows and on Linux.

SOFTWARE/OS VERSIONS
Windows: Win10
macOS: ./.
Linux/KDE Plasma: Ubuntu LTS
(available in About System)
KDE Plasma Version: 
KDE Frameworks Version: 
Qt Version: 

ADDITIONAL INFORMATION
./.
Comment 1 Maik Qualmann 2022-07-15 10:55:59 UTC

*** This bug has been marked as a duplicate of bug 384820 ***
Comment 2 Maik Qualmann 2022-07-15 11:03:31 UTC
There is currently an undocumented feature. It is possible to add "&/path/to/mount/point" to the network path in the AlbumRoots database table. I plan to make this available in the GUI.

Maik
Comment 3 spam-receiver 2022-07-15 11:13:00 UTC
(In reply to Maik Qualmann from comment #2)
> There is currently an undocumented feature. It is possible to add
> "&/path/to/mount/point" to the network path in the AlbumRoots database
> table. I plan to make this available in the GUI.
> 
> Maik

Can you please explain in detail how the database row should look like after I have added the second base path?
Comment 4 spam-receiver 2022-07-15 11:25:07 UTC
(In reply to spam-receiver from comment #3)
> (In reply to Maik Qualmann from comment #2)
> > There is currently an undocumented feature. It is possible to add
> > "&/path/to/mount/point" to the network path in the AlbumRoots database
> > table. I plan to make this available in the GUI.
> > 
> > Maik
> 
> Can you please explain in detail how the database row should look like after
> I have added the second base path?

I tried (not the question mark in the middle):

networkshareid:?mountpath=N:/path/to/photos?/run/user/1000/gvfs/dav:host=server,ssl=true,user=xyz/path/to/photos

but this broke both, WIndows and Linux... :-(
Comment 5 Maik Qualmann 2022-07-15 11:26:44 UTC
Table: AlbumRoots
Column: identifier

bevor:

networkshareid:?mountpath=N:\path\to\photos

after:

networkshareid:?mountpath=N:\path\to\photos&mountpath=/path/to/another/mountpoint

More paths could be appended, but each path may only exist once under the OS.

Maik
Comment 6 Maik Qualmann 2022-07-15 11:29:09 UTC
Use / instead of \ in my example.

Maik
Comment 7 spam-receiver 2022-07-15 11:52:58 UTC
(In reply to Maik Qualmann from comment #5)
> Table: AlbumRoots
> Column: identifier
> 
> bevor:
> 
> networkshareid:?mountpath=N:\path\to\photos
> 
> after:
> 
> networkshareid:?mountpath=N:\path\to\photos&mountpath=/path/to/another/
> mountpoint
> 
> More paths could be appended, but each path may only exist once under the OS.
> 
> Maik

Sorry, but I can't get this working. I tried with different (existing and non-existing) paths after the second question mark.
Any second path breaks the base path on both, Windows and Linux.
It seems that the feature isn't included in Digikam 7.7.
Comment 8 Maik Qualmann 2022-07-15 14:30:02 UTC
This feature has been around for a long time, your mistake is the second question mark, there is only one question mark.
The second path starts with: &mountpath=

Maik
Comment 9 spam-receiver 2022-07-15 14:46:31 UTC
(In reply to Maik Qualmann from comment #8)
> This feature has been around for a long time, your mistake is the second
> question mark, there is only one question mark.
> The second path starts with: &mountpath=
> 
> Maik

Oh no, stupid me. Can't tell an ampersand from a question mark...
Thank you very much and my apologies.

The undocumented feature works as expected.
This issue can be closed.
Comment 10 Maik Qualmann 2022-07-16 15:16:15 UTC
Git commit 6a630aeab30849225015826aa185cc8a8516cc74 by Maik Qualmann.
Committed on 16/07/2022 at 15:14.
Pushed by mqualmann into branch 'master'.

add support for alternate network paths in the collection view
Related: bug 384820
FIXED-IN: 8.0.0

M  +3    -1    NEWS
M  +4    -0    core/libs/database/collection/collectionlocation.h
M  +6    -3    core/libs/database/collection/collectionmanager.h
M  +5    -3    core/libs/database/collection/collectionmanager_location.cpp
M  +7    -2    core/libs/database/collection/collectionmanager_p.cpp
M  +2    -3    core/libs/database/collection/collectionmanager_p.h
M  +206  -17   core/utilities/setup/collections/setupcollectionview.cpp
M  +21   -8    core/utilities/setup/collections/setupcollectionview.h

https://invent.kde.org/graphics/digikam/commit/6a630aeab30849225015826aa185cc8a8516cc74