3.1.0-b18

Author: LukeWCS

ext-mgr-plus_build_changelog.md

@@ -1,6 +1,23 @@
+#### 3.1.0-b18
+* Core:
+  * Sowohl die Funktion für die VP Vorbereitung als auch die Funktion zur Anzeige der VP Daten im ExtMgr wurden robuster gestaltet bezüglich Auslesen der Ext-Metadaten aus `composer.json`.
+* Änderungen bei der Bestätigung bei teilweise geschalteten Erweiterungen:
+  * Statt eine orange Box mit schwarzer Schrift wird jetzt ein Farbverlauf von Grün zu Rot mit den phpBB Farben (`successbox`, `errorbox`) und weisser Schrift verwendet, was bei grösserem Inhalt angenehmer zu lesen ist.
+  * Die möglichen Meldungstypen für `trigger_error_` haben sich geändert:
+	* `0` (Okay: grün)
+	* `1` (Fehler: rot)
+	* `2` (Warnung: orange)
+	* `3` (Okay+Fehler: grün+rot)
+  * Hilfsfunktion `get_error_level` entsprechend angepasst.
+  * Im CSS neue Regeln für `emp_success_error` hinzugefügt.
+* Code Optimierung und weitere Modernisierung. Unter anderem Umstellung auf Union Types bei Funktionsparametern.
+* Code auch bei Listener und ACP Modul strikter gestaltet.
+* Die Voraussetzungen haben sich geändert:
+  * PHP: 8.0.0 - 8.4.x (Bisher: 7.4.0 - 8.4.x)
+
 #### 3.1.0-b17
 * Mit der EMP eigenen Variante von `trigger_error` können jetzt nicht nur positive Bestätigungen (Grün) und Fehlermeldungen (Rot) dargestellt werden, sondern auch Warnungen (Orange). Das ist eine konsequente Weiterentwicklung der Änderung in 1.0.1, bei der bereits eine einzige nicht geschaltete Ext genügte, dass für die Bestätigungsmeldung eine rote Box generiert wurde. Dazu waren Änderungen in verschiedenen Dateien notwendig:
-  * In der PHP Funktion `trigger_error_` wird basierend auf dem übergebenen Error Level in der Template Variable `EXTMGRPLUS_TRIGGER_ERROR` ein Wert von `0` (Info), `1` (Warnung) oder `2` (Fehler) definiert. Bisher wurde in dieser Variable lediglich `true` als Wert verwendet.
+  * In der PHP Funktion `trigger_error_` wird basierend auf dem übergebenen Error Level in der Template Variable `EXTMGRPLUS_TRIGGER_ERROR` ein Wert von `0` (Okay), `1` (Warnung) oder `2` (Fehler) definiert. Bisher wurde in dieser Variable lediglich `true` als Wert verwendet.
   * Im `trigger_error` Template wird anhand `EXTMGRPLUS_TRIGGER_ERROR` die passende CSS Klasse verwendet.
   * Im CSS neue Regeln für `emp_warning` hinzugefügt.
 * Basierend auf der neuen Möglichkeit Warnungen anzeigen zu können, wird jetzt bei teilweise geschalteten Erweiterungen keine rote Box mehr angezeigt, sondern eine orange Box. Eine rote Box wird nur noch dann angezeigt, wenn keine einzige Ext geschaltet werden konnte.
@@ -13,7 +30,6 @@
   * Code an die umbenannten Klassen angepasst.
 * CSS:
   * Regeln an die umbenannten Klassen angepasst.
-  * Regeln für die EMP Warnung hinzugefügt.
 
 #### 3.1.0-b16
 * Fix: Logikfehler bei b15 behoben. Beim Vergleich von Homepage Link und CDB Link wurde nicht berücksichtigt, dass der Homepage Link auch leer sein kann, was dann dazu führen konnte, dass die Zeile für Homepage mit leerem Link angezeigt wurde.

ext-mgr-plus_changelog_de.md

@@ -1,6 +1,8 @@
 ### 3.1.0
 (2025--) / CDB: --)
 
+* Die Voraussetzungen haben sich geändert:
+  * PHP: 8.0.0 - 8.4.x (Bisher: 7.4.0 - 8.4.x)
 * Fix: Wenn die PHP INI Variable `max_execution_time` den Wert `0` aufweist, dann bekommt die von phpBB generierte Variable `safe_time_limit` im original ExtMgr ebenfalls den Wert `0`, bedingt durch die Formel `safe_time_limit = max_execution_time / 2`. Das hatte zur Folge, dass beim Schalten von Erweiterungen fälschlicherweise der Timeout-Schutz von EMP gegriffen hat und die Aktion nach der ersten geschalteten Erweiterung abgebrochen wurde. Bei einem Wert `0` wird diese Prüfung jetzt korrekt übersprungen, da es in diesem Fall laut PHP Konfiguration keine Laufzeit-Begrenzung gibt. Das ist zwar ein eher seltenes Problem, ist jedoch trotzdem ein Fehler gewesen. [Meldung von Scanialady (phpBB.de)]
 * Fix: Auf der "Details" Seite wurden lange Link-Titel nicht in allen Fällen korrekt umgebrochen. Das hatte unter Umständen zur Folge, dass in der Responsive-Ansicht die Box "Informationen zur Erweiterung" breiter als die anderen Boxen dargestellt wurde, wodurch das Layout insgesamt auf der Seite nicht mehr stimmte. Dieses Problem ist auch bei phpBB selber vorhanden. [Meldung von Kirk (phpBB.de)]
 * In der Spalte "Vorgänge" gibt es den neuen Link "Einstellungen", mit dem direkt das primäre Einstellungsmodul einer Erweiterung aufgerufen werden kann, also das erste Modul das per Migration installiert wurde. Dabei gelten folgende Eigenschaften:
@@ -19,10 +21,13 @@
   * Bei CDB Erweiterungen wird jetzt das zugehörige Icon vor dem Namen eingefügt, inklusive Tooltip.
   * Bei CDB Erweiterungen wird die Anzeige des Homepage Links unterdrückt, wenn dieser Link identisch zur CDB Seite ist.
 * In den Einstellungen wurde der Schalter "Anleitungen anzeigen" in den Bereich "Experten-Einstellungen" verschoben.
+* Mit der EMP eigenen Variante von `trigger_error` können jetzt nicht nur positive Bestätigungen (Grün) und Fehlermeldungen (Rot) dargestellt werden, sondern auch Warnungen (Orange) und Positiv+Negativ (Grün+Rot).
+  * Basierend auf den neuen Meldungs-Typen, wird jetzt bei teilweise geschalteten Erweiterungen keine rote Box mehr angezeigt, sondern eine grün/rote Box mit Farbverlauf. Eine rote Box wird nur noch dann angezeigt, wenn keine einzige Ext geschaltet werden konnte.
 * Bei der Rückfrage beim Schalten mehrerer Erweiterungen, sind die Buttons jetzt mit "Deaktivieren / Abbrechen" und "Aktivieren / Abbrechen" beschriftet, wie es phpBB bei der eigenen Rückfrage beim Schalten einer Erweiterung ebenfalls macht. Bislang mussten diese Buttons bei EMP aus technischen Gründen mit "Ja / Nein" beschriftet sein, da sonst `confirm_box` von phpBB nicht funktioniert hätte. Möglich wurde diese Änderung durch eine Anpassung des EMP eigenen Templates der `confirm_box` Funktion, wodurch alternative Button Beschriftungen definiert werden können.
 * Im Tabellen-Kopf wird bei Icons jetzt ebenfalls der Hilfe-Cursor dargestellt, wie bei allen anderen Icons in der Tabelle.
 * Kleine Korrekturen in den Templates. Unter anderem wurde ein HTML Fehler behoben, der jedoch keine Auswirkung hatte, da Browser den Fehler automatisch "behoben" haben.
 * Code Optimierung bei PHP, Twig, Javascript und CSS.
+* Code Modernisierung bei PHP.
 * Javascript:
   * LukeWCSphpBBConfirmBox aktualisiert auf 1.5.1.
 

lukewcs/extmgrplus/acp/settings_module.php

@@ -15,9 +15,9 @@
 
 class settings_module
 {
-	public $page_title;
-	public $tpl_name;
-	public $u_action;
+	public string $page_title;
+	public string $tpl_name;
+	public string $u_action;
 
 	public function main()
 	{

lukewcs/extmgrplus/adm/style/acp_ext_mgr_plus.css

@@ -37,7 +37,7 @@ code {
 }
 
 .errorbox.emp_warning {
-	background-color: #ffbb33;
+	background-color: #ffaa00;
 }
 
 .errorbox.emp_warning :is(a, a:visited) {
@@ -48,6 +48,10 @@ code {
 	color: #303030;
 }
 
+.errorbox.emp_success_error {
+	background: linear-gradient(#228822, #bc2a4d);
+}
+
 .extmgrplus_footer {
 	margin-top: 1em;
 	text-align: center;

lukewcs/extmgrplus/adm/style/acp_ext_mgr_plus_message_body.html

@@ -22,7 +22,7 @@ <h1>{{ lang('EXTENSIONS_ADMIN') }}</h1>
 	<p>{{ lang('EXTENSION_' ~ EXTMGRPLUS_ACTION_MODE ~ '_EXPLAIN') }}</p>
 {%- endif %}
 
-<div class="{{ {0: 'successbox', 1: 'errorbox emp_warning', 2: 'errorbox'}[EXTMGRPLUS_TRIGGER_ERROR] }}">
+<div class="{{ {0: 'successbox', 1: 'errorbox', 2: 'errorbox emp_warning', 3: 'errorbox emp_success_error'}[EXTMGRPLUS_TRIGGER_ERROR] }}">
 	<h3>{{ MESSAGE_TITLE }}</h3>
 
 	<p>{{ MESSAGE_TEXT }}</p>

lukewcs/extmgrplus/composer.json

@@ -3,8 +3,8 @@
     "type": "phpbb-extension",
     "description": "Adds the option to disable and re-enable all installed extensions at once to the extension management. Individual selections can be made using checkboxes. In addition, sequence groups and dependencies can be defined or extensions can be ignored. The version check is carried out in blocks and provides detailed information. Additional information is displayed in the extension management.",
     "homepage": "https://github.com/LukeWCS/ext-mgr-plus",
-    "version": "3.1.0-b17",
-    "time": "2025-06-02",
+    "version": "3.1.0-b18",
+    "time": "2025-07-14",
     "license": "GPL-2.0-only",
     "authors": [
         {
@@ -19,7 +19,7 @@
         }
     ],
     "require": {
-        "php": ">=7.4.0,<8.5.0@dev",
+        "php": ">=8.0.0,<8.5.0@dev",
         "composer/installers": "~1.0.0"
     },
     "extra": {

lukewcs/extmgrplus/config/services.yml

@@ -21,8 +21,8 @@ services:
             - '@dbal.conn'
             - '@path_helper'
             - '%core.table_prefix%'
-            - '%core.root_path%'
-            - '%core.php_ext%'
+            # - '%core.root_path%'
+            # - '%core.php_ext%'
     lukewcs.extmgrplus.core.common:
         class: lukewcs\extmgrplus\core\ext_mgr_plus_common
         arguments:

lukewcs/extmgrplus/controller/acp_ext_mgr_plus_controller.php

@@ -15,27 +15,38 @@
 
 class acp_ext_mgr_plus_controller
 {
-	protected object $common;
-	protected object $language;
-	protected object $template;
-	protected object $request;
-	protected object $config;
-
+	// protected object $common;
+	// protected object $language;
+	// protected object $template;
+	// protected object $request;
+	// protected object $config;
+
+	// protected string $u_action;
+
+	// public function __construct(
+		// $common,
+		// \phpbb\language\language $language,
+		// \phpbb\template\template $template,
+		// \phpbb\request\request $request,
+		// \phpbb\config\config $config,
+	// )
+	// {
+		// $this->common	= $common;
+		// $this->language	= $language;
+		// $this->template	= $template;
+		// $this->request	= $request;
+		// $this->config	= $config;
+	// }
 	protected string $u_action;
 
 	public function __construct(
-		$common,
-		\phpbb\language\language $language,
-		\phpbb\template\template $template,
-		\phpbb\request\request $request,
-		\phpbb\config\config $config
+		protected object $common,
+		protected \phpbb\language\language $language,
+		protected \phpbb\template\template $template,
+		protected \phpbb\request\request $request,
+		protected \phpbb\config\config $config,
 	)
 	{
-		$this->common	= $common;
-		$this->language	= $language;
-		$this->template	= $template;
-		$this->request	= $request;
-		$this->config	= $config;
 	}
 
 	public function module_settings(): void