From e44751b3f136e7a438b1379a8d865bdff6cf235e Mon Sep 17 00:00:00 2001 From: Sheogorath Date: Fri, 26 Jan 2018 10:42:06 +0100 Subject: [PATCH] Fix ldap provider name in template Before this fix it's impossible to set the provider name in the sign-model since `ldap` is a boolean there and this way not able to have an attribute like `ldap.providerName`. Signed-off-by: Sheogorath --- lib/response.js | 2 ++ public/views/shared/signin-modal.ejs | 2 +- 2 files changed, 3 insertions(+), 1 deletion(-) diff --git a/lib/response.js b/lib/response.js index 445aa0d78..c874ce9c6 100644 --- a/lib/response.js +++ b/lib/response.js @@ -69,6 +69,7 @@ function showIndex (req, res, next) { dropbox: config.isDropboxEnable, google: config.isGoogleEnable, ldap: config.isLDAPEnable, + ldapProviderName: config.ldap.providerName, saml: config.isSAMLEnable, email: config.isEmailEnable, allowemailregister: config.allowemailregister, @@ -103,6 +104,7 @@ function responseHackMD (res, note) { dropbox: config.isDropboxEnable, google: config.isGoogleEnable, ldap: config.isLDAPEnable, + ldapProviderName: config.ldap.providerName, saml: config.isSAMLEnable, email: config.isEmailEnable, allowemailregister: config.allowemailregister, diff --git a/public/views/shared/signin-modal.ejs b/public/views/shared/signin-modal.ejs index 7b44cfb0a..ce01c7399 100644 --- a/public/views/shared/signin-modal.ejs +++ b/public/views/shared/signin-modal.ejs @@ -52,7 +52,7 @@
<% }%> <% if(ldap) { %> -

Via <% if (ldap.providerName) { %> <%- ldap.providerName %> (LDAP) <% } else { %> LDAP <% } %>

+

Via <% if (ldapProviderName) { %> <%= ldapProviderName %> (LDAP) <% } else { %> LDAP <% } %>