Merge pull request #18445 from owncloud/ext-only-setuservars-string

setUserVars() should only attempt substitution with strings

1 files changed, 8 insertions, 0 deletions

diff --git a/apps/files_external/service/storagesservice.php b/apps/files_external/service/storagesservice.php
index 282ac8f2db1..3e2152741e5 100644
--- a/apps/files_external/service/storagesservice.php
+++ b/apps/files_external/service/storagesservice.php
@@ -118,6 +118,8 @@ abstract class StoragesService {
 			$applicableGroups[] = $applicable;
 			$storageConfig->setApplicableGroups($applicableGroups);
 		}
+
+
 		return $storageConfig;
 	}
 
@@ -238,6 +240,12 @@ abstract class StoragesService {
 			$this->setRealStorageIds($storages, $storagesWithConfigHash);
 		}
 
+		// convert parameter values
+		foreach ($storages as $storage) {
+			$storage->getBackend()->validateStorageDefinition($storage);
+			$storage->getAuthMechanism()->validateStorageDefinition($storage);
+		}
+
 		return $storages;
 	}