X-Git-Url: http://wagnertech.de/git?a=blobdiff_plain;f=SL%2FController%2FLoginScreen.pm;h=da2c0bfffcaac5322146432d1fa7602c9b9e7f1b;hb=10d6fe63ec82e50322a1fe08902fde7a0c1efa4e;hp=e84a31505ddb1b1cf80b12a0a3c1bbee6228283d;hpb=aa9c28b7cfe1fe29eb6ca1d9922e913f22891149;p=kivitendo-erp.git diff --git a/SL/Controller/LoginScreen.pm b/SL/Controller/LoginScreen.pm index e84a31505..da2c0bfff 100644 --- a/SL/Controller/LoginScreen.pm +++ b/SL/Controller/LoginScreen.pm @@ -56,7 +56,6 @@ sub action_login { } %::myconfig = $login ? $::auth->read_user(login => $login) : (); - $::form->{login} = $login; $::locale = Locale->new($::myconfig{countrycode}) if $::myconfig{countrycode}; SL::Dispatcher::AuthHandler::User->new->handle(countrycode => $::myconfig{countrycode}); @@ -68,7 +67,8 @@ sub action_login { # Auth DB needs update? If so log the user out forcefully. if (User::LOGIN_AUTH_DBUPDATE_AVAILABLE() == $result) { $::auth->destroy_session; - return $self->render('login_screen/auth_db_needs_update'); + # must be without layout because menu rights might not exist yet + return $self->render('login_screen/auth_db_needs_update', { layout => 0 }); } # Basic client tables available? If not tell the user to create them @@ -111,8 +111,6 @@ sub keep_auth_vars_in_form { sub _redirect_to_main_script { my ($self) = @_; - $self->_ensure_employees_for_authorized_users_exist; - return $self->redirect_to($::form->{callback}) if $::form->{callback}; $self->redirect_to(controller => "login.pl", action => 'company_logo'); @@ -153,27 +151,11 @@ sub _redirect_to_main_script_if_already_logged_in { return 1; } -sub _ensure_employees_for_authorized_users_exist { - my ($self) = @_; - - my %employees_by_login = map { ($_->login => $_) } @{ SL::DB::Manager::Employee->get_all }; - - foreach my $user (@{ SL::DB::AuthClient->new(id => $::auth->client->{id})->load->users || [] }) { - my $user_config = $user->config_values; - my $employee = $employees_by_login{$user->login} || SL::DB::Employee->new(login => $user->login); - - $employee->update_attributes( - name => $user_config->{name}, - workphone => $user_config->{tel}, - deleted => 0, - ); - } -} - sub error_state { my %states = ( session => { warning => t8('The session has expired. Please log in again.') }, password => { error => t8('Incorrect username or password or no access to selected client!') }, + action => { warning => t8('The action is missing or invalid.') }, ); return %{ $states{$_[0]} || {} };