-
Notifications
You must be signed in to change notification settings - Fork 471
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] More web changes Newpct1 #2054
Conversation
Now Newpct1 and Newpct share torrent urlrewrite.
flexget/plugins/sites/newpct.py
Outdated
"'\s*:\s*'(\d+)'") | ||
torrent_ids = soup.findAll(text=torrent_id_prog) | ||
if len(torrent_ids): |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
you can do just if torrent_ids:
flexget/plugins/sites/newpct.py
Outdated
@@ -22,8 +22,8 @@ | |||
requests.add_domain_limiter(TimedLimiter('newpct1.com', '2 seconds')) | |||
requests.add_domain_limiter(TimedLimiter('newpct.com', '2 seconds')) | |||
|
|||
NEWPCT_TORRENT_FORMAT = 'http://www.newpct.com/torrents/{:0>6}.torrent' | |||
NEWPCT1_TORRENT_FORMAT = 'http://www.newpct1.com/download/{}.torrent' | |||
NEWPCT_TORRENT_FORMAT = 'http://www.newpct.com/torrents/{:0>6}.torrent' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
looks like there's an extra space there
torrent_ids = soup.findAll(text=torrent_id_prog) | ||
if len(torrent_ids): | ||
log.debug('torrent ID not found, searching openTorrent script') |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This doesn't really make sense if torrent_ids
is empty/none or maybe I'm misunderstanding what that log is referencing
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There are two attempts to find torrent id, first regexp with 'parametros', then regexp with opentorrent. The log is a reference to the first regexp failure. Should I remove the log?
Sorry for noise closed topics. I didn't want to open new topics because I didn't want to overload the error issues. |
Now Newpct1 and Newpct share torrent urlrewrite.
Motivation for changes:
Changes in newpct1
Detailed changes: