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

Unified Diff: sitescripts/urlfixer/bin/topDomains.py

Issue 29344600: Noissue - Got rid of E129, E704, E712, E731 and F821 errors (Closed)
Patch Set: Created May 26, 2016, 12:49 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 | « sitescripts/subscriptions/bin/updateStatusPage.py ('k') | tox.ini » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: sitescripts/urlfixer/bin/topDomains.py
===================================================================
--- a/sitescripts/urlfixer/bin/topDomains.py
+++ b/sitescripts/urlfixer/bin/topDomains.py
@@ -42,8 +42,9 @@
mandatory = []
for result in cursor:
domain = result["domain"]
- if ("." not in domain or not re.search(r"[a-zA-Z]", domain) or
- re.search(r"['\"_,<>:;!$%&/()*+#~]|^\.|\.$|\.\.", domain)):
+ if "." not in domain or not re.search(r"[a-zA-Z]", domain):
+ continue
+ if re.search(r"['\"_,<>:;!$%&/()*+#~]|^\.|\.$|\.\.", domain):
continue
typed = _get_weighted_count(db, result["id"], STATUS_TYPED)
« no previous file with comments | « sitescripts/subscriptions/bin/updateStatusPage.py ('k') | tox.ini » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld