sitescripts/reports/bin/parseNewReports.py - Issue 8625042: Reports - user usefullness

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/reports/bin/parseNewReports.py

Issue 8625042: Reports - user usefullness (Closed)

Patch Set: Reports - report update UI Created Oct. 18, 2012, 1:35 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

Index: sitescripts/reports/bin/parseNewReports.py

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

--- a/sitescripts/reports/bin/parseNewReports.py

+++ b/sitescripts/reports/bin/parseNewReports.py

@@ -67,7 +67,7 @@

return

source = open(xmlFile, 'rb')

- reportData = {'status': '', 'warnings': {}, 'requests': [], 'filters': [], 'subscriptions': [], 'extensions': [], 'errors': [], 'time': time()}

+ reportData = {'status': '', 'utility': 0, 'warnings': {}, 'requests': [], 'filters': [], 'subscriptions': [], 'extensions': [], 'errors': [], 'time': time()}

Wladimir Palant 2012/10/18 15:41:24 How about replacing "utility" by "helpfulness"?

Andrey Novikov 2012/10/18 15:51:29 It's synonym and it's shorter, that's why I have c

Wladimir Palant 2012/10/19 10:26:06 I don't think that "utility" is a synonym for "hel

tagStack = []

parser = ParserCreate()

« no previous file with comments | « no previous file | sitescripts/reports/static/reports.js » ('j') | sitescripts/reports/static/reports.js » ('J')