33579 - Slightly more helpful error message upon validation error dur…

…ing import

CHANGELOG.md

@@ -1,5 +1,6 @@
 - Tasks: fix several bugs in ICS import
 - OAuth2 client: return more informative exception if a session variable is missing.
+- Core: Slightly more helpful debug data upon validation errors import
 
 12-12-2024: 6.8.90
 - Core: Show database error on upgrade

www/go/core/data/convert/AbstractConverter.php

@@ -198,6 +198,8 @@ private function notifyCount(int $count, int $errorCount) {
 	}
 
 	private function notifyError(string $error) {
+		// JH20241223: TODO? This alert is overwritten immediately. Would it not make sense to make an alert for each faild import...
+		// ...so instead of using $this->>alert just create its own alert?
 		$this->alert = new Alert();
 
 
@@ -278,10 +280,13 @@ public final function importFile(File $file, array $params = array()): array
 				EntityType::push(100);
 
 				if($entity->hasValidationErrors()) {
-					$msg = "Item ". $this->index . ": ". var_export($entity->getValidationErrors(), true);
-					$this->notifyError($msg);
+					foreach ($entity->getValidationErrors() as $key =>  $validationError) {
+						$msg = "Validation error in item " . $this->index . ": " . $key . " - " . $validationError['description'];
+						$this->notifyError($msg);
 
-					$response['errors'][] = $msg;
+						$response['errors'][] = $msg;
+
+					}
 				} elseif($this->afterSave($entity)) {
 					$response['count']++;
 				} else{