From cccdfa1ac862197355abf28133e4ac4ba340b2a8 Mon Sep 17 00:00:00 2001
From: Werner Masik <werner@gefi.at>
Date: Tue, 21 Nov 2023 17:45:13 +0100
Subject: [PATCH] extracted function for generation employee kurzbz

---
 .../models/ressource/Mitarbeiter_model.php    | 35 ++++++++++++-------
 1 file changed, 22 insertions(+), 13 deletions(-)

diff --git a/application/models/ressource/Mitarbeiter_model.php b/application/models/ressource/Mitarbeiter_model.php
index 90c30927f..7b7b8d4c0 100644
--- a/application/models/ressource/Mitarbeiter_model.php
+++ b/application/models/ressource/Mitarbeiter_model.php
@@ -171,7 +171,6 @@ class Mitarbeiter_model extends DB_Model
 	 */
 	public function generateKurzbz($uid)
 	{
-		$kurzbz = '';
 		$this->addLimit(1);
 		$this->addSelect('vorname, nachname');
 		$this->addJoin('public.tbl_benutzer', 'tbl_mitarbeiter.mitarbeiter_uid = tbl_benutzer.uid');
@@ -181,25 +180,35 @@ class Mitarbeiter_model extends DB_Model
 		if (hasData($nameresult))
 		{
 			$kurzbzdata = getData($nameresult);
-			$nachname_clean = sanitizeProblemChars($kurzbzdata[0]->nachname);
-			$vorname_clean = sanitizeProblemChars($kurzbzdata[0]->vorname);
+			$genKurzbz = $this->generateKurzbzHelper($kurzbzdata[0]->vorname, $kurzbzdata[0]->nachname);
 
-			for ($nn = 6, $vn = 2; $nn != 0; $nn--, $vn++)
-			{
-				$kurzbz = mb_substr($nachname_clean, 0, $nn);
-				$kurzbz .= mb_substr($vorname_clean, 0, $vn);
+			return $genKurzbz;
+		}
+		return error('No Kurzbezeichnung could be generated');
+	}
 
-				$kurzbzexists = $this->kurzbzExists($kurzbz);
+	public function generateKurzbzHelper($vorname, $nachname)
+	{
+		$nachname_clean = sanitizeProblemChars($nachname);
+		$vorname_clean = sanitizeProblemChars($vorname);
+		$kurzbz = '';
 
-				if (hasData($kurzbzexists) && !getData($kurzbzexists)[0])
-					break;
-			}
+		for ($nn = 6, $vn = 2; $nn != 0; $nn--, $vn++)
+		{
+			$kurzbz = mb_substr($nachname_clean, 0, $nn);
+			$kurzbz .= mb_substr($vorname_clean, 0, $vn);
 
 			$kurzbzexists = $this->kurzbzExists($kurzbz);
 
-			if (hasData($kurzbzexists) && getData($kurzbzexists)[0])
-				return error('No Kurzbezeichnung could be generated');
+			if (hasData($kurzbzexists) && !getData($kurzbzexists)[0])
+				break;
 		}
+
+		$kurzbzexists = $this->kurzbzExists($kurzbz);
+
+		if (hasData($kurzbzexists) && getData($kurzbzexists)[0])
+			return error('No Kurzbezeichnung could be generated');
+	
 		return success($kurzbz);
 	}
 }