Skip to content
Snippets Groups Projects
Commit 2418a31a authored by Guillaume ROUSSE's avatar Guillaume ROUSSE
Browse files

use shorter variable names

parent 50e502de
No related branches found
No related tags found
No related merge requests found
......@@ -462,20 +462,20 @@ sub req_validate_token {
}
## create test accounts
my @test_accounts;
my @accounts;
foreach my $profile (split(/, */, $self->{configuration}->{account_profiles})) {
my $test_account = IdPAccountManager::Data::TestAccount->new(
my $account = IdPAccountManager::Data::TestAccount->new(
db => $self->{db},
account_profile => $profile,
sp_entityid => $self->{in}->{sp_entityid}
);
next unless $test_account;
next unless $test_account->save();
push @test_accounts, $test_account;
next unless $account;
next unless $account->save();
push @accounts, $account;
}
unless (@test_accounts) {
unless (@accounts) {
push @{ $self->{out}->{errors} }, "accounts_creation_failed";
$self->{logger}->errorf(
"Failed to create test accounts for SP with entityid %s",
......@@ -487,7 +487,7 @@ sub req_validate_token {
## Update simpleSAMLphp configuration to enable test accounts
unless (IdPAccountManager::Tools::update_ssp_authsources(
$self->{configuration}->{templates_dir},
$self->{configuration}->{idp_accounts_file}
$self->{configuration}->{idp_accounts_file},
)) {
push @{ $self->{out}->{errors} }, "accounts_creation_failed";
$self->{logger}->error(
......@@ -503,7 +503,7 @@ sub req_validate_token {
);
$self->{out}->{sp_entityid} = $self->{in}->{sp_entityid};
$self->{out}->{test_accounts} = \@test_accounts;
$self->{out}->{test_accounts} = \@accounts;
$self->{out}->{subtitle} = 'Complete Email Challenge';
return 1;
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment