fix: handle AppConfigTypeConflictException when array policy stored as string

When a policy with an array default (e.g. identify_methods) has its
value stored as a plain string by an external source (e.g. the Nextcloud
provisioning API), readSystemValue() was re-throwing
AppConfigTypeConflictException instead of gracefully falling back.

Return the raw string so the policy normalizer can decode it.

Signed-off-by: Vitor Mattos <[email protected]>

Author: vitormattos

lib/Service/Policy/Runtime/PolicySource.php

@@ -603,6 +603,16 @@ private function readSystemValue(string $key, mixed $defaultValue): mixed {
 				return $this->appConfig->getAppValueString($key, $defaultValue ? '1' : '0');
 			}
 
+			if (is_array($defaultValue)) {
+				// Value was stored as a scalar (e.g., by Nextcloud provisioning API).
+				// Return the raw string so the policy normalizer can decode it.
+				try {
+					return $this->appConfig->getAppValueString($key, '');
+				} catch (AppConfigTypeConflictException) {
+					return $defaultValue;
+				}
+			}
+
 			throw $exception;
 		}
 	}