Currently, we aren't entirely consistent with setting the replacementpricedate field whenever replacementprice is updated.. we should add it to the list of fields handled in the Koha::Item store triggers.
Created attachment 126037 [details] [review] Bug 29202: Trigger replacementpricedate on replacementprice change
my @fields = qw( itemlost withdrawn damaged replacementprice ); for my $field (@fields) { [...] my $field_on = "${field}_on"; $self->$field_on(undef); This will not work for replacementprice ;) No _on there Btw I have my doubts here if the code does exactly what we want here. We only want a updated timestamp if the value actually changes. Is the finishreceive script the only passing this value? We need some tests !
I've decided this is invalid.. :-)