fix update of images;

middleware/app_be/services/dropboxservice.py

@@ -70,7 +70,7 @@ class DropboxService(StorageServiceInterface):
         return file_bytes
 
     @staticmethod
-    def move_file(identifier_from: str, identifier_to: str, file: bytes) -> bool:
+    def move_file(identifier_from: str, identifier_to: str, file: bytes = None) -> bool:
         """Move/Rename a file on the dropbox storage.
 
         :param identifier_from: Location of the file to be moved

middleware/app_be/services/mongodbservice.py

@@ -74,20 +74,22 @@ class MongoDBService:
 
         old = {"identifier": identifier}  # Query for old version
 
-        metadata_orig = MongoDBService.getSingle(identifier)
-        metadata_new = metadata_orig
-        metadata_new['previous'] = identifier + '_' + str(metadata_orig['version'])
-        identifier_changed = identifier + '_' + str(metadata_orig['version'])  # Set identifier to include version
-        metadata_new['version'] = str(int(metadata_new['version']) + 1)  # Increment version by one
+        existing_meta_data_copy = MongoDBService.getSingle(identifier)
+        id_for_previous = identifier + '_' + str(meta['version'])
+        existing_meta_data_copy['previous'] = id_for_previous
+        identifier_changed = id_for_previous  # Set identifier to include version
 
-        metadata_new.update(meta)
+        del meta['filename']
+        existing_meta_data_copy.update(meta)
+
+        existing_meta_data_copy['version'] = meta['version'] + 1  # Increment version by one
 
         print("MongoDBService: identifier_changed: ", identifier_changed)
         col.update_one(old, {"$set": {"identifier": identifier_changed, "filename": identifier_changed + '.jpg'}})
 
         print("MongoDBService: Old object is ", col.find_one({"identifier": identifier_changed}))
 
-        MongoDBService.createSingle(metadata_new, identifier, decoded_image)
+        MongoDBService.createSingle(existing_meta_data_copy, identifier, decoded_image)
         print("MongoDBService: New object is ", col.find_one({"identifier": identifier}))
 
         return True