Unified Diff: filters/get_subscriptions.py
Issue 29336582:
Issue 2813 - Catch missing sitescripts module and return empty list on adblockplus.org (Closed)
Patch Set:
Created Feb. 18, 2016, 11:03 a.m.
Use n/p to move between diff chunks;
N/P to move between comments.
« no previous file with comments
|
« no previous file
|
no next file »
|
no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')
Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')
| Index: filters/get_subscriptions.py |
| diff --git a/filters/get_subscriptions.py b/filters/get_subscriptions.py |
| index e87459808f6956f4dadbb6eec51d3779cc9f4345..9a12bda08533f770f2357877e638df54d6f2a06d 100644 |
| --- a/filters/get_subscriptions.py |
| +++ b/filters/get_subscriptions.py |
| @@ -20,8 +20,10 @@ import urllib |
| from ConfigParser import SafeConfigParser |
| def get_subscriptions(_): |
| - from sitescripts.subscriptions import subscriptionParser |
| - |
| + try: |
| + from sitescripts.subscriptions import subscriptionParser |
| + except ImportError as error: |
|
kzar
2016/02/18 15:24:25
Nit: You're not using error, so no point giving it
juliandoucette
2016/02/19 17:00:10
Done.
|
| + return [] |
|
kzar
2016/02/18 15:24:25
We should also log a warning message here, so that
juliandoucette
2016/02/19 17:00:10
Done.
|
| result = {} |
| utf8_reader = codecs.getreader('utf8') |
| source = urllib.urlopen("https://hg.adblockplus.org/subscriptionlist/archive/default.tar.gz") |
« no previous file with comments
|
« no previous file
|
no next file »
|
no next file with comments »
