https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=39014 --- Comment #32 from Marcel de Rooy <m.de.rooy@rijksmuseum.nl> --- Just an observation: members/memberentry.pl: push @errors, "ERROR_guarantor_is_guarantee"; if ( $_->invalid_guarantor ) { push @errors, "ERROR_guarantor_is_guarantee"; $template->param( guarantor_is_guarantee => $_->guarantor ); => Does this code not beg for renaming invalid_guarantor to something like guarantor_is_guarantee ? Koha/Exceptions/Patron/Relationship.pm: } elsif ( $self->invalid_guarantor ) { } elsif ( $self->invalid_guarantor ) { $msg = sprintf("Guarantee patron cannot be a guarantor."); => [SAME] Does this code not beg for renaming invalid_guarantor to something like guarantor_is_guarantee ? -- You are receiving this mail because: You are watching all bug changes.