sitescripts/subscriptions/bin/updateStatusPage.py - Issue 29670611: Issue 5844 - Remove redundant parentheses in sitescripts and abpsembly

Keyboard Shortcuts

	File
u :	up to issue
m :	publish + mail comments
M :	edit review message
j / k :	jump to file after / before current file
J / K :	jump to next file with a comment after / before current file
	Side-by-side diff
i :	toggle intra-line diffs
e :	expand all comments
c :	collapse all comments
s :	toggle showing all comments
n / p :	next / previous diff chunk or comment
N / P :	next / previous comment
<Up> / <Down> :	next / previous line
<Enter> :	respond to / edit current comment
d :	mark current comment as done

	Issue
u :	up to list of issues
m :	publish + mail comments
j / k :	jump to patch after / before current patch
o / <Enter> :	open current patch in side-by-side view
i :	open current patch in unified diff view

	Issue List
j / k :	jump to issue after / before current issue
o / <Enter> :	open current issue
# :	close issue

	Comment/message editing
<Ctrl> + s or <Ctrl> + Enter :	save comment
<Esc> :	cancel edit

Unified Diff: sitescripts/subscriptions/bin/updateStatusPage.py

Issue 29670611: Issue 5844 - Remove redundant parentheses in sitescripts and abpsembly (Closed) Base URL: https://hg.adblockplus.org/sitescripts/

Patch Set: Addressed the comments in /sitescripts Created Jan. 25, 2018, 4:12 p.m.

Use n/p to move between diff chunks; N/P to move between comments.

Jump to:

Index: sitescripts/subscriptions/bin/updateStatusPage.py

===================================================================

--- a/sitescripts/subscriptions/bin/updateStatusPage.py

+++ b/sitescripts/subscriptions/bin/updateStatusPage.py

@@ -55,15 +55,15 @@

url = getattr(subscription, key)

if url != None:

urls[url] = True

- for (title, url, complete) in subscription.variants:

+ for title, url, complete in subscription.variants:

urls[url] = True

pool = eventlet.GreenPool()

- for (url, result) in pool.imap(checkURL, urls.iterkeys()):

+ for url, result in pool.imap(checkURL, urls.iterkeys()):

urls[url] = result

if result is False:

sites[urlparse(url).netloc] = True

- for (site, result) in pool.imap(checkSite, sites.iterkeys()):

+ for site, result in pool.imap(checkSite, sites.iterkeys()):

sites[site] = result

result = []

@@ -80,7 +80,7 @@

'result': urls[url],

'siteResult': site in sites and sites[site],

})

- for (title, url, complete) in subscription.variants:

+ for title, url, complete in subscription.variants:

site = urlparse(url).netloc

s['links'].append({

'url': url,