← Back to team overview

slub.team team mailing list archive

[Merge] lp:~slub.team/goobi-production/bug-1032195 into lp:goobi-production

 

Henning Gerhardt has proposed merging lp:~slub.team/goobi-production/bug-1032195 into lp:goobi-production.

Requested reviews:
  Ralf Claussnitzer (ralf-claussnitzer)
  zeutschel (zeutschel)
Related bugs:
  Bug #1032195 in Goobi.Production: "ldap group is mandatory also if ldap is not used"
  https://bugs.launchpad.net/goobi-production/+bug/1032195

For more details, see:
https://code.launchpad.net/~slub.team/goobi-production/bug-1032195/+merge/118522
-- 
https://code.launchpad.net/~slub.team/goobi-production/bug-1032195/+merge/118522
Your team Saxon State Library Team is subscribed to branch lp:goobi-production.
=== modified file 'newpages/BenutzerBearbeiten.jsp'
--- newpages/BenutzerBearbeiten.jsp	2011-12-20 08:07:09 +0000
+++ newpages/BenutzerBearbeiten.jsp	2012-08-07 10:06:43 +0000
@@ -155,8 +155,12 @@
 												</h:panelGroup>
 
 												<%-- LdapGruppe --%>
-												<h:outputLabel for="ldapGruppe" value="#{msgs.ldapgruppe}" />
-												<h:panelGroup>
+												<h:outputLabel 
+													for="ldapGruppe"
+													value="#{msgs.ldapgruppe}"
+													rendered="#{BenutzerverwaltungForm.ldapUsage}" />
+												<h:panelGroup 
+													rendered="#{BenutzerverwaltungForm.ldapUsage}">
 													<h:selectOneMenu id="ldapGruppe"
 														style="width: 300px;margin-right:15px"
 														value="#{BenutzerverwaltungForm.ldapGruppeAuswahl}"
@@ -169,13 +173,13 @@
 													<x:message for="ldapGruppe" style="color: red"
 														detailFormat="#{msgs.keineLdapgruppeAngegeben}" />
 														
-										<h:commandLink title="#{msgs.ldapKonfigurationSchreiben}"
-											action="#{BenutzerverwaltungForm.LdapKonfigurationSchreiben}"
-											rendered="#{BenutzerverwaltungForm.myClass.id != null}" >
-											<h:graphicImage value="/newpages/images/buttons/key3.gif" />
-											<x:updateActionListener
-												property="#{BenutzerverwaltungForm.myClass}" value="#{BenutzerverwaltungForm.myClass}" />
-										</h:commandLink>	
+													<h:commandLink title="#{msgs.ldapKonfigurationSchreiben}"
+														action="#{BenutzerverwaltungForm.LdapKonfigurationSchreiben}"
+														rendered="#{BenutzerverwaltungForm.myClass.id != null}" >
+														<h:graphicImage value="/newpages/images/buttons/key3.gif" />
+														<x:updateActionListener
+															property="#{BenutzerverwaltungForm.myClass}" value="#{BenutzerverwaltungForm.myClass}" />
+													</h:commandLink>
 												</h:panelGroup>
 
 												<%-- MetadatenSprache --%>

=== modified file 'src/de/sub/goobi/forms/BenutzerverwaltungForm.java'
--- src/de/sub/goobi/forms/BenutzerverwaltungForm.java	2012-04-24 13:22:01 +0000
+++ src/de/sub/goobi/forms/BenutzerverwaltungForm.java	2012-08-07 10:06:43 +0000
@@ -39,6 +39,7 @@
 import javax.faces.model.SelectItem;
 import javax.servlet.http.HttpSession;
 
+import de.sub.goobi.config.ConfigMain;
 import dubious.sub.goobi.helper.Page;
 import org.apache.log4j.Logger;
 import org.hibernate.Criteria;
@@ -67,7 +68,7 @@
 	private BenutzerDAO dao = new BenutzerDAO();
 	private boolean hideInactiveUsers = true;
 	private static final Logger logger = Logger.getLogger(BenutzerverwaltungForm.class);
-	
+
 	public String Neu() {
 		myClass = new Benutzer();
 		myClass.setVorname("");
@@ -335,4 +336,8 @@
 		this.hideInactiveUsers = hideInactiveUsers;
 	}
 
+	public boolean getLdapUsage() {
+		return ConfigMain.getBooleanParameter("ldap_use");
+	}
+
 }


Follow ups