Changed behavior of ad blocker downloading the default subscription in case of missing an internet link. 5_1_x

Sat, 26 Mar 2011 12:52:15 +0100

author
Detlev Offenbach <detlev@die-offenbachs.de>
date
Sat, 26 Mar 2011 12:52:15 +0100
branch
5_1_x
changeset 957
ff469c60f5fe
parent 955
377efd9104a7
child 959
3b5ad7224945

Changed behavior of ad blocker downloading the default subscription in case of missing an internet link.

APIs/Python3/eric5.api file | annotate | diff | comparison | revisions
Documentation/Help/source.qch file | annotate | diff | comparison | revisions
Documentation/Source/eric5.Helpviewer.AdBlock.AdBlockSubscription.html file | annotate | diff | comparison | revisions
Helpviewer/AdBlock/AdBlockManager.py file | annotate | diff | comparison | revisions
Helpviewer/AdBlock/AdBlockSubscription.py file | annotate | diff | comparison | revisions
--- a/APIs/Python3/eric5.api	Mon Mar 21 20:03:40 2011 +0100
+++ b/APIs/Python3/eric5.api	Sat Mar 26 12:52:15 2011 +0100
@@ -1700,7 +1700,7 @@
 eric5.Helpviewer.AdBlock.AdBlockSubscription.AdBlockSubscription.title?4()
 eric5.Helpviewer.AdBlock.AdBlockSubscription.AdBlockSubscription.updateNow?4()
 eric5.Helpviewer.AdBlock.AdBlockSubscription.AdBlockSubscription.url?4()
-eric5.Helpviewer.AdBlock.AdBlockSubscription.AdBlockSubscription?1(url, parent = None)
+eric5.Helpviewer.AdBlock.AdBlockSubscription.AdBlockSubscription?1(url, parent = None, default = False)
 eric5.Helpviewer.Bookmarks.AddBookmarkDialog.AddBookmarkDialog.accept?4()
 eric5.Helpviewer.Bookmarks.AddBookmarkDialog.AddBookmarkDialog.addedNode?4()
 eric5.Helpviewer.Bookmarks.AddBookmarkDialog.AddBookmarkDialog.currentIndex?4()
Binary file Documentation/Help/source.qch has changed
--- a/Documentation/Source/eric5.Helpviewer.AdBlock.AdBlockSubscription.html	Mon Mar 21 20:03:40 2011 +0100
+++ b/Documentation/Source/eric5.Helpviewer.AdBlock.AdBlockSubscription.html	Sat Mar 26 12:52:15 2011 +0100
@@ -135,7 +135,7 @@
 </table>
 <a NAME="AdBlockSubscription.__init__" ID="AdBlockSubscription.__init__"></a>
 <h4>AdBlockSubscription (Constructor)</h4>
-<b>AdBlockSubscription</b>(<i>url, parent = None</i>)
+<b>AdBlockSubscription</b>(<i>url, parent = None, default = False</i>)
 <p>
         Constructor
 </p><dl>
@@ -145,6 +145,9 @@
 </dd><dt><i>parent</i></dt>
 <dd>
 reference to the parent object (QObject)
+</dd><dt><i>default</i></dt>
+<dd>
+flag indicating a default subscription (Boolean)
 </dd>
 </dl><a NAME="AdBlockSubscription.__loadRules" ID="AdBlockSubscription.__loadRules"></a>
 <h4>AdBlockSubscription.__loadRules</h4>
--- a/Helpviewer/AdBlock/AdBlockManager.py	Mon Mar 21 20:03:40 2011 +0100
+++ b/Helpviewer/AdBlock/AdBlockManager.py	Sat Mar 26 12:52:15 2011 +0100
@@ -219,18 +219,20 @@
         if self.__subscriptionsLoaded:
             return
         
+        defaultSubscriptionUrl = \
+            "abp:subscribe?location=http://adblockplus.mozdev.org/easylist/easylist.txt&title=EasyList"
         defaultSubscriptions = []
         defaultSubscriptions.append(
             bytes(self.__customSubscriptionUrl().toEncoded()).decode())
-        defaultSubscriptions.append(
-            "abp:subscribe?location=http://adblockplus.mozdev.org/easylist/easylist.txt&title=EasyList")
+        defaultSubscriptions.append(defaultSubscriptionUrl)
         
         subscriptions = Preferences.getHelp("AdBlockSubscriptions")
         if len(subscriptions) == 0:
             subscriptions = defaultSubscriptions
         for subscription in subscriptions:
             url = QUrl.fromEncoded(subscription.encode())
-            adBlockSubscription = AdBlockSubscription(url, self)
+            adBlockSubscription = AdBlockSubscription(url, self, 
+                subscription == defaultSubscriptionUrl)
             adBlockSubscription.rulesChanged.connect(self.rulesChanged)
             adBlockSubscription.changed.connect(self.rulesChanged)
             self.__subscriptions.append(adBlockSubscription)
--- a/Helpviewer/AdBlock/AdBlockSubscription.py	Mon Mar 21 20:03:40 2011 +0100
+++ b/Helpviewer/AdBlock/AdBlockSubscription.py	Sat Mar 26 12:52:15 2011 +0100
@@ -30,18 +30,20 @@
     changed = pyqtSignal()
     rulesChanged = pyqtSignal()
     
-    def __init__(self, url, parent = None):
+    def __init__(self, url, parent = None, default = False):
         """
         Constructor
         
         @param url AdBlock URL for the subscription (QUrl)
         @param parent reference to the parent object (QObject)
+        @param default flag indicating a default subscription (Boolean)
         """
         QObject.__init__(self, parent)
         
         self.__url = url.toEncoded()
         self.__enabled = False
         self.__downloading = None
+        self.__defaultSubscription = default
         
         self.__title = ""
         self.__location = QByteArray()
@@ -261,10 +263,15 @@
         self.__downloading = None
         
         if reply.error() != QNetworkReply.NoError:
-            E5MessageBox.warning(None,
-                self.trUtf8("Downloading subscription rules"),
-                self.trUtf8("""<p>Subscription rules could not be downloaded.</p>"""
-                            """<p>Error: {0}</p>""").format(reply.errorString()))
+            if not self.__defaultSubscription:
+                # don't show error if we try to load the default
+                E5MessageBox.warning(None,
+                    self.trUtf8("Downloading subscription rules"),
+                    self.trUtf8("""<p>Subscription rules could not be downloaded.</p>"""
+                                """<p>Error: {0}</p>""").format(reply.errorString()))
+            else:
+                # reset after first download attempt
+                self.__defaultSubscription = False
             return
         
         if redirect.isValid():

eric ide

mercurial