Author: solomax
Date: Fri Apr 4 01:59:39 2014
New Revision: 1584503
URL: http://svn.apache.org/r1584503
Log:
[OPENMEETINGS-972] ldap_default_id is taken into account while building ldaps
drop down
Modified:
openmeetings/branches/3.0.x/src/install/java/org/apache/openmeetings/installation/ImportInitvalues.java
openmeetings/branches/3.0.x/src/util/java/org/apache/openmeetings/util/OpenmeetingsVariables.java
openmeetings/branches/3.0.x/src/web/java/org/apache/openmeetings/web/pages/auth/SignInDialog.java
Modified:
openmeetings/branches/3.0.x/src/install/java/org/apache/openmeetings/installation/ImportInitvalues.java
URL:
http://svn.apache.org/viewvc/openmeetings/branches/3.0.x/src/install/java/org/apache/openmeetings/installation/ImportInitvalues.java?rev=1584503&r1=1584502&r2=1584503&view=diff
==============================================================================
---
openmeetings/branches/3.0.x/src/install/java/org/apache/openmeetings/installation/ImportInitvalues.java
(original)
+++
openmeetings/branches/3.0.x/src/install/java/org/apache/openmeetings/installation/ImportInitvalues.java
Fri Apr 4 01:59:39 2014
@@ -23,6 +23,7 @@ import static org.apache.openmeetings.ut
import static
org.apache.openmeetings.util.OpenmeetingsVariables.CONFIG_DASHBOARD_SHOW_MYROOMS_KEY;
import static
org.apache.openmeetings.util.OpenmeetingsVariables.CONFIG_DASHBOARD_SHOW_RSS_KEY;
import static
org.apache.openmeetings.util.OpenmeetingsVariables.CONFIG_DEFAULT_LANDING_ZONE;
+import static
org.apache.openmeetings.util.OpenmeetingsVariables.CONFIG_DEFAULT_LDAP_ID;
import static
org.apache.openmeetings.util.OpenmeetingsVariables.CONFIG_DEFAUT_LANG_KEY;
import static
org.apache.openmeetings.util.OpenmeetingsVariables.CONFIG_FRONTEND_REGISTER_KEY;
import static
org.apache.openmeetings.util.OpenmeetingsVariables.CONFIG_IGNORE_BAD_SSL;
@@ -513,7 +514,7 @@ public class ImportInitvalues {
"A hot key code for the
'mute/unmute audio' functionality. Keycode 118 is F7");
// system-wide ldap params
- configurationDao.add("ldap_default_id", "0", null,
+ configurationDao.add(CONFIG_DEFAULT_LDAP_ID, "0", null,
"Ldap domain selected by default in the login
screen");
// set inviter's email address as ReplyTo in email invitations
Modified:
openmeetings/branches/3.0.x/src/util/java/org/apache/openmeetings/util/OpenmeetingsVariables.java
URL:
http://svn.apache.org/viewvc/openmeetings/branches/3.0.x/src/util/java/org/apache/openmeetings/util/OpenmeetingsVariables.java?rev=1584503&r1=1584502&r2=1584503&view=diff
==============================================================================
---
openmeetings/branches/3.0.x/src/util/java/org/apache/openmeetings/util/OpenmeetingsVariables.java
(original)
+++
openmeetings/branches/3.0.x/src/util/java/org/apache/openmeetings/util/OpenmeetingsVariables.java
Fri Apr 4 01:59:39 2014
@@ -32,6 +32,7 @@ public class OpenmeetingsVariables {
public static final String CONFIG_RSS_FEED2_KEY = "rss_feed2";
public static final String CONFIG_IGNORE_BAD_SSL =
"oauth2.ignore_bad_ssl";
public static final String CONFIG_DEFAULT_LANDING_ZONE =
"default.landing.zone";
+ public static final String CONFIG_DEFAULT_LDAP_ID = "ldap_default_id";
public static final String WEB_DATE_PATTERN = "dd.MM.yyyy HH:mm:ss";
//FIXME need to be made locale based
public static final String CONFIG_REDIRECT_URL_FOR_EXTERNAL_KEY =
"redirect.url.for.external.users";
public static String webAppRootKey = null;
Modified:
openmeetings/branches/3.0.x/src/web/java/org/apache/openmeetings/web/pages/auth/SignInDialog.java
URL:
http://svn.apache.org/viewvc/openmeetings/branches/3.0.x/src/web/java/org/apache/openmeetings/web/pages/auth/SignInDialog.java?rev=1584503&r1=1584502&r2=1584503&view=diff
==============================================================================
---
openmeetings/branches/3.0.x/src/web/java/org/apache/openmeetings/web/pages/auth/SignInDialog.java
(original)
+++
openmeetings/branches/3.0.x/src/web/java/org/apache/openmeetings/web/pages/auth/SignInDialog.java
Fri Apr 4 01:59:39 2014
@@ -18,6 +18,7 @@
*/
package org.apache.openmeetings.web.pages.auth;
+import static
org.apache.openmeetings.util.OpenmeetingsVariables.CONFIG_DEFAULT_LDAP_ID;
import static
org.apache.openmeetings.web.app.Application.getAuthenticationStrategy;
import static org.apache.openmeetings.web.app.Application.getBean;
import static
org.apache.openmeetings.web.pages.auth.SignInPage.allowOAuthLogin;
@@ -26,6 +27,7 @@ import static org.apache.openmeetings.we
import java.util.ArrayList;
import java.util.List;
+import org.apache.openmeetings.db.dao.basic.ConfigurationDao;
import org.apache.openmeetings.db.dao.basic.ErrorDao;
import org.apache.openmeetings.db.dao.server.LdapConfigDao;
import org.apache.openmeetings.db.dao.server.OAuth2Dao;
@@ -224,7 +226,8 @@ public class SignInDialog extends Abstra
add(passField = new PasswordTextField("pass", new
PropertyModel<String>(SignInDialog.this, "password")).setResetPassword(true));
passField.setLabel(Model.of(WebSession.getString(115)));
List<LdapConfig> ldaps =
getBean(LdapConfigDao.class).getLdapConfigs();
- domain = ldaps.get(0);
+ int selectedLdap =
getBean(ConfigurationDao.class).getConfValue(CONFIG_DEFAULT_LDAP_ID,
Integer.class, "0");
+ domain = ldaps.get(selectedLdap < ldaps.size() &&
selectedLdap > 0 ? selectedLdap : 0);
add(new WebMarkupContainer("ldap")
.add(new DropDownChoice<LdapConfig>("domain",
new PropertyModel<LdapConfig>(SignInDialog.this, "domain")
, ldaps, new
ChoiceRenderer<LdapConfig>("name", "ldapConfigId"))).setVisible(ldaps.size() >
1));
