e107inc / e107

e107 Bootstrap CMS (Content Management System) v2 with PHP, MySQL, HTML5, jQuery and Twitter Bootstrap. Issue Discussion Room: https://gitter.im/e107inc/e107
https://e107.org
GNU General Public License v3.0
318 stars 212 forks source link

[Bug]: e107_admin/theme.php PHP-8.1.25 - Bootstrap 5 theme gives empty screen.. (Dropdowns nav menu also do not work anymore) #5113

Closed Alex-e107nl closed 7 months ago

Alex-e107nl commented 7 months ago

What e107 version are you using?

Latest Github version (just updated)

Bug description

Empty main screen when you use php 8.1.25 when you try to open config of installed Bootstrap5 theme (standard) in the dutch language, in english it works, with php 7.4 both languages work perfectly. The standard bootstrap 3 theme works perfectly!

Problem_admin_themephp_23-11-2023

How to reproduce

Fresh install e107 (github 23-11-2023) PHP 8.1.25 use Dutch language for admin and go to Thema Manager

Expected behavior

To see the config of the standard Bootstrap 5 theme... With PHP 7.4 it works as expected!

What browser(s) are you seeing the problem on?

Firefox

PHP Version

PHP 8.1.25

Jimmi08 commented 7 months ago

More info: image

it would help if you fixed the Dutch translation, those strings are missing in Dutch_admin.php:

define("LAN_THEMEPREF_04", "Site Name");
define("LAN_THEMEPREF_05", "Logo");
define("LAN_THEMEPREF_06", "Logo & Site Name");
define("LAN_THEMEPREF_07", "left");
define("LAN_THEMEPREF_08", "right");
Alex-e107nl commented 7 months ago

Done that here locally, did not change anything... i thought the same... I wil edit the language files :-)

Jimmi08 commented 7 months ago

define("LAN_THEMEPREF_04", "Site Name"); define("LAN_THEMEPREF_05", "Logo"); define("LAN_THEMEPREF_06", "Logo & Site Name"); define("LAN_THEMEPREF_07", "left"); define("LAN_THEMEPREF_08", "right");

add this too, it was next error. define("LAN_THEMEPREF_00", "Branding:");

Alex-e107nl commented 7 months ago

The translation of TGTJE and me is on https://sourceforge.net/projects/e107dutch/ that is what we use....

Jimmi08 commented 7 months ago

There is another error with that pack: Fatal error: Uncaught Error: Undefined constant "PREVIEWTHEMENAME" in C:\wamp64\www\e107core\e107_languages\Dutch\admin\lan_theme.php on line 12

CaMer0n commented 7 months ago

@Alex-e107nl As this appears to be a language-pack issue, I'll close this. Feel free to open an issue here: https://github.com/e107translations/Dutch about it.