darkhelm/notifications-admin
1
0
Fork 0
mirror of https://github.com/GSA/notifications-admin.git synced 2026-09-05 21:18:25 -04:00
Code Issues Packages Projects Releases Wiki Activity

Remove redundant if statement

Browse Source 
We check for the `None` case in the parent `if` statement.
This commit is contained in:
Chris Hill-Scott
2016-08-01 10:57:05 +01:00
parent 1ea1b1fd45
commit 47e7d418eb
1 changed files with 1 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
app/main/views/jobs.py
View File

@@ -50,7 +50,7 @@ def _set_status_filters(filter_args):
all_failure_statuses = ['failed', 'temporary-failure', 'permanent-failure', 'technical-failure'] all_failure_statuses = ['failed', 'temporary-failure', 'permanent-failure', 'technical-failure']
all_statuses = ['sending', 'delivered'] + all_failure_statuses all_statuses = ['sending', 'delivered'] + all_failure_statuses
if filter_args.get('status'): if filter_args.get('status'):
if 'processed' in filter_args.get('status') or not filter_args.get('status'): if 'processed' in filter_args.get('status'):
filter_args['status'] = all_statuses filter_args['status'] = all_statuses
elif 'failed' in filter_args.get('status'): elif 'failed' in filter_args.get('status'):
filter_args['status'].extend(all_failure_statuses[1:]) filter_args['status'].extend(all_failure_statuses[1:])