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 551035 - First "Import Folder" reports "Operation not supported"
First "Import Folder" reports "Operation not supported"
Status: RESOLVED FIXED
Product: rhythmbox
Classification: Other
Component: general
HEAD
Other Linux
: Normal normal
: ---
Assigned To: RhythmBox Maintainers
RhythmBox Maintainers
: 550856 568024 (view as bug list)
Depends on:
Blocks:
 
 
Reported: 2008-09-05 18:59 UTC by Pedro Villavicencio
Modified: 2009-01-17 00:10 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
rhythmbox debug log (81.86 KB, text/plain)
2008-09-05 19:00 UTC, Pedro Villavicencio
  Details
Proposed patch (1.05 KB, patch)
2009-01-16 21:22 UTC, Michael Terry
committed Details | Review

Description Pedro Villavicencio 2008-09-05 18:59:18 UTC
rhythmbox version is trunk and GNOME is trunk also, steps to reproduce it:

1. Start Rhythmbox for the first time (or after deleting .gnome2/rhythmbox).
2. Choose "Music" > "Import Folder".

What happens: A filepicker appears with the error "The folder contents could not be displayed - Operation not supported".
What should happen: A filepicker appears with no error message.

this bug was also reported in Ubuntu here:

https://bugs.edge.launchpad.net/ubuntu/+source/rhythmbox/+bug/242013

Ubuntu rhythmbox version is 0.11.6svn20080903

screenshot of the error:

http://launchpadlibrarian.net/15684986/Screenshot-rhythmbox.png

Will attach the rhythmbox log to the report, thanks you.
Comment 1 Pedro Villavicencio 2008-09-05 19:00:53 UTC
Created attachment 118118 [details]
rhythmbox debug log
Comment 2 Jonathan Matthew 2008-09-05 23:18:46 UTC
*** Bug 550856 has been marked as a duplicate of this bug. ***
Comment 3 Jonathan Matthew 2008-09-07 06:31:32 UTC
We do two things with the file chooser that other apps may not:
- add a shortcut to the user's music directory
- set local-only to FALSE

The shortcut is unlikely to be the cause (in the attached debug output, the folder is /home/pvillavi, which is pretty safe).  Do you have any network shares that show up in the file chooser?
Comment 4 Pacho Ramos 2008-09-18 12:08:43 UTC
In my case I get the same problem with and without any network share
Comment 5 Michael Terry 2009-01-16 21:22:10 UTC
Created attachment 126612 [details] [review]
Proposed patch

Whoops, didn't see this bug before filing bug 568024 (which has easy reproduction steps, btw).  Here's a patch that fixes this error by checking for the empty string.
Comment 6 Michael Terry 2009-01-16 21:22:49 UTC
*** Bug 568024 has been marked as a duplicate of this bug. ***
Comment 7 Michael Terry 2009-01-16 21:28:48 UTC
I'm reopening bug since I've provided easy reproducability steps.  (i.e. the questions previously asked don't matter)

I'll copy and paste from the dup, just so you don't have to hunt them down:

To reproduce:
1. Open gconf-editor
2. Find /apps/rhythmbox/state/add_dir
3. Right click and "Unset Key"
4. Open Rhythmbox
5. Choose "Music->Import Folder..."
Comment 8 Jonathan Matthew 2009-01-17 00:10:58 UTC
Thanks for tracking this down.  I don't see much of a reason to use rb_music_dir() when state/add_dir isn't set, though.  If the user's music library is actually in that directory, anything in there should already have been imported.  If not, it's not a useful guess as to where they want to import from.  We already add a shortcut to the file chooser for that directory.  If the directory doesn't exist, you get the same 'operation not supported' error that we're trying to fix here.

2009-01-17  Jonathan Matthew  <jonathan@d14n.org>

        based on patch by:  Michael Terry  <michael.terry@canonical.com>

        * shell/rb-shell.c: (rb_shell_cmd_add_folder_to_library),
        (rb_shell_cmd_add_file_to_library):
        Don't try to set the file chooser's current location to '' when the
        state/add_dir gconf key isn't set.  Fixes #551035.