Skip to content

Commit

Permalink
Merge pull request #4127 from nextcloud/backport/4124/stable29
Browse files Browse the repository at this point in the history
[stable29] Fix: Fix Admin Settings page when Watermark label selected
  • Loading branch information
elzody authored Oct 15, 2024
2 parents 04db20f + 9f7742b commit 3feca29
Showing 1 changed file with 9 additions and 0 deletions.
9 changes: 9 additions & 0 deletions lib/AppConfig.php
Original file line number Diff line number Diff line change
Expand Up @@ -54,6 +54,11 @@ class AppConfig {
'watermark_linkTagsList' => 'array'
];

private const INTEGER_LIST_KEYS = [
'watermark_allTagsList' => true,
'watermark_linkTagsList' => true,
];

public function __construct(
private IConfig $config,
private IAppManager $appManager,
Expand Down Expand Up @@ -123,7 +128,11 @@ public function getAppSettings() {
$value = $value === 'yes' ? true : $value;
$result[$key] = $value === 'no' ? false : $value;
}
if (array_key_exists($key, self::INTEGER_LIST_KEYS)) {
$result[$key] = array_map('intval', $result[$key] ?? []);
}
}

return $result;
}

Expand Down

0 comments on commit 3feca29

Please sign in to comment.