http://bugs.contribs.org/show_bug.cgi?id=8823

--- Comment #14 from Stefano Zamboni <[email protected]> ---
ok.. now it works.

if I choose an user and save, I got the expected result


we still have an issue with the "expire on" field value if password ageing is
not selected..

it should be "never" and not "never 7"

the issue is in the following code:

for my $user ($adb->users)
                        {
                        my $passset = $user->prop('PasswordSet') || 'no';
                        my $star = '';
                        if ( $passset eq 'no' ) {
                        $star = $self->localise('ACCOUNT_LOCKED');
                        }
                        my $key = $user->key;
                        my $cmd = "chage -l $key | grep -i 'password expires' |
cut -d ':' -f 2";
                        my $expiration =  `$cmd 2>/dev/null`;
                        $cmd = "chage -l $key | grep -i 'last' | cut -d ':' -f
2";
                        my $lastchange = `$cmd 2>/dev/null`;
                        my $checked = '';
                        my $passage = $user->prop('PasswordAge') || 'no';
                        if ((( $passage ) || 'no' ) eq 'yes' ) {
                        $checked = ' checked';
                        }
                        my $checkbox = $key . $checked  ;
                        push @data,
                        {
                                User => $user->key,
                                FullName => $user->prop('FirstName') . " " .
$user->prop('LastName') ,
                                Passwordaging => $self->localise(uc($passage))
,
                                Lastchange => $lastchange ,
                                Passwordexpire => $expiration ,
                                Modify => $checkbox,
                                Note => $star
                                }
                        }


line 
my $cmd = "chage -l $key | grep -i 'password expires' | cut -d ':' -f 2";

should be

my $cmd = "chage -l $key | grep 'Password expires' | cut -d ':' -f 2";

in this way it returns only one value, the expected one

-- 
You are receiving this mail because:
You are the QA Contact for the bug.
_______________________________________________
Mail for each SME Contribs bug report
To unsubscribe, e-mail [email protected]
Searchable archive at http://lists.contribs.org/mailman/public/contribteam/

Reply via email to