Unified Diff: sitescripts/reports/web/updateReport.py
Issue 29345348:
Noissue - Got rid of redundant paranthesis around if condition (Closed)
Patch Set:
Created May 30, 2016, 11:05 a.m.
Use n/p to move between diff chunks;
N/P to move between comments.
« no previous file with comments
|
« sitescripts/management/bin/installChanges.py ('k')
|
tox.ini » ('j')
|
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 |
| @@ -59,7 +59,7 @@ |
| oldusefulness = reportData.get('usefulness', '0') |
| reportData['usefulness'] = params.get('usefulness', '0') |
| - if ('email' in reportData): |
| + if 'email' in reportData: |
| updateUserUsefulness(getUserId(reportData['email']), reportData['usefulness'], oldusefulness) |
| saveReport(guid, reportData) |
« no previous file with comments
|
« sitescripts/management/bin/installChanges.py ('k')
|
tox.ini » ('j')
|
no next file with comments »
