fix: Apply code review #94

Author: afbora

commands/clean/content.php

@@ -23,10 +23,11 @@
 		// get the keys and normalize to lowercase
 		$originalContentKeys = array_keys($contentFields);
 		$contentFieldKeys = array_map('strtolower', $originalContentKeys);
+		$contentFieldKeys = array_map('mb_strtolower', $originalContentKeys);
 
 		// get all field keys from blueprint and normalize to lowercase
 		$blueprintFields = array_keys($item->blueprint()->fields());
-		$blueprintFieldKeys = array_map('strtolower', $blueprintFields);
+		$blueprintFieldKeys = array_map('mb_strtolower', $blueprintFields);
 
 		// get all field keys that are in $contentFieldKeys but not in $blueprintFieldKeys
 		$fieldsToBeDeleted = array_diff($contentFieldKeys, $blueprintFieldKeys);
@@ -36,16 +37,15 @@
 			// create a mapping: lowercase => original field name
 			$lowercaseToOriginal = array_combine($contentFieldKeys, $originalContentKeys);
 
-			// flip keys and values and set new values to null
-			$data = array_map(
-				fn ($field) => null,
-				array_flip(array_intersect_key($lowercaseToOriginal, array_flip($fieldsToBeDeleted)))
-			);
+			// build data array with original field names as keys and null as values
+			$data = [];
+			foreach ($fieldsToBeDeleted as $lowercaseField) {
+				$originalField = $lowercaseToOriginal[$lowercaseField];
+				$data[$originalField] = null;
+			}
 
 			// try to update the page with the data
-			if (count($data) > 0) {
-				$item->update($data, $lang);
-			}
+			$item->update($data, $lang);
 		}
 	}
 };