Unified Diff: sitescripts/reports/web/updateReport.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.
« sitescripts/reports/static/reports.js ('K')
|
« sitescripts/reports/template/report.html ('k')
|
no next file »
|
no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')
Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')
| Index: sitescripts/reports/web/updateReport.py |
| =================================================================== |
| --- a/sitescripts/reports/web/updateReport.py |
| +++ b/sitescripts/reports/web/updateReport.py |
| @@ -44,6 +44,9 @@ |
| if len(reportData['status']) > 1024: |
| reportData['status'] = reportData['status'][:1024] |
| + if (reportData.get('email', None)): |
| + reportData['utility'] = params.get('utility', 0) |
|
Wladimir Palant
2012/10/18 15:41:24
Isn't this value a string? The default value shoul
Andrey Novikov
2012/10/18 15:51:29
Done.
|
| + |
| saveReport(guid, reportData) |
| if params.get('notify', '') and 'email' in reportData: |
« sitescripts/reports/static/reports.js ('K')
|
« sitescripts/reports/template/report.html ('k')
|
no next file »
|
no next file with comments »
