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 ruTorrent utility functions #14

Merged
merged 4 commits into from
Jan 13, 2022
Merged

Fix ruTorrent utility functions #14

merged 4 commits into from
Jan 13, 2022

Conversation

stickz
Copy link
Contributor

@stickz stickz commented Jan 10, 2022

The utility functions were recently changed in ruTorrent. Novik/ruTorrent#2247 They are no longer in the global scope. It's required to access these static methods from their respective classes now.

This PR is currently pending review to fix auto loading the utility classes from namespaces. Novik/ruTorrent#2253

I was not able to setup a testing environment conducive for this pull request. There might be more changes required.

@nelu
Copy link
Owner

nelu commented Jan 10, 2022

Hi,
The changes you have been working on sound great. i will give it a try on my dev env to see how things work with the updated code.

im using rutorrent-dock project https://github.com/nelu/rutorrent-dock for my local dev emv, with rutorrent setup in ./src/rutorrent

src/Archive.php Outdated Show resolved Hide resolved
Copy link
Owner

@nelu nelu left a comment

Choose a reason for hiding this comment

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

Seems it is running fine with the new changes on the dev env. Great job!

@nelu nelu merged commit 32901b0 into nelu:master Jan 13, 2022
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