From 520e9208847ae259cc886301cf754b0a12928d4f Mon Sep 17 00:00:00 2001 From: Marcel de Rooy Date: Fri, 16 Jan 2026 09:30:14 +0100 Subject: [PATCH] Bug 41060: (QA follow-up) Check in change, trigger once in Blur Content-Type: text/plain; charset=utf-8 If we already checked and nothing changes, no need to check again. Set focus to field if we warn. For consistency copied these changes to validate_isbn. Test plan: Try various valid and invalid ISSNs. Tab through the field repeatedly; you should only have a warning once, if the ISSN is invalid (yellow colored). Save a bad ISSN into a record. Open it again and tab through ISSN; you should be warned. Repeat similarly for ISBN field. Signed-off-by: Marcel de Rooy --- cataloguing/value_builder/validate_isbn.pl | 14 ++++++++++---- cataloguing/value_builder/validate_issn.pl | 14 ++++++++++---- 2 files changed, 20 insertions(+), 8 deletions(-) diff --git a/cataloguing/value_builder/validate_isbn.pl b/cataloguing/value_builder/validate_isbn.pl index 5e51a49576..4bc977e239 100755 --- a/cataloguing/value_builder/validate_isbn.pl +++ b/cataloguing/value_builder/validate_isbn.pl @@ -30,20 +30,26 @@ my $builder = sub { return qq| |; }; diff --git a/cataloguing/value_builder/validate_issn.pl b/cataloguing/value_builder/validate_issn.pl index 164af3ef16..ad712bc572 100755 --- a/cataloguing/value_builder/validate_issn.pl +++ b/cataloguing/value_builder/validate_issn.pl @@ -30,20 +30,26 @@ my $builder = sub { return qq| |; }; -- 2.39.5