Unified Diff: filters/get_subscriptions.py
Issue 29336582:
Issue 2813 - Catch missing sitescripts module and return empty list on adblockplus.org (Closed)
Patch Set: Added line break to warning message
Created Feb. 23, 2016, 2:50 p.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..5a41ae460a86ce4e4eea6c0fd43f63d9f14c4e6f 100644 |
| --- a/filters/get_subscriptions.py |
| +++ b/filters/get_subscriptions.py |
| @@ -17,10 +17,16 @@ import codecs |
| import os |
| import tarfile |
| import urllib |
| +import logging |
| from ConfigParser import SafeConfigParser |
| def get_subscriptions(_): |
| - from sitescripts.subscriptions import subscriptionParser |
| + try: |
| + from sitescripts.subscriptions import subscriptionParser |
| + except ImportError: |
| + logging.warning("Unable to import sitescripts, proceeding with empty " |
| + "subscriptions list.") |
|
kzar
2016/02/23 15:01:38
Nit: The indentation here looks wrong, the string
juliandoucette
2016/02/23 15:08:12
My mistake.
|
| + return [] |
| result = {} |
| utf8_reader = codecs.getreader('utf8') |
« no previous file with comments
|
« no previous file
|
no next file »
|
no next file with comments »
