From c502ab95009fc5814f1b96b45c6503313551578d Mon Sep 17 00:00:00 2001 From: Lauri Ojansivu Date: Mon, 24 Dec 2018 18:18:41 +0200 Subject: - Revert "Improve authentication" and "Default Authentication Method" to make login work again. - Fixes to docker-compose.yml so that Wekan Meteor 1.6.x version would work. Most likely Meteor 1.8.x version is still broken. Thanks to xet7 ! --- client/components/main/layouts.js | 109 +++++++++++++++++--------------------- 1 file changed, 49 insertions(+), 60 deletions(-) (limited to 'client/components/main/layouts.js') diff --git a/client/components/main/layouts.js b/client/components/main/layouts.js index 89dcca2d..a50d167e 100644 --- a/client/components/main/layouts.js +++ b/client/components/main/layouts.js @@ -6,14 +6,29 @@ const i18nTagToT9n = (i18nTag) => { return i18nTag; }; -Template.userFormsLayout.onCreated(function() { - Meteor.call('getDefaultAuthenticationMethod', (error, result) => { - this.data.defaultAuthenticationMethod = new ReactiveVar(error ? undefined : result); - }); +const validator = { + set(obj, prop, value) { + if (prop === 'state' && value !== 'signIn') { + $('.at-form-authentication').hide(); + } else if (prop === 'state' && value === 'signIn') { + $('.at-form-authentication').show(); + } + // The default behavior to store the value + obj[prop] = value; + // Indicate success + return true; + }, +}; + +Template.userFormsLayout.onCreated(() => { Meteor.subscribe('setting'); + }); Template.userFormsLayout.onRendered(() => { + + AccountsTemplates.state.form.keys = new Proxy(AccountsTemplates.state.form.keys, validator); + const i18nTag = navigator.language; if (i18nTag) { T9n.setLanguage(i18nTagToT9n(i18nTag)); @@ -86,11 +101,13 @@ Template.userFormsLayout.events({ } }); }, - 'click #at-btn'(event, instance) { - const email = $('#at-field-username_and_email').val(); - const password = $('#at-field-password').val(); - - if (FlowRouter.getRouteName() !== 'atSignIn' || password === '' || email === '') { + 'click #at-btn'(event) { + /* All authentication method can be managed/called here. + !! DON'T FORGET to correctly fill the fields of the user during its creation if necessary authenticationMethod : String !! + */ + const authenticationMethodSelected = $('.select-authentication').val(); + // Local account + if (authenticationMethodSelected === 'password') { return; } @@ -98,11 +115,29 @@ Template.userFormsLayout.events({ event.preventDefault(); event.stopImmediatePropagation(); - Meteor.subscribe('user-authenticationMethod', email, { - onReady() { - return authentication.call(this, instance, email, password); - }, - }); + const email = $('#at-field-username_and_email').val(); + const password = $('#at-field-password').val(); + + // Ldap account + if (authenticationMethodSelected === 'ldap') { + // Check if the user can use the ldap connection + Meteor.subscribe('user-authenticationMethod', email, { + onReady() { + const user = Users.findOne(); + if (user === undefined || user.authenticationMethod === 'ldap') { + // Use the ldap connection package + Meteor.loginWithLDAP(email, password, function(error) { + if (!error) { + // Connection + return FlowRouter.go('/'); + } + return error; + }); + } + return this.stop(); + }, + }); + } }, }); @@ -111,49 +146,3 @@ Template.defaultLayout.events({ Modal.close(); }, }); - -function authentication(instance, email, password) { - const user = Users.findOne(); - - // Authentication with password - if (user && user.authenticationMethod === 'password') { - $('#at-pwd-form').submit(); - return this.stop(); - } - - const authenticationMethod = user - ? user.authenticationMethod - : instance.data.defaultAuthenticationMethod.get(); - - switch (authenticationMethod) { - case 'ldap': - // Use the ldap connection package - Meteor.loginWithLDAP(email, password, function(error) { - if (error) { - displayError('error-ldap-login'); - return this.stop(); - } else { - return FlowRouter.go('/'); - } - }); - break; - - default: - displayError('error-undefined'); - } - - return this.stop(); -} - -function displayError(code) { - const translated = TAPi18n.__(code); - - if (translated === code) { - return; - } - - if(!$('.at-error').length) { - $('.at-pwd-form').before('

'); - } - $('.at-error p').text(translated); -} -- cgit v1.2.3-1-g7c22