Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code

Unified Diff: src/org/adblockplus/android/ABPEngine.java

Issue 5271235290202112: Issue #577 - Subscription change/update fix (Closed)
Patch Set: Created May 28, 2014, 3:40 p.m.
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: src/org/adblockplus/android/ABPEngine.java
diff --git a/src/org/adblockplus/android/ABPEngine.java b/src/org/adblockplus/android/ABPEngine.java
index 3be672be429e8642a79f86a13871577e1c2b9fd5..ab56b87a439eac785c444605d22e7554e2685ec7 100644
--- a/src/org/adblockplus/android/ABPEngine.java
+++ b/src/org/adblockplus/android/ABPEngine.java
@@ -200,15 +200,12 @@ public final class ABPEngine
public void setSubscription(final String url)
{
- Subscription sub = null;
for (final Subscription s : this.filterEngine.getListedSubscriptions())
{
- if (url.equals(s.getProperty("url").toString()))
- {
- sub = s;
- }
s.removeFromList();
}
+
+ final Subscription sub = this.filterEngine.getSubscription(url);
if (sub != null)
{
sub.addToList();
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld