| Summary: | Add a value_builder to check if ISSN is valid | ||
|---|---|---|---|
| Product: | Koha | Reporter: | Arthur Suzuki <arthur.suzuki> |
| Component: | Cataloging | Assignee: | 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
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. 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> 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 ? |