[Digikam-devel] [Bug 137282] New: Comments are lost when copying or moving a picture to another album

[Fabien-5]

------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.
         
http://bugs.kde.org/show_bug.cgi?id=137282         
           Summary: Comments are lost when copying or moving a picture to
                    another album
           Product: digikam
           Version: unspecified
          Platform: Ubuntu Packages
        OS/Version: Linux
            Status: UNCONFIRMED
          Severity: normal
          Priority: NOR
         Component: general
        AssignedTo: digikam-devel kde org
        ReportedBy: fabien.ubuntu gmail com


Version:           0.9-svn (using KDE KDE 3.5.2)
Installed from:    Ubuntu Packages
OS:                Linux

If you try to copy or move a picture to another album, comments are lost.

This only happens if "Save image comments as embedded text" is not active in
digikam settings (Metadata section).
_______________________________________________
Digikam-devel mailing list
[hidden email]
https://mail.kde.org/mailman/listinfo/digikam-devel

[Gilles Caulier-2]

------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.
         
http://bugs.kde.org/show_bug.cgi?id=137282         
caulier.gilles kdemail net changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
          Component|general                     |Metadata
           Priority|NOR                         |VHI
_______________________________________________
Digikam-devel mailing list
[hidden email]
https://mail.kde.org/mailman/listinfo/digikam-devel

[Gilles Caulier-2]

------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.
         
http://bugs.kde.org/show_bug.cgi?id=137282         




------- Additional Comments From caulier.gilles kdemail net  2006-11-13 19:01 -------
This problem must be fixed before 0.9.0-final release
 
Gilles Caulier
_______________________________________________
Digikam-devel mailing list
[hidden email]
https://mail.kde.org/mailman/listinfo/digikam-devel

[Marcel Wiesweg]

------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.
         
http://bugs.kde.org/show_bug.cgi?id=137282         




------- Additional Comments From marcel.wiesweg gmx de  2006-11-14 20:19 -------
DIO::copy is invoked correctly with digikamalbums:// URLs. The question is, why is the ioslave's copy method not invoked?
_______________________________________________
Digikam-devel mailing list
[hidden email]
https://mail.kde.org/mailman/listinfo/digikam-devel

[Gilles Caulier-2]

------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.
         
http://bugs.kde.org/show_bug.cgi?id=137282         
caulier.gilles kdemail net changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |NEW
      everconfirmed|0                           |1



------- Additional Comments From caulier.gilles kdemail net  2006-11-14 20:39 -------
I can confirm this problem on my computers.

Gilles
_______________________________________________
Digikam-devel mailing list
[hidden email]
https://mail.kde.org/mailman/listinfo/digikam-devel

[Marcel Wiesweg]

------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.
         
http://bugs.kde.org/show_bug.cgi?id=137282         




------- Additional Comments From marcel.wiesweg gmx de  2006-11-15 17:38 -------
This is the problem:
If we specify KIO::UDS_LOCAL_PATH in kio_digikamalbums::createUDSEntry, KIO::CopyJob will use the local paths instead, bypassing the copy method builtin in the ioslave.
Specifying KIO::UDS_LOCAL_PATH was added to fix the "Picture is not stored on the local host. Click to load" message in the "File already exists" dialog, fixing a bug report.
(this only applies for KDE >3.4)

Shall we revert the fix for the minor bug, to fix the major bug for the 0.9 release?
_______________________________________________
Digikam-devel mailing list
[hidden email]
https://mail.kde.org/mailman/listinfo/digikam-devel

[Gilles Caulier-2]

------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.
         
http://bugs.kde.org/show_bug.cgi?id=137282         




------- Additional Comments From caulier.gilles kdemail net  2006-11-15 18:09 -------
yes Marcel, and re-open the file in B.K.O with this explanations util we find an alternative solution.

Gilles
_______________________________________________
Digikam-devel mailing list
[hidden email]
https://mail.kde.org/mailman/listinfo/digikam-devel

[Marcel Wiesweg]

------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.
         
http://bugs.kde.org/show_bug.cgi?id=137282         




------- Additional Comments From marcel.wiesweg gmx de  2006-11-15 19:05 -------
SVN commit 605173 by mwiesweg:

Revert fix for bug #122653, as a temporary fix for bug #137282

CCBUG: 137282, 122653


 M  +7 -0      digikamalbums.cpp  


--- trunk/extragear/graphics/digikam/kioslave/digikamalbums.cpp #605172:605173
 @ -1151,11 +1151,18  @
     atom.m_str = QFileInfo(path).fileName();
     entry.append(atom);
 
+    /*
+    // If we provide the local path, a KIO::CopyJob will optimize away
+    // the use of our custom digikamalbums:/ ioslave, which breaks
+    // copying the database entry:
+    // Disabling this as a temporary solution for bug #137282
+    // This code is intended as a fix for bug #122653.
 #if KDE_IS_VERSION(3,4,0)
     atom.m_uds = KIO::UDS_LOCAL_PATH;
     atom.m_str = path;
     entry.append(atom);
 #endif
+    */
 
     return true;
 }
_______________________________________________
Digikam-devel mailing list
[hidden email]
https://mail.kde.org/mailman/listinfo/digikam-devel

[Fabien-5]

------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.
         
http://bugs.kde.org/show_bug.cgi?id=137282         
fabien.ubuntu gmail com changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
         Resolution|                            |WORKSFORME



------- Additional Comments From fabien.ubuntu gmail com  2006-11-17 15:59 -------
I tested it, it's ok. I close the bug.
_______________________________________________
Digikam-devel mailing list
[hidden email]
https://mail.kde.org/mailman/listinfo/digikam-devel