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 647917 - Device playlist names sometimes display %-encoded
Device playlist names sometimes display %-encoded
Status: RESOLVED FIXED
Product: banshee
Classification: Other
Component: User Interface
git master
Other All
: Normal normal
: ---
Assigned To: Banshee Maintainers
Banshee Maintainers
: 657526 (view as bug list)
Depends on:
Blocks:
 
 
Reported: 2011-04-16 02:40 UTC by Dustin C. Hatch
Modified: 2011-08-30 17:17 UTC
See Also:
GNOME target: ---
GNOME version: ---

Attachments
%-encoded playlist names (9.13 KB, image/png)
2011-04-16 02:40 UTC, Dustin C. Hatch 		  Details
URL Decode playlist names (1.56 KB, patch)
2011-08-27 23:25 UTC, William Witt 		none Details | Review

Description Dustin C. Hatch 2011-04-16 02:40:23 UTC 
Created attachment 186062 [details]
%-encoded playlist names

The second issue discussed in bug #639947 isn't related to the character encoding problem for the sync options. Names of playlists found on DAP devices can sometimes be displayed in a %-encoded form. Attachment #178723 [details] shows this, and I have attached a screenshot of my own to display it.

For me, performing a sync operation redraws the list and the names are then displayed correctly formatted.
Comment 1 Andrés G. Aragoneses (IRC: knocte) 2011-08-27 22:35:00 UTC 
*** Bug 657526 has been marked as a duplicate of this bug. ***
Comment 2 Andrés G. Aragoneses (IRC: knocte) 2011-08-27 22:37:18 UTC 
I reproduced this problem too. Confirming.

BTW, the screenshot is nice thanks, but even better is the video linked in bug 657526 :)
Comment 3 William Witt 2011-08-27 23:25:46 UTC 
Created attachment 194944 [details] [review]
URL Decode playlist names

This fixes mass storage devices, it doesn't appear to be a problem for Apple or MTP devices, but I don't have either to test.
Comment 4 Andrés G. Aragoneses (IRC: knocte) 2011-08-30 17:17:46 UTC 
Thanks!

I committed your patch with a slight modification (but keeping you as author): use SafeUri.UriToFilename() instead of HttpUtility, and use a new var to prevent the line to grow even longer.

This problem has been fixed in the development version. The fix will be available in the next major software release. Thank you for your bug report.