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

Read correct process_methods from config on startup #11172

Merged
merged 1 commit into from
Mar 11, 2023

Conversation

cyatta
Copy link
Contributor

@cyatta cyatta commented Mar 11, 2023

  • PR is based on the DEVELOP branch
  • Don't send big changes all at once. Split up big PRs into multiple smaller PRs that are easier to manage and review
  • Read the contribution guide

On startup, init will overwrite previous values set for process_method_torrent and process_method_nzb because it's actually reading process_method for each setting.

This change fixes that by setting PROCESS_METHOD_NZB and PROCESS_METHOD_TORRENT to read from the correct string within config.

For quick reference, the config names for the specific process methods:

Medusa/medusa/__main__.py

Lines 1714 to 1715 in 49ad1f9

new_config['General']['process_method_torrent'] = app.PROCESS_METHOD_TORRENT
new_config['General']['process_method_nzb'] = app.PROCESS_METHOD_NZB

On startup, init will overwrite previous values set for process_method_torrent and process_method_nzb because it's actually reading process_method for each setting.
@medariox medariox merged commit 99e0afa into pymedusa:develop Mar 11, 2023
@medariox
Copy link
Contributor

Good catch, thank you!

@medariox medariox added this to the 1.0.13 milestone Mar 11, 2023
@cyatta cyatta deleted the correct-process-method branch March 12, 2023 06:48
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