1.3.0-b2

Author: LukeWCS

lukewcs/togglectrl/adm/style/acp_togglectrl_settings.html

@@ -12,8 +12,6 @@
 
 {% INCLUDE 'overall_header.html' %}
 
-{#% set switch_type = TOGGLECTRL_TYPE ?? 'toggle' %#}
-
 <a id="maincontent"></a>
 
 <h1>{{ lang('TOGGLECTRL_CONFIG_TITLE') }}</h1>
@@ -31,7 +29,6 @@ <h1>{{ lang('TOGGLECTRL_CONFIG_TITLE') }}</h1>
 				<span>{{ lang('TOGGLECTRL_ENABLED_EXP') }}</span>
 			</dt>
 			<dd>
-				{#{ _self.switch('togglectrl_enabled', TOGGLECTRL_ENABLED, switch_type) }#}
 				{{ _self.switch('togglectrl_enabled', TOGGLECTRL_ENABLED) }}
 			</dd>
 		</dl>
@@ -66,9 +63,8 @@ <h1>{{ lang('TOGGLECTRL_CONFIG_TITLE') }}</h1>
 {% INCLUDECSS '@lukewcs_togglectrl/acp_togglectrl_settings.css' %}
 {% INCLUDEJS '@lukewcs_togglectrl/acp_togglectrl_settings.js' %}
 
-{#% macro switch(name, checked = false, type = 'toggle') -%#}
 {% macro switch(name, checked = false, type = null) -%}
-	{% set type = type ?? TOGGLECTRL_TYPE ?? 'toggle' %}
+	{% set type = type ?? TOGGLECTRL_TYPE ?? 'toggle' -%}
 	{% if type == 'toggle' || type == 'checkbox' -%}
 		<input type="checkbox" class="{{ type }}" name="{{ name }}" value="1"{{ checked ? ' checked' }}>
 	{%- elseif type == 'radio' -%}

lukewcs/togglectrl/adm/style/acp_togglectrl_settings.js

@@ -43,8 +43,8 @@
 	$(function () {
 		setState();
 
-		$('#togglectrl_settings')			.on('keypress'	, disableEnter);
-		$('#togglectrl_settings')			.on('reset'		, formReset);
-		$('[name="togglectrl_enabled"]')	.on('change'	, setState);
+		$('#togglectrl_settings')		.on('keypress'	, disableEnter);
+		$('#togglectrl_settings')		.on('reset'		, formReset);
+		$('[name="togglectrl_enabled"]').on('change'	, setState);
 	});
 })(jQuery);

lukewcs/togglectrl/composer.json

@@ -3,8 +3,8 @@
     "type": "phpbb-extension",
     "description": "Makes it possible to decide centrally for all extensions which visual style should be used for yes/no switches, provided extensions support Toggle Control.",
     "homepage": "https://github.com/LukeWCS/toggle-control",
-    "version": "1.3.0-b1",
-    "time": "2026-03-25",
+    "version": "1.3.0-b2",
+    "time": "2026-03-26",
     "license": "GPL-2.0-only",
     "authors": [
         {

lukewcs/togglectrl/config/services.yml

@@ -3,7 +3,6 @@ services:
         class: lukewcs\togglectrl\event\listener
         arguments:
             - '@config'
-            # - '@template'
             - '@template.twig.environment'
         tags:
             - { name: event.listener }

lukewcs/togglectrl/controller/acp_togglectrl_controller.php

@@ -55,19 +55,6 @@ public function module_settings(): void
 			$notes[] = $lang_outdated_msg;
 		}
 
-		// $template_vars = [
-			// 'TOGGLECTRL_NOTES'				=> $notes,
-			// 'TOGGLECTRL_ENABLED'			=> (bool) $this->config['togglectrl_enabled'],
-			// 'TOGGLECTRL_TYPES' 				=> $this->select_struct((string) $this->config['togglectrl_type'], [
-				// 'TOGGLECTRL_TYPE_TOGGLE'	=> 'toggle',
-				// 'TOGGLECTRL_TYPE_CHECKBOX'	=> 'checkbox',
-				// 'TOGGLECTRL_TYPE_RADIO'		=> 'radio',
-			// ]),
-		// ];
-		// $template_vars += !$this->config['togglectrl_enabled'] ? [
-			// 'TOGGLECTRL_TYPE'				=> (string) $this->config['togglectrl_type'],
-		// ] : [];
-		// $this->template->assign_vars($template_vars);
 		$this->template->assign_vars([
 			'TOGGLECTRL_NOTES'				=> $notes,
 			'TOGGLECTRL_ENABLED'			=> (bool) $this->config['togglectrl_enabled'],

lukewcs/togglectrl/event/listener.php

@@ -19,7 +19,6 @@ class listener implements EventSubscriberInterface
 {
 	public function __construct(
 		protected \phpbb\config\config $config,
-		// protected \phpbb\template\template $template,
 		protected \phpbb\template\twig\environment $twig_env,
 	)
 	{
@@ -38,7 +37,6 @@ public function generate_template_vars(): void
 	{
 		if ($this->config['togglectrl_enabled'])
 		{
-			// $this->template->assign_var('TOGGLECTRL_TYPE', (string) $this->config['togglectrl_type']);
 			$this->twig_env->addGlobal('TOGGLECTRL_TYPE', (string) $this->config['togglectrl_type']);
 		}
 	}

toggle-control_build_changelog.md

@@ -1,11 +1,14 @@
+#### 1.3.0-b2
+* Code (PHP, HTML) bezüglich neuer Schnittstellen-Integration bereinigt.
+
 #### 1.3.0-b1
 * Optimierung der TC Schnittstelle:
   * Die Zeile mit der Deklaration für `switch_type` am Anfang des Templates entfällt komplett.
   * Der dritte Parameter `type` bei `switch()` entfällt bei sämtlichen Schaltern. Somit reduziert sich der Makro Aufruf auf das Nötigste.
-  * Die Template Variable `TOGGLECTRL_TYPE` wird nicht mehr über das Template System lokal generiert, sondern über Twig direkt als globale Variable.
-  * Neues Makro `switch()` das mit der neuen globalen Template Variable arbeitet.
+  * Die Template Variable `TOGGLECTRL_TYPE` wird nicht mehr über das Template System lokal generiert, sondern direkt über Twig als globale Variable.
+  * Aktualisiertes Makro `switch()` das mit der neuen globalen Template Variable arbeitet.
   * Sämtlicher alter Code ist kurzfristig noch vorhanden, aber deaktiviert.
-* Der eingestellte Schalter-Stil wird nur dann angewendet, wenn die TC Funktion auch aktiviert ist.
+* Der eingestellte Schalter-Stil wird im TC ACP Modul nur noch dann angewendet, wenn die TC Funktion auch aktiviert ist.
 * PHP Voraussetzung: 
   * Max. 8.4.0 -> 8.5.0.
 

toggle-control_changelog_de.md

@@ -1,13 +1,13 @@
 ### 1.3.0
-WIP
+(WIP)
 
 * Die Voraussetzungen haben sich geändert:
   * PHP: 8.0.0 - 8.5.x (Bisher: 8.0.0 - 8.4.x)
 * Für Erweiterungen Entwickler ist die Integration der TC Schnittstelle einfacher geworden:
-  * Die Zeile mit der Deklaration für `switch_type` am Anfang des Templates entfällt komplett.
-  * Der dritte Parameter `type` bei `switch()` entfällt bei sämtlichen Schaltern. Somit reduziert sich der Makro Aufruf auf das Nötigste.
+  * Die Zeile mit der Deklaration für `switch_type` am Anfang des Templates entfällt komplett, also `{% set switch_type = ...`.
+  * Der dritte Parameter bei `switch(name, checked, type)` ist bei sämtlichen Schaltern nur noch optional und wird nicht mehr zwingend benötigt. Für Abwärtskompatibilität ist der Parameter aber weiterhin funktionsfähig. Somit reduziert sich der Makro Aufruf auf das Nötigste.
   * Die oben genannten Eigenschaften sind deswegen nicht mehr nötig, da TC jetzt eine echte globale Template Variable generieren kann, die auch im Makro verfügbar ist. Es muss also lediglich das neue `switch()` Makro integriert werden um TC Kompatibilität zu schaffen, um alles Weitere kümmert sich das Makro.
-* Bisher hatte TC unabhängig vom Aktivierungszustand der TC Funktion immer den eingestellten Schalter-Stil angezeigt. Jetzt verhält sich TC konsequent wie jeder andere Erweiterung, das heisst der eingestellte Schalter-Stil wird nur dann angewendet, wenn die TC Funktion auch aktiviert ist.
+* Bisher hatte TC im eigenen ACP Modul unabhängig vom Aktivierungszustand der TC Funktion immer den eingestellten Schalter-Stil angewendet. Jetzt verhält sich TC auch beim eigenen ACP Modul konsequent wie jeder andere Erweiterung, das heisst der eingestellte Schalter-Stil wird nur noch dann angewendet, wenn die TC Funktion auch aktiviert ist. Ansonsten gilt der Standard, also `toggle`.
 
 ### 1.2.0
 (2025-10-20)