Bug 41060

Summary: Add a value_builder to check if ISSN is valid
Product: Koha Reporter: Arthur Suzuki <arthur.suzuki>
Component: CatalogingAssignee: Arthur Suzuki <arthur.suzuki>
Status: Failed QA --- QA Contact: Marcel de Rooy <m.de.rooy>
Severity: enhancement    
Priority: P5 - low CC: david, m.de.rooy
Version: Main   
Hardware: All   
OS: All   
See Also: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=29980
GIT URL: Initiative type: ---
Sponsorship status: --- Crowdfunding goal: 0
Patch complexity: Trivial patch Documentation contact:
Documentation submission: Text to go in the release notes:
This enhancement adds a new cataloging plugin (value builder) for validating ISSNs. If you enter an invalid ISSN, a browser pop-up window warns that the ISSN is invalid, but you can ignore this (the field is highlighted in yellow) and still save the record. For new installations, the validate_issn.pl plugin is added to the bibliographic frameworks by default for 022$a (MARC21) or 011$a (UNIMARC). For existing installations, update your frameworks manually.
Version(s) released in:
Circulation function:
Attachments: Bug 41060: Add a cataloguing plugin to check ISSN
Bug 41060: Add a cataloguing plugin to check ISSN

Description Arthur Suzuki 2025-10-21 08:53:58 UTC
same as Bug29980 but for ISSN.
Comment 1 Arthur Suzuki 2025-10-21 08:55:45 UTC
Created attachment 188200 [details] [review]
Bug 41060: Add a cataloguing plugin to check ISSN

Test plan:
- without patch login to Koha
- edit a biblio and put a wrong ISSN in 022a (011a for unimarc).
- leave the input field (looses focus), no warning
- apply patch and do "reset_all" to make a full re-install.
- edit the biblio again and put an invalid ISSN in 022a (011a for unimarc).
- when you leave the input field you should have a warning saying "invalid ISSN".
- the input field should turn yellow
- you can still save the biblio with a wrong issn, it is just a warning.
- if you apply the patch on an existing install, you must manually link the value builder to your existing frameworks to get it working.
Comment 2 Owen Leonard 2025-10-24 16:04:15 UTC
Created attachment 188418 [details] [review]
Bug 41060: Add a cataloguing plugin to check ISSN

Test plan:
- without patch login to Koha
- edit a biblio and put a wrong ISSN in 022a (011a for unimarc).
- leave the input field (looses focus), no warning
- apply patch and do "reset_all" to make a full re-install.
- edit the biblio again and put an invalid ISSN in 022a (011a for unimarc).
- when you leave the input field you should have a warning saying
  "invalid ISSN".
- the input field should turn yellow
- you can still save the biblio with a wrong issn, it is just a warning.
- if you apply the patch on an existing install, you must manually link
  the value builder to your existing frameworks to get it working.

Signed-off-by: Owen Leonard <oleonard@myacpl.org>
Comment 3 Marcel de Rooy 2025-10-31 10:41:27 UTC
Some observations:

If I leave 022 empty, there is a warn about invalid ISSN. Thats weird. Please fix.

If I enter a valid ISSN like 1234-5679, it is ok. If I enter 1234-5679A, it is ok too and that is confusing. If I enter 1234-5679X, it is wrong.
Coming from Business::ISSN. So out of scope ?