Keep old behavior if no JS (#1976)
This commit is contained in:
parent
6428cd0a0a
commit
55071d5210
|
@ -483,11 +483,16 @@ sub display {
|
||||||
$self->loadedModules->{
|
$self->loadedModules->{
|
||||||
"Lemonldap::NG::Portal::Plugins::FindUser"};
|
"Lemonldap::NG::Portal::Plugins::FindUser"};
|
||||||
my $fields = [];
|
my $fields = [];
|
||||||
$fields = $plugin->buildForm()
|
if (
|
||||||
if ( $plugin
|
$plugin
|
||||||
&& $self->conf->{findUser}
|
&& $self->conf->{findUser}
|
||||||
&& $self->conf->{impersonationRule}
|
&& $self->conf->{impersonationRule}
|
||||||
&& $self->conf->{findUserSearchingAttributes} );
|
&& $self->conf->{findUserSearchingAttributes}
|
||||||
|
)
|
||||||
|
{
|
||||||
|
$login = $req->data->{findUser};
|
||||||
|
$fields = $plugin->buildForm();
|
||||||
|
}
|
||||||
|
|
||||||
# Authentication loop
|
# Authentication loop
|
||||||
if ( $self->conf->{authentication} eq 'Choice'
|
if ( $self->conf->{authentication} eq 'Choice'
|
||||||
|
@ -503,6 +508,7 @@ sub display {
|
||||||
DISPLAY_OPENID_FORM => 0,
|
DISPLAY_OPENID_FORM => 0,
|
||||||
DISPLAY_YUBIKEY_FORM => 0,
|
DISPLAY_YUBIKEY_FORM => 0,
|
||||||
FIELDS => $fields,
|
FIELDS => $fields,
|
||||||
|
SPOOFID => $login,
|
||||||
FINDUSER => scalar @$fields
|
FINDUSER => scalar @$fields
|
||||||
);
|
);
|
||||||
}
|
}
|
||||||
|
@ -536,6 +542,7 @@ sub display {
|
||||||
( $displayType eq "logo" ? $self->conf->{portal} : 0 ),
|
( $displayType eq "logo" ? $self->conf->{portal} : 0 ),
|
||||||
MSG => $req->info(),
|
MSG => $req->info(),
|
||||||
FIELDS => $fields,
|
FIELDS => $fields,
|
||||||
|
SPOOFID => $login,
|
||||||
FINDUSER => scalar @$fields
|
FINDUSER => scalar @$fields
|
||||||
);
|
);
|
||||||
}
|
}
|
||||||
|
|
|
@ -8,7 +8,7 @@ require 't/test-lib.pm';
|
||||||
my $res;
|
my $res;
|
||||||
my $json;
|
my $json;
|
||||||
my $request;
|
my $request;
|
||||||
my $maintests = 23;
|
my $maintests = 24;
|
||||||
|
|
||||||
SKIP: {
|
SKIP: {
|
||||||
skip 'LLNGTESTLDAP is not set', $maintests unless ( $ENV{LLNGTESTLDAP} );
|
skip 'LLNGTESTLDAP is not set', $maintests unless ( $ENV{LLNGTESTLDAP} );
|
||||||
|
@ -70,6 +70,8 @@ m%<input id="spoofIdfield" name="spoofId" type="text" class="form-control" value
|
||||||
),
|
),
|
||||||
'Post FindUser request'
|
'Post FindUser request'
|
||||||
);
|
);
|
||||||
|
ok( $res->[2]->[0] =~ m%value="dwho"%, 'value="dwho"' )
|
||||||
|
or explain( $res->[2]->[0], 'value="dwho"' );
|
||||||
ok( $res->[2]->[0] =~ m%autocomplete="off"%, 'autocomplete="off"' )
|
ok( $res->[2]->[0] =~ m%autocomplete="off"%, 'autocomplete="off"' )
|
||||||
or explain( $res->[2]->[0], 'autocomplete="off"' );
|
or explain( $res->[2]->[0], 'autocomplete="off"' );
|
||||||
ok(
|
ok(
|
||||||
|
|
|
@ -5,7 +5,7 @@ use IO::String;
|
||||||
|
|
||||||
require 't/test-lib.pm';
|
require 't/test-lib.pm';
|
||||||
|
|
||||||
my $maintests = 53;
|
my $maintests = 54;
|
||||||
my $userdb = tempdb();
|
my $userdb = tempdb();
|
||||||
|
|
||||||
SKIP: {
|
SKIP: {
|
||||||
|
@ -105,6 +105,8 @@ m%<input id="spoofIdfield" name="spoofId" type="text" class="form-control" value
|
||||||
),
|
),
|
||||||
'Post FindUser request'
|
'Post FindUser request'
|
||||||
);
|
);
|
||||||
|
ok( $res->[2]->[0] =~ m%value="dwho"%, 'value="dwho"' )
|
||||||
|
or explain( $res->[2]->[0], 'value="dwho"' );
|
||||||
ok( $res->[2]->[0] =~ m%autocomplete="off"%, 'autocomplete="off"' )
|
ok( $res->[2]->[0] =~ m%autocomplete="off"%, 'autocomplete="off"' )
|
||||||
or explain( $res->[2]->[0], 'autocomplete="off"' );
|
or explain( $res->[2]->[0], 'autocomplete="off"' );
|
||||||
ok(
|
ok(
|
||||||
|
|
Loading…
Reference in New Issue
Block a user