Skip to content

Commit

Permalink
Merge pull request #4128 from nextcloud/backport/4124/stable30
Browse files Browse the repository at this point in the history
[stable30] Fix: Fix Admin Settings page when Watermark label selected
  • Loading branch information
elzody authored Oct 15, 2024
2 parents e830d98 + 16ac63f commit 6342855
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 @@ -48,6 +48,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 @@ -117,7 +122,11 @@ public function getAppSettings() {
$value = $value === 'yes' ? true : $value;
$result[$key] = $value === 'no' ? false : $value;
}
if (!empty(self::INTEGER_LIST_KEYS[$key])) {
$result[$key] = array_map('intval', $result[$key] ?? []);
}
}

return $result;
}

Expand Down

0 comments on commit 6342855

Please sign in to comment.