After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 342424 - Patch to correct drag-and-drop from a source to another
Patch to correct drag-and-drop from a source to another
Status: RESOLVED OBSOLETE
Product: rhythmbox
Classification: Other
Component: User Interface
0.9.3
Other Linux
: Normal normal
: ---
Assigned To: RhythmBox Maintainers
RhythmBox Maintainers
Depends on:
Blocks:
 
 
Reported: 2006-05-20 15:40 UTC by Sylvain Defresne
Modified: 2006-05-22 10:05 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
correct-dnd-from-one-source-to-another.patch (983 bytes, patch)
2006-05-20 15:41 UTC, Sylvain Defresne
none Details | Review

Description Sylvain Defresne 2006-05-20 15:40:40 UTC
Drag-and-drop from a source to another in rhythmbox (for example from an intelligent playlist to the queued song list) does not work if the filename contains some character (for example an '?'). This is because the drag-n-drop data is the full uri to the file, fully escaped, whereas the path stored in the rhythmbox database does not escape some characters.
Comment 1 Sylvain Defresne 2006-05-20 15:41:20 UTC
Created attachment 65904 [details] [review]
correct-dnd-from-one-source-to-another.patch

The sus-mentioned patch. It simply call rb_canonicalise_uri before performing a lookup in the database. Another option would have been to perform to use the fully escaped uri when inserting the song in the database.
Comment 2 James "Doc" Livingston 2006-05-22 10:05:54 UTC
Thanks for working on this, however hewer versions of RB should canonicalise URIs before inserting them in the db - so this shouldn't be needed (I can't reproduce your problem).

Because of that I'm marking OBSOLETE, but feel free to re-open if you can reproduce the issue against a newer version (preferably cvs).