Bug 30430 - UNIMARC XSLT : displaying field B_214
Summary: UNIMARC XSLT : displaying field B_214
Status: Signed Off
Alias: None
Product: Koha
Classification: Unclassified
Component: MARC Bibliographic data support (show other bugs)
Version: unspecified
Hardware: All All
: P5 - low enhancement (vote)
Assignee: François Pichenot
QA Contact: Testopia
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2022-04-01 09:37 UTC by François Pichenot
Modified: 2022-05-11 09:59 UTC (History)
4 users (show)

See Also:
Change sponsored?: ---
Patch complexity: Medium patch
Documentation contact:
Documentation submission:
Text to go in the release notes:
Version(s) released in:


Attachments
Bug 30430: UNIMARC XSLT add field B214 display (156.22 KB, patch)
2022-04-01 11:33 UTC, Arthur Suzuki
Details | Diff | Splinter Review
Bug 30430: UNIMARC XSLT add field B214 display (96.88 KB, patch)
2022-04-01 12:53 UTC, François Pichenot
Details | Diff | Splinter Review
Bug 30430: UNIMARC XSLT add field B214 display (96.79 KB, patch)
2022-04-01 19:33 UTC, Arthur Suzuki
Details | Diff | Splinter Review
Bug 30430: UNIMARC XSLT add field B214 display (5.98 KB, patch)
2022-04-13 11:32 UTC, François Pichenot
Details | Diff | Splinter Review
Bug 30430: UNIMARC XSLT add field B214 display (13.64 KB, patch)
2022-05-03 21:40 UTC, Arthur Suzuki
Details | Diff | Splinter Review
Bug 30430 : (QA follow-up) fix display of 210 only if no 214 + added 214r/s (7.70 KB, patch)
2022-05-05 15:33 UTC, Arthur Suzuki
Details | Diff | Splinter Review
example record (125.00 KB, application/xml)
2022-05-05 15:35 UTC, Arthur Suzuki
Details
Bug 30430: UNIMARC XSLT add field B214 display (13.69 KB, patch)
2022-05-05 23:09 UTC, David Nind
Details | Diff | Splinter Review
Bug 30430 : (QA follow-up) fix display of 210 only if no 214 + added 214r/s (7.74 KB, patch)
2022-05-05 23:09 UTC, David Nind
Details | Diff | Splinter Review

Note You need to log in before you can comment on or make changes to this bug.
Description François Pichenot 2022-04-01 09:37:11 UTC
We need to update XSLT files to display new UNIMARC B_214 field.
See https://cdn.ifla.org/wp-content/uploads/files/assets/uca/unimarc_updates/BIBLIOGRAPHIC/u_b_214_update2019_online_final.pdf
Comment 1 Arthur Suzuki 2022-04-01 11:33:16 UTC
Created attachment 132842 [details] [review]
Bug 30430: UNIMARC XSLT add field B214 display

test plan :
- Apply patch
- Find a record with a B214
- Check on both Opac-Details and Opac-Result that the field is displayed.
- Do the same checks on catalogue/search results and catalogue/details
in Staff interface.
Comment 2 François Pichenot 2022-04-01 12:53:01 UTC
Created attachment 132864 [details] [review]
Bug 30430: UNIMARC XSLT add field B214 display

test plan :
- Apply patch
- Find a record with a B214
- Check on both Opac-Details and Opac-Result that the field is displayed.
- Do the same checks on catalogue/search results and catalogue/details
in Staff interface.
Comment 3 Arthur Suzuki 2022-04-01 19:33:59 UTC
Created attachment 132872 [details] [review]
Bug 30430: UNIMARC XSLT add field B214 display

test plan :
- Apply patch
- Find a record with a B214
- Check on both Opac-Details and Opac-Result that the field is displayed.
- Do the same checks on catalogue/search results and catalogue/details
in Staff interface.
Comment 4 François Pichenot 2022-04-13 11:32:03 UTC
Created attachment 133235 [details] [review]
Bug 30430: UNIMARC XSLT add field B214 display

test plan :
- Apply patch
- Find a record with a B214
- Check on both Opac-Details and Opac-Result that the field is
  displayed.
- Do the same checks on catalogue/search results and catalogue/details
  in Staff interface.
Comment 5 Arthur Suzuki 2022-05-03 21:40:51 UTC
Created attachment 134560 [details] [review]
Bug 30430: UNIMARC XSLT add field B214 display

test plan :
- Apply patch
- Find a record with a B214
- Check on both Opac-Details and Opac-Result that the field is
  displayed.
- Do the same checks on catalogue/search results and catalogue/details
  in Staff interface.
- Add a 210 field to the record (legacy)
- It should be displayed as well
Comment 6 Arthur Suzuki 2022-05-03 22:13:30 UTC
rebasé sur master + affichage du 210 (ancien) en plus du 214 (pour rétrocompatibilité)
Comment 7 Arthur Suzuki 2022-05-05 15:33:57 UTC
Created attachment 134652 [details] [review]
Bug 30430 : (QA follow-up) fix display of 210 only if no 214 + added 214r/s
Comment 8 Arthur Suzuki 2022-05-05 15:35:03 UTC
Created attachment 134653 [details]
example record
Comment 9 David Nind 2022-05-05 23:09:49 UTC
Created attachment 134667 [details] [review]
Bug 30430: UNIMARC XSLT add field B214 display

test plan :
- Apply patch
- Find a record with a B214
- Check on both Opac-Details and Opac-Result that the field is
  displayed.
- Do the same checks on catalogue/search results and catalogue/details
  in Staff interface.
- Add a 210 field to the record (legacy)
- It should be displayed as well

Signed-off-by: David Nind <david@davidnind.com>
Comment 10 David Nind 2022-05-05 23:09:55 UTC
Created attachment 134668 [details] [review]
Bug 30430 : (QA follow-up) fix display of 210 only if no 214 + added 214r/s

Signed-off-by: David Nind <david@davidnind.com>
Comment 11 David Nind 2022-05-05 23:10:47 UTC
Thanks Arthur for providing a sample record - that made it much easier to test!
Comment 12 Katrin Fischer 2022-05-06 09:42:26 UTC
Can you explain how to catalog a B214? (Why the B and not just 214?)
Comment 13 Arthur Suzuki 2022-05-11 08:57:36 UTC
Hi david,
thx for the signoff, actually, the mrc file has been provided by Severine Queune, I've just uploaded it to bugzilla.

Hi Katrin,
actually the bz title may be wrong, it displays any 214 fields.
Have a look at the attached record for some examples.

Maybe François can confirm that "b214" is a naming convention for "a 214 field"?

Arthur
Comment 14 Séverine Queune 2022-05-11 09:59:58 UTC
Ha (In reply to Katrin Fischer from comment #12)
> Can you explain how to catalog a B214? (Why the B and not just 214?)

Hi Katrin !
'B_xxx' is means 'bibliographic'.
It's quite an automatism that we have to precede the name of the field in order not to confuse it with the same number field of the authorities which would be 'A_xxx'.
So in that case, saying 'field 214' works fine too :)

> (In reply to David Nind from comment #11)
> > Thanks Arthur for providing a sample record - that made it much easier to
> > test!
(In reply to Arthur Suzuki from comment #13)
> Hi david,
> thx for the signoff, actually, the mrc file has been provided by Severine
> Queune, I've just uploaded it to bugzilla.

Thank you for testing David ! I didn't because the code partially comes the BULAC stylesheet, I don't think it would be fair signing it :)