Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix deluged move_torrent() when seed location is specified in the configuration #7586

Merged
merged 1 commit into from
Jan 15, 2020
Merged

Fix deluged move_torrent() when seed location is specified in the configuration #7586

merged 1 commit into from
Jan 15, 2020

Conversation

jdrouhard
Copy link
Contributor

The deluge RPC client is not initialized on a deluged API client until after the call to connect(), so we need to ensure connect() is called to initialize the RPC client and verify the authentication before calling any RPC methods. move_torrent() is called on a fresh deluged client from process_tv.py without any other previous connection attempt.

The RPC method for moving torrents (core.move_storage) also takes a list of torrent IDs instead of a single ID, so pass a list (of one) to the call instead.

This fix has been tested and verified.

…figuration

The deluge RPC client is not initialized on a deluged API client until
after the call to connect(), so we need to ensure connect() is called to
initialize the RPC client and verify the authentication before calling
any rpc methods. move_torrent() is called on a fresh deluged client from
process_tv.py without any other previous connection attempt.

The RPC method for moving torrents (core.move_storage) also takes a list
of torrent IDs instead of a single ID, so pass a list (of one) to the
call instead.
Copy link
Contributor

@medariox medariox left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks! 👍

@medariox medariox merged commit 1c3adc3 into pymedusa:develop Jan 15, 2020
@jdrouhard jdrouhard deleted the fix_deluge_move_torrent branch January 15, 2020 14:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants