From bec2074810b6ad6a7b61044ed6b6bf7941ca341e Mon Sep 17 00:00:00 2001
From: Michael Dollinger <michael.dollinger@noreply.%(DOMAIN)s>
Date: Fri, 22 Jan 2021 20:20:34 +0100
Subject: [PATCH] =?UTF-8?q?=E2=80=9Esrc/main/webapp/group.jsp=E2=80=9C=20?=
 =?UTF-8?q?=C3=A4ndern?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Die lästige Befüllung von Handy und Passwort habe ich mit autocomplete="new-password" unterbinden können
Die Gruppenliste zeigt jetzt zuerst die zugewiesenen Gruppen (Haken gesetzt) an, und danach alle anderen

Anmelde-Kennung und Name getrennt (eigene Spalte)
Anmelde-Kennung verlinkt - man kann direkt zum Account klicken
und dadurch stimmt jetzt die alphabetische Reihenfolge
---
 src/main/webapp/group.jsp | 42 +++++++++++++++++++++++++++++++++------
 1 file changed, 36 insertions(+), 6 deletions(-)

diff --git a/src/main/webapp/group.jsp b/src/main/webapp/group.jsp
index 6de5118..5e898c7 100644
--- a/src/main/webapp/group.jsp
+++ b/src/main/webapp/group.jsp
@@ -48,14 +48,44 @@
 					<div class="form-group">
 						<label for="membership" class="col-sm-3 control-label"><fmt:message key="groups.label.members"/></label> 
 						<div class="col-sm-9" id ="membership">
-							<c:forEach items="${users}" var="userentry" >
-								<div class="checkbox">
-									<jsp:useBean id="checker" class="de.jalin.ldapadmin.beans.MembershipCheck" />
+							<jsp:useBean id="checker" class="de.jalin.ldapadmin.beans.MembershipCheck" />
+							<table>
+								<c:forEach items="${users}" var="userentry" >
 									<jsp:setProperty property="user" name="checker" value="${userentry.value}" />
 									<jsp:setProperty property="group" name="checker" value="${group}" />
-									<label><input type="checkbox" id="${userentry.value.login}" name="check_user_${userentry.value.login}" ${checker.checked}>${userentry.value.firstname} ${userentry.value.lastname} (${userentry.value.login})</label>
-								</div>
-							</c:forEach>
+									<c:url var="userurl" value="/user/${userentry.key}" />
+									<c:if test="${ not empty checker.checked }">
+										<tr>
+											<td style="padding: 0px 30px 0px 0px">
+												<input type="checkbox" id="${userentry.value.login}" name="check_user_${userentry.value.login}" ${checker.checked}>
+												<a href="${userurl}">${userentry.value.login}</a>
+											</td>
+											<td style="padding: 0px 10px 0px 0px">${userentry.value.firstname}</td>
+											<td>${userentry.value.lastname}</td>
+										</tr>
+									</c:if>
+								</c:forEach>
+							</table>
+							<div class="checkbox">
+								------------------------------------------------------------------------------------------------------------------------------------------
+							</div>
+							<table>
+								<c:forEach items="${users}" var="userentry" >
+									<jsp:setProperty property="user" name="checker" value="${userentry.value}" />
+									<jsp:setProperty property="group" name="checker" value="${group}" />
+									<c:url var="userurl" value="/user/${userentry.key}" />
+									<c:if test="${ empty checker.checked }">
+										<tr>
+											<td style="padding: 0px 30px 0px 0px">
+												<input type="checkbox" id="${userentry.value.login}" name="check_user_${userentry.value.login}" ${checker.checked}>
+												<a href="${userurl}">${userentry.value.login}</a>
+											</td>
+											<td style="padding: 0px 10px 0px 0px">${userentry.value.firstname}</td>
+											<td>${userentry.value.lastname}</td>
+										</tr>
+									</c:if>
+								</c:forEach>
+							</table>
 						</div>
 					</div>
 					<c:if test="${ formdisabled ne 'disabled' }">