From 38272c3b11e884a6fb615f634cd0e167b32bee2f Mon Sep 17 00:00:00 2001
From: Moritz Weinmann <mow@breadcrumb-solutions.de>
Date: Mon, 22 Jun 2026 09:44:50 +0200
Subject: [PATCH] Intranet Cardform: zwei Nummernserien auf ein Array
 konsolidieren
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Statt $type_array (1-basiert) + $type_name_array (0-basiert) gibt es
jetzt ein einziges assoziatives Array $types [typ_nummer => name].
input_select bekommt array_keys/array_values — keine Verhaltensänderung,
aber das Muster ist ab sofort konsistent und erweiterbar.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
---
 .../siteparts/edit_intranet_cardform.inc.php  | 92 +++++++++----------
 1 file changed, 44 insertions(+), 48 deletions(-)

diff --git a/module/intranet/siteparts/edit_intranet_cardform.inc.php b/module/intranet/siteparts/edit_intranet_cardform.inc.php
index e4f446a..20454dd 100644
--- a/module/intranet/siteparts/edit_intranet_cardform.inc.php
+++ b/module/intranet/siteparts/edit_intranet_cardform.inc.php
@@ -41,54 +41,50 @@ if ($messages !== null) {
     } 
  }
 
-//Arrays für Select füllen
-$type_array = array(
-	1, 2, 3, 4, 5, 6, 7, 8, 9, 10, 11, 12, 13, 14, 15, 16, 17, 18, 19, 20, 21, 22, 23, 24, 25, 26, 27,
-	28, 29, 30, 31, 32, 33, 34, 35, 36, 37, 38, 39
-);
-$type_name_array = array(
-	0  => "Kontaktdaten",
-	1  => "Schwarzes Brett",
-	2  => "Kontakt bearbeiten",
-	3  => "Essenslieferung",
-	4  => "Profil",
-	5  => "Infoboard",
-	6  => "Knowledgecenter",
-	7  => "Firmeninformationen",
-	8  => "Meine Kontakte",
-	9  => "Knowledgecenter - Weitere Beiträge",
-	10 => "E-Learning",
-	11 => "Zertifikate - Kontaktkarte",
-	12 => "Essenslieferung Übersicht",
-	13 => "Zukünftige Termine",
-	14 => "E-Learning - Kontaktkarte",
-	15 => "Playground",
-	16 => "Tasks",
-	17 => "Tasks Vorschau",
-	18 => "Knowledgecenter - Custom Feed",
-	19 => "Arbeitsplatz Übersicht",
-	20 => "Personenkarten",
-	21 => "Organigramm",
-	22 => "PDF Suchen",
-	23 => "PDF Vorschau",
-	24 => "QM Updates",
-	25 => "QR-Code",
-	26 => "Ticketcenter",
-	27 => "Knowledgecenter Update",
-	28 => "Globale Suche",
-	29 => "Aufgaben Admin Aufgabenliste",
-	30 => "Aufgaben Admin Vorschau",
-	31 => "Aufgaben Admin Eskalations Dashboard",
-	32 => "Aufgaben Admin Freigabe Dashboard",
-	33 => "Aufgaben Admin Kategorien",
-	34 => "Aufgaben User Aufgaben",
-	35 => "Aufgaben User Aufgabe Bearbeitung",
-	36 => "Aufgaben User Überfällig",
-	37 => "Aufgaben User Hinweisleiste",
-	38 => "Aufgaben User Eskalations Overlay",
-);
+//Arrays für Select füllen — Typ-Nummer direkt als Key
+$types = [
+	1  => "Kontaktdaten",
+	2  => "Schwarzes Brett",
+	3  => "Kontakt bearbeiten",
+	4  => "Essenslieferung",
+	5  => "Profil",
+	6  => "Infoboard",
+	7  => "Knowledgecenter",
+	8  => "Firmeninformationen",
+	9  => "Meine Kontakte",
+	10 => "Knowledgecenter - Weitere Beiträge",
+	11 => "E-Learning",
+	12 => "Zertifikate - Kontaktkarte",
+	13 => "Essenslieferung Übersicht",
+	14 => "Zukünftige Termine",
+	15 => "E-Learning - Kontaktkarte",
+	16 => "Playground",
+	17 => "Tasks",
+	18 => "Tasks Vorschau",
+	19 => "Knowledgecenter - Custom Feed",
+	20 => "Arbeitsplatz Übersicht",
+	21 => "Personenkarten",
+	22 => "Organigramm",
+	23 => "PDF Suchen",
+	24 => "PDF Vorschau",
+	25 => "QM Updates",
+	26 => "QR-Code",
+	27 => "Ticketcenter",
+	28 => "Knowledgecenter Update",
+	29 => "Globale Suche",
+	30 => "Aufgaben Admin Aufgabenliste",
+	31 => "Aufgaben Admin Vorschau",
+	32 => "Aufgaben Admin Eskalations Dashboard",
+	33 => "Aufgaben Admin Freigabe Dashboard",
+	34 => "Aufgaben Admin Kategorien",
+	35 => "Aufgaben User Aufgaben",
+	36 => "Aufgaben User Aufgabe Bearbeitung",
+	37 => "Aufgaben User Überfällig",
+	38 => "Aufgaben User Hinweisleiste",
+	39 => "Aufgaben User Eskalations Overlay",
+];
 
-$input_description = (empty($input_intranet['description']) ? $type_name_array[$input_intranet['type']] : $input_intranet['description']);
+$input_description = (empty($input_intranet['description']) ? ($types[$input_intranet['type']] ?? '') : $input_intranet['description']);
 
 ?>
 
@@ -104,7 +100,7 @@ $input_description = (empty($input_intranet['description']) ? $type_name_array[$
 			<td>
 				<div class='label'><label for="input_description"><?= $translation->get('description') ?></label></div>
 				<div class='input'><input type="text" name="input_description" id="input_description" value="<?= $input_description ?>" class="text"></div>
-				<? input_select($translation->get('content_type'),'input_type',$type_array, $type_name_array, $input_intranet['type']); ?>
+				<? input_select($translation->get('content_type'), 'input_type', array_keys($types), array_values($types), $input_intranet['type']); ?>
 				<?php
 				if ($input_intranet['id'] != "" && $input_intranet['type'] == 12 || $input_intranet['type'] == 15) { // E-Learning - Kontaktkarte
 					echo input('Mit Personenkarte anzeigen', "input_learning_certificate_type", "checkbox", $input_intranet["learning_certificate_type"], 255);