Opened 13 years ago

Closed 12 years ago

#1893 closed bug (Fixed)

Error when adding torrent from Firefox

Reported by: Koroboru Owned by:
Priority: minor Milestone: 1.3.4
Component: GTK UI Version: 1.3.3
Keywords: Cc:

Description

When I'm adding torrent from Firefox using "Open with: Deluge Gtk" I get an error from Deluge like this one: [Errno 2] No such file or directory: u'/home/koroboru/file:/tmp/%5Brutracker.org%5D.t2905444.torrent' This bug appeared after updating deluge to 1.3.2 and firefox to 5.0 so actually I'm not sure this is a deluge's bug.

Attachments (1)

0001-Actually-fix-Firefox-file-prefix-path-error.patch​ (1.7 KB) - added by Simon80 12 years ago.

Download all attachments as: .zip

Change History (5)

comment:1 Changed 13 years ago by Cas

  • Component changed from other to gtkui
  • Milestone changed from Future to 1.3.3
  • Resolution set to fixed
  • Status changed from new to closed
  • Version changed from other (please specify) to 1.3.2

This will be fixed in next release: ​34f816

Workaround from ​forum:

The simplest solution is to edit /usr/share/applications/deluge.desktop and remove %U after deluge-gtk 

Exec=deluge-gtk %U

becomes 

Exec=deluge-gtk

Changed 12 years ago by Simon80

comment:2 Changed 12 years ago by Simon80

  • Milestone changed from 1.3.3 to 1.3.4
  • Resolution fixed deleted
  • Status changed from closed to reopened
  • Version changed from 1.3.2 to 1.3.3

This bug still occurs for some files, because there's more to file URLs than just the ​file:// prefix. Special characters are expressed in URLs % escape sequences, and this needs to be reversed before treating the URL like a local path. Fortunately, urllib provides a convenient function to do the conversion for us.

comment:3 Changed 12 years ago by Cas

The problem was actually a change I made to the gtk client and I have suggested a fix in #1918 which also includes url paths conversion. What is really needed though is a test created to make sure it works for all OS's and that we don't break it again in the future without noticing.

comment:4 Changed 12 years ago by Cas

  • Resolution set to fixed
  • Status changed from reopened to closed

Fix committed to 1.3-stable:​9c9064b2

Note: See TracTickets for help on using tickets.