download: optimization of command parameters

bgmi/controllers.py

@@ -194,15 +194,7 @@ def cal(ret):
 
 
 def download_manager(ret):
-    print_info('Download status value: Not Downloaded: 0 / Downloading: 1 / Downloaded: 2\n', indicator=False)
-
-    if ret.list:
-        status = ret.status
-        status = int(status) if status is not None else None
-        delegate = get_download_class(instance=False)
-        delegate.download_status(status=status)
-
-    elif ret.id:
+    if ret.id:
         download_id = ret.id
         status = ret.status
         if download_id is None or status is None:
@@ -216,6 +208,11 @@ def download_manager(ret):
         download_obj.status = status
         download_obj.save()
         print_success('Download status has been marked as {0}'.format(DOWNLOAD_CHOICE_LIST_DICT.get(int(status))))
+    else:
+        status = ret.status
+        status = int(status) if status is not None else None
+        delegate = get_download_class(instance=False)
+        delegate.download_status(status=status)
 
 
 def mark(ret):

bgmi/main.py

@@ -111,7 +111,9 @@ def main():
     sub_parser_download = sub_parser.add_parser(ACTION_DOWNLOAD, help='Download manager.')
     sub_parser_download.add_argument('--list', help='List download queue.', action='store_true')
     sub_parser_download.add_argument('--mark', help='Mark download status with a specific id.', dest='id', type=int)
-    sub_parser_download.add_argument('--status', type=int, help='Download items status (0, 1, 2).', choices=[0, 1, 2])
+    sub_parser_download.add_argument('--status', type=int, help='Download items status (0: not download, 1: '
+                                                                'downloading, 2: alrealy downloaded).',
+                                     choices=[0, 1, 2])
 
     sub_parser_fetch = sub_parser.add_parser(ACTION_FETCH, help='Fetch bangumi.')
     sub_parser_fetch.add_argument('name', help='Bangumi name', type=unicode_)