Bug 21596: (follow-up) Handle updated_on

Default value should only used if not exist.
We got a failing test with the previous patch:
 #   Failed test 'borrowers.updated_on should have been kept to what we set on creating'
 #   at t/db_dependent/Patrons.t line 78.
 #          got: '1'
 #     expected: '0'

Test plan:
prove -r t/Auth_with_shibboleth.t t/db_dependent/Patrons.t t/db_dependent/Koha/Patron*

Signed-off-by: Nick Clemens <nick@bywatersolutions.com>
This commit is contained in:
Jonathan Druart 2018-10-25 10:24:46 -03:00 committed by Nick Clemens
parent 1ce984ea1d
commit d76fde9467

View file

@ -200,7 +200,7 @@ sub store {
if ( defined $self->updated_on and not $self->updated_on ) {
$self->updated_on(undef);
} else {
} elsif ( not $self->updated_on ) {
# This is bad we should use columns_info instead
# But it will avoid unecessary processing
$self->updated_on(\"current_timestamp");