rTorrent Remove and Delete Data option shows incorrectly

[GoogleCodeExporter]

I don't know if it's a bug or a missing feature. I recently switched seedbox 
and torrent client from Deluge to rtorrent. And I'v notice the remove and 
delete data feature does not actually delete the data. This use to work fine 
with deluge. I believe rtorrent supports this since I can do it using the 
ruTorrent web ui.

Thanks.

Original issue reported on code.google.com by h.gag...@gmail.com on 31 Mar 2012 at 4:15

[GoogleCodeExporter]

Ah... Actually this is not supported, since rtorrent doesn't directly support 
it. (There might be some way with custom methods, like how rutorrent does it, 
but I never figured this out.) 

on the other hand, the option shows up in Transdroid. That's a bug. It should 
shouldn't display this. 

Original comment by erickok@gmail.com on 31 Mar 2012 at 7:57

• Changed title: _rTorrent Remove and Delete Data option shows incorrectly _
• Changed state: Accepted

[GoogleCodeExporter]

[deleted comment]

[GoogleCodeExporter]

[deleted comment]

[GoogleCodeExporter]

[deleted comment]

[GoogleCodeExporter]

If the "remove and delete data" feature could work I would be so so so happy. I 
know rutorrent has has feature and it's very use full. Right now I have to stop 
and remove then torrent in transdroid the login to my mobile ftp client and 
delete the data. Not hard or impossible, but if you delete more than one 
torrent at a time I have to remember each torrent I delete (and my memory isn't 
that good). This is the only feature keeping transdroid for android from being 
a complete solution as a mobile client. 

Original comment by Collin...@gmail.com on 16 Aug 2012 at 10:35

[GoogleCodeExporter]

I too would love this feature. Using transdroid on my galaxy s3 with rutorrent 
+ rtorrent on ubuntu seedbox. 

Its unfortunate you cant send shell cmd like rm -rf $(xmlrpc 
localhost:scgi_port d.get_base_path) ... i would consider adding my ssh private 
key to transdroid for that to work.

Original comment by kevinf...@gmail.com on 7 Oct 2012 at 9:26

[GoogleCodeExporter]

If you already have rutorrent + erasedata plugin you can easily achieve this by 
adding 

RemoveTask removeTask = (RemoveTask) task;
if (removeTask.includingData()) {
makeRtorrentCall("d.set_custom5", new String[] { 
task.getTargetTorrent().getUniqueID(), "1" });
}

to the Remove switch statement in RtorrentAdapter.java (before the call to 
d.erase).

I think the plugin works by setting up an 'on erase' command that calls its own 
scripts if d.get_custom5=1 for that torrent (and it gets set to 1 just before 
d.erase is called, like above). Maybe you can do something similar or add some 
kind of 'enable ruTorrent commands' option that makes use of ruTorrent and its 
plugins for rtorrent servers?

Original comment by tbri...@googlemail.com on 12 Nov 2012 at 1:05

[GoogleCodeExporter]

as a side note, i wouldn't use the 

rm -rf $(xmlrpc localhost:scgi_port d.get_base_path)

idea as d.get_base_path may contain more than the torrents files! (especially 
if you use ruTorrents "don't add torrent's name to path" feature)

Original comment by tbri...@googlemail.com on 12 Nov 2012 at 1:07

[GoogleCodeExporter]

Nice! Well reseached. I will definitely add this for the next release.

Original comment by erickok@gmail.com on 12 Nov 2012 at 2:37

• Changed state: Started

[GoogleCodeExporter]

I too will be excited to see this implemented.

Thanks

Original comment by kma...@gmail.com on 1 Dec 2012 at 12:53

[GoogleCodeExporter]

definatly another +1, as the last few updates for chrome and firefox mobile 
seem to have broken my long press remove/delete function using my tablet with 
rutorrent, hence transdroid is my only hope :)

Original comment by scottm8...@gmail.com on 12 Dec 2012 at 5:15

[GoogleCodeExporter]

Instead of removing the option, the feaure was actually added. Many thanks to 
Tom Biden! Will be in the next release. See r18d64e18356c Note that you need 
the ruTorrent erasedata plugin installed.

Original comment by erickok@gmail.com on 16 Jan 2013 at 2:44

• Changed state: Fixed

[GoogleCodeExporter]

Great news! 

Thanks a lot for this.

Original comment by h.gag...@gmail.com on 16 Jan 2013 at 3:29

[GoogleCodeExporter]

Any ETA on the next release date that will contain this ?

Original comment by kma...@gmail.com on 16 Jan 2013 at 3:40

[GoogleCodeExporter]

Should be later today.

Original comment by erickok@gmail.com on 16 Jan 2013 at 5:09

[GoogleCodeExporter]

That is AWESOME news :)  Thanks!

Original comment by kma...@gmail.com on 16 Jan 2013 at 5:11