{"action":"created","comment":{"url":"https://api.github.com/repos/SiCKRAGETV/SickRage/pulls/comments/41178372","id":41178372,"diff_hunk":"@@ -228,19 +228,16 @@ def _doSearch(self, search_params, search_mode='eponly', epcount=0, age=0, epObj\n                                 continue\n \n \n-                            #Filter unseeded torrent\n-                            if mode != 'RSS' and (seeders < self.minseed or leechers < self.minleech):\n-                                logger.log(u\"Discarding torrent because it doesn't meet the minimum seeders or leechers: {0} (S:{1} L:{2})\".format(title, seeders, leechers), logger.DEBUG)\n+                            if not all([title, download_url, id, seeders, leechers]):","path":"sickbeard/providers/morethantv.py","position":16,"original_position":16,"commit_id":"11db141c06e4d4c034527ce5a216ed394490f5a2","original_commit_id":"11db141c06e4d4c034527ce5a216ed394490f5a2","user":{"login":"miigotu","id":390379,"avatar_url":"https://avatars.githubusercontent.com/u/390379?v=3","gravatar_id":"","url":"https://api.github.com/users/miigotu","html_url":"https://github.com/miigotu","followers_url":"https://api.github.com/users/miigotu/followers","following_url":"https://api.github.com/users/miigotu/following{/other_user}","gists_url":"https://api.github.com/users/miigotu/gists{/gist_id}","starred_url":"https://api.github.com/users/miigotu/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/miigotu/subscriptions","organizations_url":"https://api.github.com/users/miigotu/orgs","repos_url":"https://api.github.com/users/miigotu/repos","events_url":"https://api.github.com/users/miigotu/events{/privacy}","received_events_url":"https://api.github.com/users/miigotu/received_events","type":"User","site_admin":false},"body":"...","created_at":"2015-10-05T18:20:28Z","updated_at":"2015-10-05T18:20:28Z","html_url":"https://github.com/SiCKRAGETV/SickRage/pull/2754#discussion_r41178372","pull_request_url":"https://api.github.com/repos/SiCKRAGETV/SickRage/pulls/2754","_links":{"self":{"href":"https://api.github.com/repos/SiCKRAGETV/SickRage/pulls/comments/41178372"},"html":{"href":"https://github.com/SiCKRAGETV/SickRage/pull/2754#discussion_r41178372"},"pull_request":{"href":"https://api.github.com/repos/SiCKRAGETV/SickRage/pulls/2754"}}},"pull_request":{"url":"https://api.github.com/repos/SiCKRAGETV/SickRage/pulls/2754","id":46814191,"html_url":"https://github.com/SiCKRAGETV/SickRage/pull/2754","diff_url":"https://github.com/SiCKRAGETV/SickRage/pull/2754.diff","patch_url":"https://github.com/SiCKRAGETV/SickRage/pull/2754.patch","issue_url":"https://api.github.com/repos/SiCKRAGETV/SickRage/issues/2754","number":2754,"state":"open","locked":false,"title":"Fix discard message in providers","user":{"login":"fernandog","id":2620870,"avatar_url":"https://avatars.githubusercontent.com/u/2620870?v=3","gravatar_id":"","url":"https://api.github.com/users/fernandog","html_url":"https://github.com/fernandog","followers_url":"https://api.github.com/users/fernandog/followers","following_url":"https://api.github.com/users/fernandog/following{/other_user}","gists_url":"https://api.github.com/users/fernandog/gists{/gist_id}","starred_url":"https://api.github.com/users/fernandog/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/fernandog/subscriptions","organizations_url":"https://api.github.com/users/fernandog/orgs","repos_url":"https://api.github.com/users/fernandog/repos","events_url":"https://api.github.com/users/fernandog/events{/privacy}","received_events_url":"https://api.github.com/users/fernandog/received_events","type":"User","site_admin":false},"body":"@miigotu can you check this with caution?\r\n\r\nI converted all variables to download_url, leechers, seeders and item\r\nSome providers were using diff names\r\n\r\nThe code is 95% duplicate for all providers. Maybe in the future we can't create something in 'generic'\r\n\r\nI didn't get it why some providers return\r\n`item = title, download_url, id, seeders, leechers`\r\n\r\nothers\r\n`item = title, download_url, seeders, leechers` \r\n\r\nif we don't have ID in some shouldn't we return \r\n`item = title, download_url, id=None, seeders, leechers`","created_at":"2015-10-05T18:15:15Z","updated_at":"2015-10-05T18:20:28Z","closed_at":null,"merged_at":null,"merge_commit_sha":"23b094a89f6e0cab7a2327579fdc70d8d8568411","assignee":null,"milestone":null,"commits_url":"https://api.github.com/repos/SiCKRAGETV/SickRage/pulls/2754/commits","review_comments_url":"https://api.github.com/repos/SiCKRAGETV/SickRage/pulls/2754/comments","review_comment_url":"https://api.github.com/repos/SiCKRAGETV/SickRage/pulls/comments{/number}","comments_url":"https://api.github.com/repos/SiCKRAGETV/SickRage/issues/2754/comments","statuses_url":"https://api.github.com/repos/SiCKRAGETV/SickRage/statuses/11db141c06e4d4c034527ce5a216ed394490f5a2","head":{"label":"fernandog:discard_message","ref":"discard_message","sha":"11db141c06e4d4c034527ce5a216ed394490f5a2","user":{"login":"fernandog","id":2620870,"avatar_url":"https://avatars.githubusercontent.com/u/2620870?v=3","gravatar_id":"","url":"https://api.github.com/users/fernandog","html_url":"https://github.com/fernandog","followers_url":"https://api.github.com/users/fernandog/followers","following_url":"https://api.github.com/users/fernandog/following{/other_user}","gists_url":"https://api.github.com/users/fernandog/gists{/gist_id}","starred_url":"https://api.github.com/users/fernandog/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/fernandog/subscriptions","organizations_url":"https://api.github.com/users/fernandog/orgs","repos_url":"https://api.github.com/users/fernandog/repos","events_url":"https://api.github.com/users/fernandog/events{/privacy}","received_events_url":"https://api.github.com/users/fernandog/received_events","type":"User","site_admin":false},"repo":{"id":30988606,"name":"SickRage","full_name":"fernandog/SickRage","owner":{"login":"fernandog","id":2620870,"avatar_url":"https://avatars.githubusercontent.com/u/2620870?v=3","gravatar_id":"","url":"https://api.github.com/users/fernandog","html_url":"https://github.com/fernandog","followers_url":"https://api.github.com/users/fernandog/followers","following_url":"https://api.github.com/users/fernandog/following{/other_user}","gists_url":"https://api.github.com/users/fernandog/gists{/gist_id}","starred_url":"https://api.github.com/users/fernandog/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/fernandog/subscriptions","organizations_url":"https://api.github.com/users/fernandog/orgs","repos_url":"https://api.github.com/users/fernandog/repos","events_url":"https://api.github.com/users/fernandog/events{/privacy}","received_events_url":"https://api.github.com/users/fernandog/received_events","type":"User","site_admin":false},"private":false,"html_url":"https://github.com/fernandog/SickRage","description":"SickRage - Searches TheTVDB and TVRage for shows","fork":true,"url":"https://api.github.com/repos/fernandog/SickRage","forks_url":"https://api.github.com/repos/fernandog/SickRage/forks","keys_url":"https://api.github.com/repos/fernandog/SickRage/keys{/key_id}","collaborators_url":"https://api.github.com/repos/fernandog/SickRage/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/fernandog/SickRage/teams","hooks_url":"https://api.github.com/repos/fernandog/SickRage/hooks","issue_events_url":"https://api.github.com/repos/fernandog/SickRage/issues/events{/number}","events_url":"https://api.github.com/repos/fernandog/SickRage/events","assignees_url":"https://api.github.com/repos/fernandog/SickRage/assignees{/user}","branches_url":"https://api.github.com/repos/fernandog/SickRage/branches{/branch}","tags_url":"https://api.github.com/repos/fernandog/SickRage/tags","blobs_url":"https://api.github.com/repos/fernandog/SickRage/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/fernandog/SickRage/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/fernandog/SickRage/git/refs{/sha}","trees_url":"https://api.github.com/repos/fernandog/SickRage/git/trees{/sha}","statuses_url":"https://api.github.com/repos/fernandog/SickRage/statuses/{sha}","languages_url":"https://api.github.com/repos/fernandog/SickRage/languages","stargazers_url":"https://api.github.com/repos/fernandog/SickRage/stargazers","contributors_url":"https://api.github.com/repos/fernandog/SickRage/contributors","subscribers_url":"https://api.github.com/repos/fernandog/SickRage/subscribers","subscription_url":"https://api.github.com/repos/fernandog/SickRage/subscription","commits_url":"https://api.github.com/repos/fernandog/SickRage/commits{/sha}","git_commits_url":"https://api.github.com/repos/fernandog/SickRage/git/commits{/sha}","comments_url":"https://api.github.com/repos/fernandog/SickRage/comments{/number}","issue_comment_url":"https://api.github.com/repos/fernandog/SickRage/issues/comments{/number}","contents_url":"https://api.github.com/repos/fernandog/SickRage/contents/{+path}","compare_url":"https://api.github.com/repos/fernandog/SickRage/compare/{base}...{head}","merges_url":"https://api.github.com/repos/fernandog/SickRage/merges","archive_url":"https://api.github.com/repos/fernandog/SickRage/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/fernandog/SickRage/downloads","issues_url":"https://api.github.com/repos/fernandog/SickRage/issues{/number}","pulls_url":"https://api.github.com/repos/fernandog/SickRage/pulls{/number}","milestones_url":"https://api.github.com/repos/fernandog/SickRage/milestones{/number}","notifications_url":"https://api.github.com/repos/fernandog/SickRage/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/fernandog/SickRage/labels{/name}","releases_url":"https://api.github.com/repos/fernandog/SickRage/releases{/id}","created_at":"2015-02-18T21:43:24Z","updated_at":"2015-05-07T11:00:39Z","pushed_at":"2015-10-05T18:11:31Z","git_url":"git://github.com/fernandog/SickRage.git","ssh_url":"git@github.com:fernandog/SickRage.git","clone_url":"https://github.com/fernandog/SickRage.git","svn_url":"https://github.com/fernandog/SickRage","homepage":"http://www.sickrage.tv/","size":43919,"stargazers_count":0,"watchers_count":0,"language":"Python","has_issues":false,"has_downloads":true,"has_wiki":true,"has_pages":false,"forks_count":0,"mirror_url":null,"open_issues_count":0,"forks":0,"open_issues":0,"watchers":0,"default_branch":"develop"}},"base":{"label":"SiCKRAGETV:develop","ref":"develop","sha":"c62fd2196586658d21b8953c5463bf86beeb5f57","user":{"login":"SiCKRAGETV","id":9251163,"avatar_url":"https://avatars.githubusercontent.com/u/9251163?v=3","gravatar_id":"","url":"https://api.github.com/users/SiCKRAGETV","html_url":"https://github.com/SiCKRAGETV","followers_url":"https://api.github.com/users/SiCKRAGETV/followers","following_url":"https://api.github.com/users/SiCKRAGETV/following{/other_user}","gists_url":"https://api.github.com/users/SiCKRAGETV/gists{/gist_id}","starred_url":"https://api.github.com/users/SiCKRAGETV/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/SiCKRAGETV/subscriptions","organizations_url":"https://api.github.com/users/SiCKRAGETV/orgs","repos_url":"https://api.github.com/users/SiCKRAGETV/repos","events_url":"https://api.github.com/users/SiCKRAGETV/events{/privacy}","received_events_url":"https://api.github.com/users/SiCKRAGETV/received_events","type":"Organization","site_admin":false},"repo":{"id":17582613,"name":"SickRage","full_name":"SiCKRAGETV/SickRage","owner":{"login":"SiCKRAGETV","id":9251163,"avatar_url":"https://avatars.githubusercontent.com/u/9251163?v=3","gravatar_id":"","url":"https://api.github.com/users/SiCKRAGETV","html_url":"https://github.com/SiCKRAGETV","followers_url":"https://api.github.com/users/SiCKRAGETV/followers","following_url":"https://api.github.com/users/SiCKRAGETV/following{/other_user}","gists_url":"https://api.github.com/users/SiCKRAGETV/gists{/gist_id}","starred_url":"https://api.github.com/users/SiCKRAGETV/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/SiCKRAGETV/subscriptions","organizations_url":"https://api.github.com/users/SiCKRAGETV/orgs","repos_url":"https://api.github.com/users/SiCKRAGETV/repos","events_url":"https://api.github.com/users/SiCKRAGETV/events{/privacy}","received_events_url":"https://api.github.com/users/SiCKRAGETV/received_events","type":"Organization","site_admin":false},"private":false,"html_url":"https://github.com/SiCKRAGETV/SickRage","description":"SickRage - Searches TheTVDB for shows","fork":false,"url":"https://api.github.com/repos/SiCKRAGETV/SickRage","forks_url":"https://api.github.com/repos/SiCKRAGETV/SickRage/forks","keys_url":"https://api.github.com/repos/SiCKRAGETV/SickRage/keys{/key_id}","collaborators_url":"https://api.github.com/repos/SiCKRAGETV/SickRage/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/SiCKRAGETV/SickRage/teams","hooks_url":"https://api.github.com/repos/SiCKRAGETV/SickRage/hooks","issue_events_url":"https://api.github.com/repos/SiCKRAGETV/SickRage/issues/events{/number}","events_url":"https://api.github.com/repos/SiCKRAGETV/SickRage/events","assignees_url":"https://api.github.com/repos/SiCKRAGETV/SickRage/assignees{/user}","branches_url":"https://api.github.com/repos/SiCKRAGETV/SickRage/branches{/branch}","tags_url":"https://api.github.com/repos/SiCKRAGETV/SickRage/tags","blobs_url":"https://api.github.com/repos/SiCKRAGETV/SickRage/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/SiCKRAGETV/SickRage/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/SiCKRAGETV/SickRage/git/refs{/sha}","trees_url":"https://api.github.com/repos/SiCKRAGETV/SickRage/git/trees{/sha}","statuses_url":"https://api.github.com/repos/SiCKRAGETV/SickRage/statuses/{sha}","languages_url":"https://api.github.com/repos/SiCKRAGETV/SickRage/languages","stargazers_url":"https://api.github.com/repos/SiCKRAGETV/SickRage/stargazers","contributors_url":"https://api.github.com/repos/SiCKRAGETV/SickRage/contributors","subscribers_url":"https://api.github.com/repos/SiCKRAGETV/SickRage/subscribers","subscription_url":"https://api.github.com/repos/SiCKRAGETV/SickRage/subscription","commits_url":"https://api.github.com/repos/SiCKRAGETV/SickRage/commits{/sha}","git_commits_url":"https://api.github.com/repos/SiCKRAGETV/SickRage/git/commits{/sha}","comments_url":"https://api.github.com/repos/SiCKRAGETV/SickRage/comments{/number}","issue_comment_url":"https://api.github.com/repos/SiCKRAGETV/SickRage/issues/comments{/number}","contents_url":"https://api.github.com/repos/SiCKRAGETV/SickRage/contents/{+path}","compare_url":"https://api.github.com/repos/SiCKRAGETV/SickRage/compare/{base}...{head}","merges_url":"https://api.github.com/repos/SiCKRAGETV/SickRage/merges","archive_url":"https://api.github.com/repos/SiCKRAGETV/SickRage/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/SiCKRAGETV/SickRage/downloads","issues_url":"https://api.github.com/repos/SiCKRAGETV/SickRage/issues{/number}","pulls_url":"https://api.github.com/repos/SiCKRAGETV/SickRage/pulls{/number}","milestones_url":"https://api.github.com/repos/SiCKRAGETV/SickRage/milestones{/number}","notifications_url":"https://api.github.com/repos/SiCKRAGETV/SickRage/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/SiCKRAGETV/SickRage/labels{/name}","releases_url":"https://api.github.com/repos/SiCKRAGETV/SickRage/releases{/id}","created_at":"2014-03-10T05:20:31Z","updated_at":"2015-10-05T17:19:59Z","pushed_at":"2015-10-05T18:15:15Z","git_url":"git://github.com/SiCKRAGETV/SickRage.git","ssh_url":"git@github.com:SiCKRAGETV/SickRage.git","clone_url":"https://github.com/SiCKRAGETV/SickRage.git","svn_url":"https://github.com/SiCKRAGETV/SickRage","homepage":"http://www.sickrage.tv/","size":101863,"stargazers_count":1218,"watchers_count":1218,"language":"Python","has_issues":false,"has_downloads":true,"has_wiki":true,"has_pages":false,"forks_count":710,"mirror_url":null,"open_issues_count":11,"forks":710,"open_issues":11,"watchers":1218,"default_branch":"master"}},"_links":{"self":{"href":"https://api.github.com/repos/SiCKRAGETV/SickRage/pulls/2754"},"html":{"href":"https://github.com/SiCKRAGETV/SickRage/pull/2754"},"issue":{"href":"https://api.github.com/repos/SiCKRAGETV/SickRage/issues/2754"},"comments":{"href":"https://api.github.com/repos/SiCKRAGETV/SickRage/issues/2754/comments"},"review_comments":{"href":"https://api.github.com/repos/SiCKRAGETV/SickRage/pulls/2754/comments"},"review_comment":{"href":"https://api.github.com/repos/SiCKRAGETV/SickRage/pulls/comments{/number}"},"commits":{"href":"https://api.github.com/repos/SiCKRAGETV/SickRage/pulls/2754/commits"},"statuses":{"href":"https://api.github.com/repos/SiCKRAGETV/SickRage/statuses/11db141c06e4d4c034527ce5a216ed394490f5a2"}}}}