[locations] Modularize additional column handling

Additional columns are now added through a hook,
moving specialized code where it belongs.

3 files changed, 75 insertions, 16 deletions

diff --git a/modules-available/sysconfig/hooks/locations-column.inc.php b/modules-available/sysconfig/hooks/locations-column.inc.php
new file mode 100644
index 00000000..f60a852d
--- /dev/null
+++ b/modules-available/sysconfig/hooks/locations-column.inc.php
@@ -0,0 +1,57 @@
+<?php
+
+if (!User::hasPermission('.sysconfig.config.*') || !Module::isAvailable('sysconfig'))
+	return null;
+
+class SysconfigLocationColumn extends AbstractLocationColumn
+{
+
+	private $lookup = [];
+
+	public function __construct()
+	{
+		$confs = SysConfig::getAll();
+		foreach ($confs as $conf) {
+			if (strlen($conf['locs']) === 0)
+				continue;
+			$confLocs = explode(',', $conf['locs']);
+			foreach ($confLocs as $locId) {
+				$this->lookup[$locId] = $conf['title'];
+			}
+		}
+	}
+
+	public function getColumnHtml(int $locationId): string
+	{
+		return htmlspecialchars($this->lookup[$locationId] ?? '');
+	}
+
+	public function getEditUrl(int $locationId): string
+	{
+		if (!User::hasPermission('.sysconfig.config.assign', $locationId))
+			return '';
+		return '?do=sysconfig&locationid=' . $locationId;
+	}
+
+	public function header(): string
+	{
+		return Dictionary::translateFileModule('sysconfig', 'module', 'location-column-header');
+	}
+
+	public function priority(): int
+	{
+		return 2000;
+	}
+
+	public function propagateColumn(): bool
+	{
+		return true;
+	}
+
+	public function propagateDefaultHtml(): string
+	{
+		return htmlspecialchars($this->lookup[0] ?? '');
+	}
+}
+
+return new SysconfigLocationColumn();
\ No newline at end of file
diff --git a/modules-available/sysconfig/lang/de/module.json b/modules-available/sysconfig/lang/de/module.json
index 1dbb268b..4bc1642f 100644
--- a/modules-available/sysconfig/lang/de/module.json
+++ b/modules-available/sysconfig/lang/de/module.json
@@ -6,8 +6,12 @@
     "lang_moduleAssign": "Modul zu Systemkonfigurationen zuweisen",
     "lang_noModuleFromThisGroup": "(Kein Modul dieser Gruppe)",
     "lang_unknwonTaskManager": "Unbekannter Taskmanager-Fehler",
+    "location-column-header": "Lokalisierung",
     "module_name": "Lokalisierung + Integration",
     "page_title": "Lokalisierung + Integration",
+    "saver_DescriptionIdleKill": "Ein Bildschirmschoner mit Timeout, nach dessen Ablauf alle Anwendungen ohne weitere Nachfragen geschlossen werden und der Nutzer ausgeloggt wird.",
+    "saver_DescriptionNoTimeout": "Ein Bildschirmschoner ohne Timeout.",
+    "saver_DescriptionShutdown": "Ein Bildschirmschoner mit Timeout, nach dessen Ablauf alle Anwendungen ohne weitere Nachfragen geschlossen werden und der PC heruntergefahren oder neugestartet wird.",
     "saver_MessageDefaultIdleKill": "Diese Sitzung wird bei Inaktivit\u00e4t in %1 beendet.",
     "saver_MessageDefaultIdleKillLocked": "Diese Sitzung wird in %1 beendet, wenn sie nicht entsperrt wird.",
     "saver_MessageDefaultNoTimeout": "Dieser Bildschirm wird gerade geschont.",
@@ -15,13 +19,10 @@
     "saver_MessageDefaultShutdown": "Achtung: Rechner wird in %1 heruntergefahren!",
     "saver_MessageDefaultShutdownLocked": "Achtung: Rechner wird in %1 heruntergefahren!",
     "saver_QssDefault": "#Saver {\r\n  background: qlineargradient(spread:pad, x1:0, y1:0, x2:0, y2:1, stop:0 #443, stop:1 #000)\r\n}\r\n\r\nQLabel {\r\n  color: #f64;\r\n}\r\n\r\n#lblClock {\r\n  color: #999;\r\n  font-size: 20pt;\r\n}\r\n\r\n#lblHeader {\r\n  font-size: 20pt;\r\n}\r\n",
-    "saver_DescriptionIdleKill": "Ein Bildschirmschoner mit Timeout, nach dessen Ablauf alle Anwendungen ohne weitere Nachfragen geschlossen werden und der Nutzer ausgeloggt wird.",
-    "saver_DescriptionNoTimeout": "Ein Bildschirmschoner ohne Timeout.",
-    "saver_DescriptionShutdown": "Ein Bildschirmschoner mit Timeout, nach dessen Ablauf alle Anwendungen ohne weitere Nachfragen geschlossen werden und der PC heruntergefahren oder neugestartet wird.",
-    "saver_TitleIdleKill": "Idle Kill",
-    "saver_TitleNoTimeout": "Ohne Timeout",
-    "saver_TitleShutdown": "Herunterfahren",
     "saver_TextDefaultIdleKill": "<html><body>Keine Nutzeraktivit\u00e4t festgestellt. <br>Zum oben angegebenen Zeitpunkt wird die aktuell laufende Sitzung beendet, wenn der Rechner nicht mehr verwendet wird. <br>Alle noch laufenden Programme <br>werden ohne Nachfrage geschlossen. Stellen Sie daher sicher, bis zum angegebenen Zeitpunkt <br>s\u00e4mtliche sich in Bearbeitung befindlichen Daten abzuspeichern. <br><br>Dies dient dazu zu vermeiden, dass ein Rechner stundenlang gesperrt wird und somit <br>anderen Nutzern nicht zur Verf\u00fcgung steht.<\/body><\/html>",
     "saver_TextDefaultIdleKillLocked": "<html><body><br>Zum oben angegebenen Zeitpunkt wird die aktuell laufende Sitzung beendet, wenn sie zuvor nicht wieder entsperrt wird. <br>Alle noch laufenden Programme werden ohne Nachfrage geschlossen. <br>Stellen Sie daher sicher, bis zum angegebenen Zeitpunkt <br>s\u00e4mtliche sich in Bearbeitung befindlichen Daten abzuspeichern, bzw. die Sitzung wieder zu entsperren. <br><br>Dies dient dazu zu vermeiden, dass ein Rechner stundenlang gesperrt wird und somit<br>anderen Nutzern nicht zur Verf\u00fcgung steht.<\/body><\/html>",
-    "saver_TextDefaultShutdown": "<html><body>Achtung: Zum oben angegebenen Zeitpunkt wird der Computer heruntergefahren bzw. neugestartet. <br>Alle noch laufenden Programme werden ohne Nachfrage beendet. Stellen Sie daher sicher, bis <br>zum angegebenen Zeitpunkt s\u00e4mtliche Daten abzuspeichern und die Sitzung zu verlassen.<\/body><\/html>"
-}
+    "saver_TextDefaultShutdown": "<html><body>Achtung: Zum oben angegebenen Zeitpunkt wird der Computer heruntergefahren bzw. neugestartet. <br>Alle noch laufenden Programme werden ohne Nachfrage beendet. Stellen Sie daher sicher, bis <br>zum angegebenen Zeitpunkt s\u00e4mtliche Daten abzuspeichern und die Sitzung zu verlassen.<\/body><\/html>",
+    "saver_TitleIdleKill": "Idle Kill",
+    "saver_TitleNoTimeout": "Ohne Timeout",
+    "saver_TitleShutdown": "Herunterfahren"
+}
\ No newline at end of file
diff --git a/modules-available/sysconfig/lang/en/module.json b/modules-available/sysconfig/lang/en/module.json
index b49cc1cf..dbfacdb8 100644
--- a/modules-available/sysconfig/lang/en/module.json
+++ b/modules-available/sysconfig/lang/en/module.json
@@ -6,8 +6,12 @@
     "lang_moduleAssign": "Assign Module to System Configurations",
     "lang_noModuleFromThisGroup": "(No module from this group)",
     "lang_unknwonTaskManager": "Unknown Task Manager error",
+    "location-column-header": "SysConfig",
     "module_name": "Localization",
     "page_title": "Localize and integrate",
+    "saver_DescriptionIdleKill": "A screensaver with a timeout which on it's expiration will close all running applications without further requests and logout the user.",
+    "saver_DescriptionNoTimeout": "A screensaver without a timeout.",
+    "saver_DescriptionShutdown": "A screensaver with a timeout which on it's expiration the PC will shutdown or restart. All applications will be closed without further requests.",
     "saver_MessageDefaultIdleKill": "This session will end in %1 when inactive.",
     "saver_MessageDefaultIdleKillLocked": "This session will end in %1 if the session is not unlocked.",
     "saver_MessageDefaultNoTimeout": "This screen is in saving mode.",
@@ -15,13 +19,10 @@
     "saver_MessageDefaultShutdown": "Caution: Computer will shutdown in %1!",
     "saver_MessageDefaultShutdownLocked": "Caution: Computer will shutdown in %1!",
     "saver_QssDefault": "#Saver {\r\n  background: qlineargradient(spread:pad, x1:0, y1:0, x2:0, y2:1, stop:0 #443, stop:1 #000)\r\n}\r\n\r\nQLabel {\r\n  color: #f64;\r\n}\r\n\r\n#lblClock {\r\n  color: #999;\r\n  font-size: 20pt;\r\n}\r\n\r\n#lblHeader {\r\n  font-size: 20pt;\r\n}\r\n",
-    "saver_DescriptionIdleKill": "A screensaver with a timeout which on it's expiration will close all running applications without further requests and logout the user.",
-    "saver_DescriptionNoTimeout": "A screensaver without a timeout.",
-    "saver_DescriptionShutdown": "A screensaver with a timeout which on it's expiration the PC will shutdown or restart. All applications will be closed without further requests.",
-    "saver_TitleIdleKill": "Idle Kill",
-    "saver_TitleNoTimeout": "No Timeout",
-    "saver_TitleShutdown": "Shutdown",
     "saver_TextDefaultIdleKill": "<html><body>No user activity detected. <br>If the computer is not used until the time specified above, the session will end. <br> All running applications <br>will be closed without further requests. Make sure that all files and changes are saved <br> before the time runs out. <br><br>It prevents computers from beeing locked for hours and <br>not beeing available for other users.<\/body><\/html>",
     "saver_TextDefaultIdleKillLocked": "<html><body><br>The current session will end by the time specified above if the computer isn't unlocked before. <br>All running applications will be closed without further requests. <br>Make sure that all files and changes are saved <br>or the session is unlocked before the time runs out. <br><br>It prevents computers from beeing locked for hours and <br>not beeing available for other users.<\/body><\/html>",
-    "saver_TextDefaultShutdown": "<html><body>Caution: The computer will shutdown or restart respectively at the specified time above. <br>All running applications will be closed without further requests. Make sure to save all files and changes and leave the session<br>before the time runs out.<\/body><\/html>"
-}
+    "saver_TextDefaultShutdown": "<html><body>Caution: The computer will shutdown or restart respectively at the specified time above. <br>All running applications will be closed without further requests. Make sure to save all files and changes and leave the session<br>before the time runs out.<\/body><\/html>",
+    "saver_TitleIdleKill": "Idle Kill",
+    "saver_TitleNoTimeout": "No Timeout",
+    "saver_TitleShutdown": "Shutdown"
+}
\ No newline at end of file