import_csv: my try so error is actually caught

digitalmethodsinitiative · Dec 20, 2023 · f3e01ea · f3e01ea
1 parent 0049de6
commit f3e01ea
Showing 1 changed file with 10 additions and 10 deletions.
diff --git a/datasources/upload/import_csv.py b/datasources/upload/import_csv.py
@@ -120,17 +120,17 @@ def process(self):
 
                     # pseudonymise or anonymise as needed
                     filtering = self.parameters.get("pseudonymise")
-                    if filtering:
-                        for field, value in item.items():
-                            if field is None:
-                                raise ValueError("Field is None") # This would normally be caught when writerow is called
-                            if field.startswith("author"):
-                                if filtering == "anonymise":
-                                    item[field] = "REDACTED"
-                                elif filtering == "pseudonymise":
-                                    item[field] = hash_cache.update_cache(value)
-
                     try:
+                        if filtering:
+                            for field, value in item.items():
+                                if field is None:
+                                    raise ValueError("Field is None") # This would normally be caught when writerow is called
+                                if field.startswith("author"):
+                                    if filtering == "anonymise":
+                                        item[field] = "REDACTED"
+                                    elif filtering == "pseudonymise":
+                                        item[field] = hash_cache.update_cache(value)
+
                         writer.writerow(item)
                     except ValueError as e:
                         self.dataset.log(f"Error ({e}) writing item {i}: {item}")