From e38d0709b76db85838183ddbdd7b23f5209b5455 Mon Sep 17 00:00:00 2001
From: ralf <rb@egroupware.org>
Date: Wed, 10 Aug 2022 15:44:34 +0200
Subject: [PATCH] fix <description /> causes all sorts of problems (by
 replacing it separate) for <select rows="N" also switch on multiple, as old
 eTemplate2 did fixes tracker site configuration did not allow to select more
 than one staff member

---
 api/etemplate.php | 6 ++++++
 1 file changed, 6 insertions(+)

diff --git a/api/etemplate.php b/api/etemplate.php
index 8301dc0fe3..59e5377884 100644
--- a/api/etemplate.php
+++ b/api/etemplate.php
@@ -176,6 +176,7 @@ function send_template()
 			}, $str);
 
 		// replace just description, as they often contain >, like label="> %s"
+		$str = preg_replace('#<description\s*/>#', '<et2-description></et2-description>', $str);
 		$str = preg_replace('#<description\s(.*?")\s*/>#s', '<et2-description $1></et2-description>', $str);
 
 		// modify <(vfs-mime|link-string|link-list) --> <et2-*
@@ -246,6 +247,11 @@ function send_template()
 				$attrs['multiple'] = 'true';
 				unset($attrs['expand_multiple_rows']);
 			}
+			// <select rows="N" (to show N rows) previously also switched multiple on
+			if (!empty($attrs['rows']))
+			{
+				$attrs['multiple'] = true;
+			}
 			// automatic convert empty_label for multiple=true to a placeholder
 			if (!empty($attrs['empty_label']) && !empty($attrs['multiple']))
 			{