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

Moved seeders sorting out of providers code and into the sickbeard/se… #658

Merged
merged 2 commits into from
Jun 19, 2016

Conversation

p0psicles
Copy link
Contributor

  • 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 contribution guide

Curious what the results are, as I don't have my torrent providers.
@duramato can you test it?

@p0psicles p0psicles added Enhancement In progress Needs testing Requires testing to make sure it's working as intended labels Jun 3, 2016
@medariox medariox mentioned this pull request Jun 4, 2016
…arch.py searchProviders() code.

* Removed the lambda sort from all providers
* Corrected bug introducted in TVCache
* Removed bogus condition from GenericProvider. That condition can never be true.
@p0psicles
Copy link
Contributor Author

What are your test results? I expect some issues. So lets try to get these known, so we can fix them.

@p0psicles p0psicles added this to the 0.1.2 milestone Jun 11, 2016
* Standardize first 10 providers

* Small anizb update

* Small bluetigers update

* Next 10 providers

* Added unicode_literals to GenericProvider (#677)

* Added unicode_literals to GenericProvider

* Also adapted all providers, to make use of the future import unicode_literals
* Removed the decode()/encode()
* Cleaned up some double to single quotes
* Added proper exceptions for the provider results items
* Some logging cleanup using format()

* Now Really remove the .decodes()

* Also removed the encodes.

* Fixed after a search/replace

* Fixed docstrings

* Next 11 providers

* Next 11 providers, removed sceneelite

* Last 9 providers

* Remove sceneelite from init

* Renamed all search_params to search_strings

* Fix for GFTracker

* Fix TNTVillage

* Fix HDTorrents

* Fix Extratorrent

* Fix HDSpace

* Use string in SQL with unicode_literals in GenericProvider

* Fix BITHDTV

* Fix TVChaosUK

* Added flag to newznab, for torznab providers. If it's torznab then results are sorted by seeders in search.py.

* Improve BitSnoop

* Improve Anizb

* Improve Bluetigers

* Cleanup BTdigg

* Improve Hounddawgs

* Improve FreshOn

* More improvements and cleanups

* Fix ThePirateBay

* Fix for omgwtfnzb, needed a default value, cause getattr doesn't do that by default.

* Add size to freshon, cleanup, fix for tvchaosuk

* Improve size parsing code Freshon

* Fixes for ExtraTorrent and HDTorrents

* Fixed bithdtv

* For when it's not getting back the 750px tables.

* Fix tokyotoshokan provider errors

* Fixed properSearch.

* listPropers does an sql, but accessing the row, requires it to use b''

* Added newznab search by search_query fallback, when search by tvdbid does not give back results.

* Fix HDTorrents, use urljoin, partial rewrite

* Fix rare Zooqle error

* Improve HDTorrents, bring back ugly hack

* Improve TNTVillage, fix daily search, much more

* Fix BIT-HDTV

* More standardization

* More standardization

* Bring back eng releases only option

* small fixup

* Small tnt change

* Update daily search url

* Remove freeleech option for MTV

* Remove TypeError from connection time out

* FIx repeated keyword in dict

* More standardization

* Standardize method names and order

* FIx missed URL join

* Standardize string formatting

* Last small changes

* Change TPB url, update cache to 20 min

* More providers (#698)

* Remove redundant character escapes

* Use augmented assignment

* Fix indentation

* Use six.iteritems for py3 compatibility

* Store hash for torrentproject
@p0psicles
Copy link
Contributor Author

p0psicles commented Jun 19, 2016

Approved

Approved with PullApprove

@p0psicles p0psicles merged commit d78a963 into reduce-release-parsing Jun 19, 2016
@p0psicles p0psicles deleted the move-seeder-sorting branch June 19, 2016 11:25
@fernandog fernandog removed the Needs testing Requires testing to make sure it's working as intended label Feb 9, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants