Add Torrent Dialog's ' Download Location' not set correctly when directory typed into Other->Location field

[Serge]

When I add a new torrent from Mozilla Firefox by choosing "Open with" it is difficult to set up desired save path in adding torrent dialog. I sighted that if I switch between tabs in that dialog then save path sets correctly, but else it sets to my home directory. I run it in KDE.

[Calum]

I am sorry but I am not sure what you are asking for. You could try wording it differently or maybe supply a screenshot of the issue or proposed solution.

[Serge]

1. In FireFox choose some URI with torrent-file; when FF ask what to do with it choose point "Open with Deluge"
2. Deluge starts (or pops up if started -- no matter) and the "Add torrents" dialog brings up.
3. Switch to options tab and try to choose a save path.
4. Press Add button and see that save path (need to turn on this column first) is... your home directory.

I was trying to research this issue but as far as I see this maybe related to that I running KDE. And one more: if between 3 and 4 points above I switched between tabs inside the dialog save path was set correctly.

[Calum]

Thanks I understand the issue now. This could well be a KDE specific issue but I will need to download a KDE distro in order to test it.

[ferp]

I also have this exact same issue in Deluge 1.3.5 on MacOS 10.7.5 Re-adding the torrent (without exiting Deluge) will save to the user-specified location. Re-adding after restarting Deluge will ignore the user-specified location and save to $HOME.

[Calum]

This is odd as it has also been reported on ​Fedora F17

I will look into this further but I could not replicate on KDE specific distro and unless I can replicate it will be very difficult to determine the issue.

[hadrons123]

I have gnome 3.4 in Fedora 17 and Arch linux and it happens to me too. I have the latest Deluge 1.3.5.

[Calum]

I have just downloaded and tested with a fresh virutalbox Fedora 17 and I still cannot replicate this. There must be something in the steps that is being omitted.

Are there any errors in logs?

[hadrons123]

Did you try like this?

Description of problem: When you add a torrent you should be able to download it to a user specified location on your hard disk. But in Deluge 1.3.5 when you select file->add a torrent->options->downloadlocation-> other(scroll down to the lowest in the menu). What ever location you set in "other" is not recognized by the application and it sets itself to default /home/user/Downloads.

So basically the "other" option is not working.

[Calum]

I could select any directory with the Other option and it would select it and start downloading correctly.

Could try with a new Deluge config (-c switch) or test for yourself with a ​Virtualbox Fedora it may provide more information.

[hadrons123]

I could not make it download to my preferred folder with the Other option, Yet. The download starts but at the wrong location. I am using 64bit of Arch, Fedora. I tried both of them right now.

But about the config switch, how do I do it?

[Calum]

| But about the config switch, how do I do it?

deluge -c ~/deluge_config

Are you using libtorrent 0.16?

[hadrons123]

my libtorrent version is libtorrent: 0.15.9.0 in fedora 17.

I tried your method but I am getting the same behaviour. deluge -c ~/deluge_config

If there is a way I could post a log output I would be happy to post.

[ferp]

The above log is from MacOS 10.7.5 and shows me adding a .torrent from $HOME/Downloads, specifying a save location of /var/tmp/, and you can see the torrent get added with options[download_location] set to $HOME instead of /var/tmp.

[ferp]

Same behavior on Natty with a fresh install of 1.3.1-1ubuntu2, by the way.

[hadrons123]

The debug log

[Calum]

I have made a ​fedora screencast to show what I think is being reported and what I am seeing, just in case I have got the wrong end of the stick...

[ferp]

Yup that's the right flow. Except, in my case, I specified the directory by typing /var/tmp, which brings up the "Location" field in the "Select A Folder" dialog. When I do it your way, navigating to /var/tmp/ with the mouse, it works.

[Calum]

Ah! I can finally replicate the issue by typing in the location field, thanks ferp.

Well now we know neither me nor anyone else here is crazy :P

[Calum]

Looking into this issue it appears that using get_current_folder() in the file chooser will not return the correct folder in the above situation and the solution is to simply replace it with get_filename().

​PyGTK Doc: get-filename()

[Calum]

Fixed 1.3-stable: ​aa969fd830

[Serge]

I applied your patch, but...

1. In FireFox choose some URI with torrent-file; when FF ask what to do with it choose point "Open with Deluge"
2. Deluge starts (or pops up if started -- no matter) and the "Add torrents" dialog brings up.
3. Switch to options tab and try to choose a save path (using "Other..." option).
4. Press Add button and see that save path (need to turn on this column first) is... STILL your home directory.

If I try to reopen torrent file and if I choose previously chosen (and therefore saved in pop up list which contain $HOME, Desktop, File system and "Other..." options) then Download location set correctly.

[hadrons123]

I still have the same issue too. The patch doesn't resolve it. thanks beaux_monde for bringing it up.

[Calum]

Please take a screencast and share the link to it.

[Serge]

Replying to Cas:

Please take a screencast and share the link to it.

​http://rghost.net/40808058

[Calum]

Please add the following log lines and post the log output from both working and non-working steps.

deluge/ui/gtkui/addtorrentdialog.py

@@ -406 +406 @@
         if client.is_localhost():
             options["download_location"] = \
                 self.glade.get_widget("button_location").get_filename()
+            log.error("DwnLoc filename: %s", self.glade.get_widget("button_location").get_filename())
+            log.error("DwnLoc uri: %s", self.glade.get_widget("button_location").get_uri())
+            log.error("DwnLoc curr fold: %s", self.glade.get_widget("button_location").get_current_folder())
+           log.error("DwnLoc config: %s", options["download_location"])
         else:
             options["download_location"] = \
                 self.glade.get_widget("entry_download_path").get_text()

[Serge]

~$ deluge

** (deluge:16202): WARNING **: Trying to register gtype 'GMountMountFlags' as enum when in fact it is of type 'GFlags'

** (deluge:16202): WARNING **: Trying to register gtype 'GDriveStartFlags' as enum when in fact it is of type 'GFlags'

** (deluge:16202): WARNING **: Trying to register gtype 'GSocketMsgFlags' as enum when in fact it is of type 'GFlags'
Traceback (most recent call last):
  File "/usr/bin/deluge", line 9, in <module>
    load_entry_point('deluge==1.3.5', 'gui_scripts', 'deluge')()
  File "/usr/lib/python2.7/site-packages/deluge/main.py", line 132, in start_ui
    UI(options, args, options.args)
  File "/usr/lib/python2.7/site-packages/deluge/ui/ui.py", line 149, in __init__
    from deluge.ui.gtkui.gtkui import GtkUI
  File "/usr/lib/python2.7/site-packages/deluge/ui/gtkui/__init__.py", line 1, in <module>
    from gtkui import start
  File "/usr/lib/python2.7/site-packages/deluge/ui/gtkui/gtkui.py", line 91, in <module>
    from addtorrentdialog import AddTorrentDialog
  File "/usr/lib/python2.7/site-packages/deluge/ui/gtkui/addtorrentdialog.py", line 413
    log.error("DwnLoc curr fold: %s", self.glade.get_widget("button_location").get_current_
      ^
SyntaxError: invalid syntax

[Serge]

Replying to beaux_monde:

~$ deluge

** (deluge:16202): WARNING **: Trying to register gtype 'GMountMountFlags' as enum when in fact it is of type 'GFlags'

** (deluge:16202): WARNING **: Trying to register gtype 'GDriveStartFlags' as enum when in fact it is of type 'GFlags'

** (deluge:16202): WARNING **: Trying to register gtype 'GSocketMsgFlags' as enum when in fact it is of type 'GFlags'
Traceback (most recent call last):
  File "/usr/bin/deluge", line 9, in <module>
    load_entry_point('deluge==1.3.5', 'gui_scripts', 'deluge')()
  File "/usr/lib/python2.7/site-packages/deluge/main.py", line 132, in start_ui
    UI(options, args, options.args)
  File "/usr/lib/python2.7/site-packages/deluge/ui/ui.py", line 149, in __init__
    from deluge.ui.gtkui.gtkui import GtkUI
  File "/usr/lib/python2.7/site-packages/deluge/ui/gtkui/__init__.py", line 1, in <module>
    from gtkui import start
  File "/usr/lib/python2.7/site-packages/deluge/ui/gtkui/gtkui.py", line 91, in <module>
    from addtorrentdialog import AddTorrentDialog
  File "/usr/lib/python2.7/site-packages/deluge/ui/gtkui/addtorrentdialog.py", line 413
    log.error("DwnLoc curr fold: %s", self.glade.get_widget("button_location").get_current_
      ^
SyntaxError: invalid syntax

Sorry, it looks like I'm wrong

[Calum]

Sorry its my copy and paste mistake, try again with amended diff.

[Serge]

[ERROR   ] 14:48:49 addtorrentdialog:411 DwnLoc filename: /home/serge
[ERROR   ] 14:48:49 addtorrentdialog:412 DwnLoc uri: file:///home/serge
[ERROR   ] 14:48:49 addtorrentdialog:413 DwnLoc curr fold: /home/serge
[ERROR   ] 14:48:49 addtorrentdialog:414 DwnLoc config: /home/serge
[ERROR   ] 14:48:51 addtorrentdialog:411 DwnLoc filename: /home/serge
[ERROR   ] 14:48:51 addtorrentdialog:412 DwnLoc uri: file:///home/serge
[ERROR   ] 14:48:51 addtorrentdialog:413 DwnLoc curr fold: /home/serge
[ERROR   ] 14:48:51 addtorrentdialog:414 DwnLoc config: /home/serge
[ERROR   ] 14:49:02 addtorrentdialog:411 DwnLoc filename: /home/serge
[ERROR   ] 14:49:02 addtorrentdialog:412 DwnLoc uri: file:///home/serge
[ERROR   ] 14:49:02 addtorrentdialog:413 DwnLoc curr fold: /home/serge
[ERROR   ] 14:49:02 addtorrentdialog:414 DwnLoc config: /home/serge
[ERROR   ] 14:49:35 addtorrentdialog:411 DwnLoc filename: /home/serge/temp
[ERROR   ] 14:49:35 addtorrentdialog:412 DwnLoc uri: file:///home/serge/temp
[ERROR   ] 14:49:35 addtorrentdialog:413 DwnLoc curr fold: /home/serge/temp
[ERROR   ] 14:49:35 addtorrentdialog:414 DwnLoc config: /home/serge/temp
[ERROR   ] 14:49:49 addtorrentdialog:411 DwnLoc filename: /home/serge/temp
[ERROR   ] 14:49:49 addtorrentdialog:412 DwnLoc uri: file:///home/serge/temp
[ERROR   ] 14:49:49 addtorrentdialog:413 DwnLoc curr fold: /home/serge/temp
[ERROR   ] 14:49:49 addtorrentdialog:414 DwnLoc config: /home/serge/temp

[Calum]

What is that log of, the bug? Please provide context.

Also please use the WikiFormatting code tags.

[Serge]

How do you fix it?

[Calum]

I cannot replicate any further issues to the one I fixed and you have not provided any more information which I asked for in my last comment, so I closed it.

[pvgoran]

It's a bug in GTK+, not in Deluge: ​https://bugzilla.gnome.org/show_bug.cgi?id=684128

Though Deluge devs could try to find and implement a work-around...

From the user perspective, I found out that the following allows me to use the directory I choose: select "Other", choose a directory, then switch to "File System" in the drop-down and switch back to the chosen directory again.

The other option is to downgrade GTK+ to 2.24.10.

[Serge]

It looks like a true 'cause I had some investigation of this bug (in deluge) and I have got the same result.

[hadrons123]

The bug is real trouble for lot of people.see here ​https://bbs.archlinux.org/viewtopic.php?pid=1173740

[Serge]

wow! I use just the arch distro... :-)

[hadrons123]

It happens in fedora 17 too.

[Calum]

I had a feeling from the last debug logs that it was GTK issue and it's strange that I did not see this in F17 when testing previously however updated it and now see the issue along with a temporary workaround:

deluge/ui/gtkui/addtorrentdialog.py

@@ -147 +147 @@
         self.core_config = {}
 
         self.glade.get_widget("notebook1").connect("switch-page", self._on_switch_page)
+        self.glade.get_widget("button_location").connect("current-folder-changed", self._on_folder_selected)
+        self.on_folder_selected_path = ""
+
+    def _on_folder_selected(self, widget):
+        log.debug("Selected folder path: %s", widget.get_filename())
+        self.on_folder_selected_path = widget.get_filename()
 
     def start(self):
         self.update_core_config()
@@ -405 +411 @@
 
         if client.is_localhost():
             options["download_location"] = \
-                self.glade.get_widget("button_location").get_filename()
+                self.on_folder_selected_path
+            log.debug("Dl folder: %s", self.glade.get_widget("button_location").get_filename())
+            log.debug("Dl signal: %s", self.on_folder_selected_path)
         else:
             options["download_location"] = \
                 self.glade.get_widget("entry_download_path").get_text()

[Serge]

It looks like this workaround solves the problem.

[MasterCATZ]

self.core_config = {}

        self.builder.get_object("notebook1").connect("switch-page", self._on_switch_page)

    def start(self):
        self.update_core_config()

mine did not have

self.glade.get_widget("notebook1").connect("switch-page", self._on_switch_page)  

either so do I also add that and remove

self.builder.get_object("notebook1").connect("switch-page", self._on_switch_page)

??

do I change these self.builder to self.glade as well ?

if client.is_localhost():
            options["download_location"] = \
                self.builder.get_object("button_location").get_filename()
        else:
            options["download_location"] = \
                self.builder.get_object

using git master afak I would have thought the patch would have already been applied to it ?

[Calum]

Since this is a bug in GTK, this hacky workaround will not be applied to our codebase. I am keeping the ticket open to keep track of the upstream bug: ​https://bugzilla.gnome.org/show_bug.cgi?id=684128

If you are using git master then self.builder is the new object reference so you simply replace in the diff code above self.glade.get_widget with self.builder.get_object and it should work.

[Calum]

Fixed in GTK