fix(FilenameValidator): use `_` as default replacement for invalid charsfix/validation-defaults

Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>

2 files changed, 2 insertions, 2 deletions

diff --git a/lib/private/Files/FilenameValidator.php b/lib/private/Files/FilenameValidator.php
index 57a62b0b219..a78c6d3cc3c 100644
--- a/lib/private/Files/FilenameValidator.php
+++ b/lib/private/Files/FilenameValidator.php
@@ -232,7 +232,7 @@ class FilenameValidator implements IFilenameValidator {
 		$forbiddenCharacters = $this->getForbiddenCharacters();
 
 		if ($charReplacement === null) {
-			$charReplacement = array_diff([' ', '_', '-'], $forbiddenCharacters);
+			$charReplacement = array_diff(['_', '-', ' '], $forbiddenCharacters);
 			$charReplacement = reset($charReplacement) ?: '';
 		}
 		if (mb_strlen($charReplacement) !== 1) {
diff --git a/lib/public/Files/IFilenameValidator.php b/lib/public/Files/IFilenameValidator.php
index d8bd06d179d..9b7fa1e2e2e 100644
--- a/lib/public/Files/IFilenameValidator.php
+++ b/lib/public/Files/IFilenameValidator.php
@@ -43,7 +43,7 @@ interface IFilenameValidator {
 	 * If no sanitizing is needed the same name is returned.
 	 *
 	 * @param string $name The filename to sanitize
-	 * @param null|string $charReplacement Character to use for replacing forbidden ones - by default space, dash or underscore is used if allowed.
+	 * @param null|string $charReplacement Character to use for replacing forbidden ones - by default underscore, dash or space is used if allowed.
 	 * @throws \InvalidArgumentException if no character replacement was given (and the default could not be applied) or the replacement is not valid.
 	 * @since 32.0.0
 	 */