Skip to content

[stable31] fix(theming): enforce theme also for login - #51753

Merged
skjnldsv merged 1 commit into
stable31from
backport/50121/stable31
Mar 28, 2025
Merged

[stable31] fix(theming): enforce theme also for login#51753
skjnldsv merged 1 commit into
stable31from
backport/50121/stable31

Conversation

@backportbot

@backportbotbackportbotBot commented Mar 27, 2025

Copy link
Copy Markdown

Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
@skjnldsv
skjnldsv merged commit fa2b03d into stable31Mar 28, 2025
@skjnldsv
skjnldsv deleted the backport/50121/stable31 branch March 28, 2025 08:36
@AltahrimAltahrim mentioned this pull request Apr 3, 2025
2 tasks
Comment on lines +38 to +40
<body id="<?php p($_['bodyid']);?>" <?php foreach ($_['enabledThemes'] as $themeId) {
p("data-theme-$themeId ");
}?> data-themes="<?php p(join(',', $_['enabledThemes'])) ?>">

@nickvergessennickvergessenApr 3, 2025

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This breaks public/error pages it seems:

2025-04-03T20:40:27.000 m88 IP PHP Undefined array key "enabledThemes" at /var/www/cloud.nextcloud.com/nextcloud/core/templates/layout.guest.php#38 │
2025-04-03T20:40:28.000 m88 IP PHP foreach() argument must be of type array|object, null given at /var/www/cloud.nextcloud.com/nextcloud/core/templates/layout.guest.php#38 │
2025-04-03T20:40:28.000 m88 IP PHP Undefined array key "enabledThemes" at /var/www/cloud.nextcloud.com/nextcloud/core/templates/layout.guest.php#40 │
2025-04-03T20:40:28.000 m88 IP core Rendering themed error page failed. Falling back to un-themed error page. │
2025-04-03T20:40:28.000 m88 IP PHP Undefined array key "enabledThemes" at /var/www/cloud.nextcloud.com/nextcloud/core/templates/layout.guest.php#38 │
2025-04-03T20:40:29.000 m88 IP PHP foreach() argument must be of type array|object, null given at /var/www/cloud.nextcloud.com/nextcloud/core/templates/layout.guest.php#38 │
2025-04-03T20:40:29.000 m88 IP PHP Undefined array key "enabledThemes" at /var/www/cloud.nextcloud.com/nextcloud/core/templates/layout.guest.php#40 │
2025-04-03T20:40:29.000 m88 IP index join(): Argument #1 ($pieces) must be of type array, string given │
2025-04-03T20:40:29.000 m88 IP PHP Undefined array key "enabledThemes" at /var/www/cloud.nextcloud.com/nextcloud/core/templates/layout.guest.php#38 │
2025-04-03T20:40:29.000 m88 IP PHP foreach() argument must be of type array|object, null given at /var/www/cloud.nextcloud.com/nextcloud/core/templates/layout.guest.php#38 │
2025-04-03T20:40:30.000 m88 IP PHP Undefined array key "enabledThemes" at /var/www/cloud.nextcloud.com/nextcloud/core/templates/layout.guest.php#40 │
2025-04-03T20:40:30.000 m88 IP PHP TypeError: join(): Argument #1 ($pieces) must be of type array, string given at /var/www/cloud.nextcloud.com/nextcloud/core/templates/layout.guest.php#40 

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Easiest to demo is:
https://localhost/index.php/doesnotexist instead of an error message you will only see the background.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

@nickvergessen@susnux@artonge@skjnldsv