Ursprungspfad beim Kopieren der RD_Teilenummern.csv korrigiert. Bisher wurde nur auf das Directory als Ursprungspfad verwiesen, damit shutil.copyfile funktioniert muss aber der Dateipfad mit im Pfad angegeben sein, nicht nur das Directory, in dem die Datei liegt. Zudem wird nun Argparse-Help automatisch angezeigt, wenn Flags fehlen und Argparse-Help wurde 'verschönert'

lib/update_database.py

@@ -12,16 +12,16 @@ Dieses Programm:
 
 def get_rd_dbase(out_dir):
 
-    rd_csv_source_dir = os.environ.get('RD_DATABASE_SOURCE')
-    rd_csv_filename = os.environ.get('RD_DATABASE_NAME')
-    csv_source_filepath = os.path.join(rd_csv_source_dir, rd_csv_filename)
+    rd_csv_source_dir = os.environ.get("RD_DATABASE_SOURCE")
+    rd_csv_filename = os.environ.get("RD_DATABASE_NAME")
+    csv_source_path = os.path.join(rd_csv_source_dir, rd_csv_filename)
 
-    if os.path.isfile(csv_source_filepath):
-        csv_out_filepath = os.path.join(out_dir, rd_csv_filename)
+    if os.path.isfile(csv_source_path):
+        csv_destiny_path = os.path.join(out_dir, rd_csv_filename)
 
         if os.path.isdir(out_dir):
             print("Copying CSV-file of RuleDesigner database...")
-            shutil.copyfile(rd_csv_source_dir, csv_out_filepath)
+            shutil.copyfile(csv_source_path, csv_destiny_path)
             print("done")
         else:
             print(f"Can't copy RuleDesigner-database, because output directory ${out_dir} doesn't exist. Check path of %CREMIG_DATA% in cremig_setenv.bat")
@@ -51,7 +51,7 @@ def get_sivas_dbase(sivas_id, out_dir):
 
 if __name__ == '__main__':
     parser = argparse.ArgumentParser(description='fetches data from sivas or ruledesigner fusion', prog='update_database')
-    parser.add_argument('-s', '--sivas', action='store', help='fetch data of this id from sivas fo local database folder')
+    parser.add_argument('-s', '--sivas', action='store', help='fetch data of this id from sivas fo local database folder', metavar='id')
     parser.add_argument('-r', '--rd', action='store_true', help='fetch list of existing parts from network drive')
 
     args = parser.parse_args()
@@ -65,4 +65,4 @@ if __name__ == '__main__':
         get_sivas_dbase(args.sivas, out_dir)
 
     else:
-        print("ERROR - Missing flags!")
+        parser.print_help()