[Koha-bugs] [Bug 27268] Move C4::Biblio::GetMarcNotes to Koha namespace
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27268 Tomás Cohen Arazi changed: What|Removed |Added Status|Pushed to master|RESOLVED CC||tomasco...@gmail.com Resolution|--- |FIXED -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 27268] Move C4::Biblio::GetMarcNotes to Koha namespace
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27268 Fridolin Somers changed: What|Removed |Added Blocks||29871 Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=29871 [Bug 29871] Remove marcflavour param in Koha::Biblio->get_marc_* -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 27268] Move C4::Biblio::GetMarcNotes to Koha namespace
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27268 Tomás Cohen Arazi changed: What|Removed |Added Blocks||29652 Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=29652 [Bug 29652] Move Koha::Biblio->get_marc_* to Koha::Biblio::Metadata -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 27268] Move C4::Biblio::GetMarcNotes to Koha namespace
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27268 Martin Renvoize changed: What|Removed |Added See Also||https://bugs.koha-community ||.org/bugzilla3/show_bug.cgi ||?id=29441 -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 27268] Move C4::Biblio::GetMarcNotes to Koha namespace
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27268 --- Comment #11 from Aleisha Amohia --- I have added a patch to fix this at Bug 28734 -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 27268] Move C4::Biblio::GetMarcNotes to Koha namespace
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27268 Aleisha Amohia changed: What|Removed |Added Blocks||28734 Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=28734 [Bug 28734] Koha::Biblio->get_marc_notes should use XSLTParse4Display -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 27268] Move C4::Biblio::GetMarcNotes to Koha namespace
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27268 --- Comment #10 from Aleisha Amohia --- Thanks both I will try and get to this later this week, has a new bug been made for the required follow-up yet or should I just attach a follow-up here? -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 27268] Move C4::Biblio::GetMarcNotes to Koha namespace
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27268 Elaine Bradtke changed: What|Removed |Added CC||e...@efdss.org --- Comment #9 from Elaine Bradtke --- Here's what I posted on the list regarding our 590 fields. There is a subfield x Non Public Note which contains information for the staff regarding the content of the work, and a subfield z Public note that contains a generic statement governed by an authorized value. What we see in the normal view in both the staff interface and the OPAC is the Non Public note and the code for the authorized value for the public note. In the OPAC, it displays correctly in labeled MARC. In plain MARC it only displays the authorized value code. See https://catalogue.efdss.org/cgi-bin/koha/opac-detail.pl?biblionumber=65305 for an example. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 27268] Move C4::Biblio::GetMarcNotes to Koha namespace
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27268
Jonathan Druart changed:
What|Removed |Added
CC||jonathan.druart+koha@gmail.
||com
--- Comment #8 from Jonathan Druart ---
This patch introduces a bad side-effect.
From the mailing list
https://lists.katipo.co.nz/pipermail/koha/2021-June/056413.html
If you have a note in 590$z that is linked with an authorised value, then the
code will be displayed instead of the description.
What's happening?
Before this patch we did (the big lines):
my $record = GetMarcBiblio({ biblionumber => $biblionumber });
XSLTParse4Display($record);
my $notes = GetMarcNotes($record);
=> XSLTParse4Display *is modifying* the record (see the $field->replace_with
call in transformMARCXML4XSLT) and so GetMarcNotes returned the descriptions of
the notes
After this patch:
my $notes = $biblio->get_marc_notes;
=> We are no longer using the descriptions!
Aleisha, Can you please fix this?
--
You are receiving this mail because:
You are watching all bug changes.
___
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 27268] Move C4::Biblio::GetMarcNotes to Koha namespace
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27268 Fridolin Somers changed: What|Removed |Added CC||fridolin.som...@biblibre.co ||m --- Comment #7 from Fridolin Somers --- Enhancement not pushed to 20.11.x -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 27268] Move C4::Biblio::GetMarcNotes to Koha namespace
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27268 --- Comment #6 from Jonathan Druart --- Pushed to master for 21.05, thanks to everybody involved! -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 27268] Move C4::Biblio::GetMarcNotes to Koha namespace
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27268 Jonathan Druart changed: What|Removed |Added Status|Passed QA |Pushed to master Version(s)||21.05.00 released in|| -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 27268] Move C4::Biblio::GetMarcNotes to Koha namespace
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27268 Marcel de Rooy changed: What|Removed |Added QA Contact|testo...@bugs.koha-communit |m.de.r...@rijksmuseum.nl |y.org | -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 27268] Move C4::Biblio::GetMarcNotes to Koha namespace
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27268 --- Comment #5 from Marcel de Rooy --- Created attachment 117786 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=117786=edit Bug 27268: (QA follow-up) Add Koha::Biblios to basket scripts Signed-off-by: Marcel de Rooy -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 27268] Move C4::Biblio::GetMarcNotes to Koha namespace
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27268 Marcel de Rooy changed: What|Removed |Added Attachment #117240|0 |1 is obsolete|| --- Comment #4 from Marcel de Rooy --- Created attachment 117785 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=117785=edit Bug 27268: Move GetMarcNotes to Koha namespace This patch moves C4::Biblio::GetMarcNotes to Koha::Biblio->get_marc_notes. This is so get_marc_notes can be used in templates and notices. To test: 1. Find a record that has a note (3xx for UNIMARC, 5xx for MARC21). Confirm the notes still show as normal under the Descriptions tab. 2. Add the record to the cart. 3. View your cart and click 'more details'. Confirm notes show as normal. 4. Log in to the OPAC. Find the record and add it to the cart 5. View the cart and click 'more details'. Confirm notes show as normal. 6. View the record detail page and confirm the notes show as normal under the Title Notes tab. 7. Confirm tests pass: - t/Biblio.t - t/db_dependent/Koha/Biblio.t Sponsored-by: Bibliotheksservice-Zentrum Baden-Württemberg (BSZ) Signed-off-by: David Nind Signed-off-by: Marcel de Rooy -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 27268] Move C4::Biblio::GetMarcNotes to Koha namespace
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27268 Marcel de Rooy changed: What|Removed |Added Status|Signed Off |Passed QA Patch complexity|--- |Small patch -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 27268] Move C4::Biblio::GetMarcNotes to Koha namespace
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27268 Marcel de Rooy changed: What|Removed |Added CC||m.de.r...@rijksmuseum.nl --- Comment #3 from Marcel de Rooy --- With but also without this patch I have a test failure on Koha/Biblio.t: # Subtest: AddBiblio 1..4 not ok 1 - expected warnings when adding too long LCCN # Failed test 'expected warnings when adding too long LCCN' # at t/db_dependent/Biblio.t line 61. # didn't find a warning # expected to find warning: (?^u:Data too long for column 'lccn') # expected to find warning: (?^u:Data too long for column 'lccn') not ok 2 - AddBiblio returns undef for biblionumber if something went wrong # Failed test 'AddBiblio returns undef for biblionumber if something went wrong' # at t/db_dependent/Biblio.t line 63. # got: '1529' # expected: undef not ok 3 - AddBiblio returns undef for biblioitemnumber if something went wrong # Failed test 'AddBiblio returns undef for biblioitemnumber if something went wrong' # at t/db_dependent/Biblio.t line 65. # got: '1472' # expected: undef not ok 4 - No biblio should have been added if something went wrong # Failed test 'No biblio should have been added if something went wrong' # at t/db_dependent/Biblio.t line 68. # got: '1016' # expected: '1015' # Looks like you failed 4 tests of 4. not ok 2 - AddBiblio # Failed test 'AddBiblio' # at t/db_dependent/Biblio.t line 70. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 27268] Move C4::Biblio::GetMarcNotes to Koha namespace
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27268 David Nind changed: What|Removed |Added Attachment #114504|0 |1 is obsolete|| --- Comment #2 from David Nind --- Created attachment 117240 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=117240=edit Bug 27268: Move GetMarcNotes to Koha namespace This patch moves C4::Biblio::GetMarcNotes to Koha::Biblio->get_marc_notes. This is so get_marc_notes can be used in templates and notices. To test: 1. Find a record that has a note (3xx for UNIMARC, 5xx for MARC21). Confirm the notes still show as normal under the Descriptions tab. 2. Add the record to the cart. 3. View your cart and click 'more details'. Confirm notes show as normal. 4. Log in to the OPAC. Find the record and add it to the cart 5. View the cart and click 'more details'. Confirm notes show as normal. 6. View the record detail page and confirm the notes show as normal under the Title Notes tab. 7. Confirm tests pass: - t/Biblio.t - t/db_dependent/Koha/Biblio.t Sponsored-by: Bibliotheksservice-Zentrum Baden-Württemberg (BSZ) Signed-off-by: David Nind -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 27268] Move C4::Biblio::GetMarcNotes to Koha namespace
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27268 David Nind changed: What|Removed |Added Status|Needs Signoff |Signed Off -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 27268] Move C4::Biblio::GetMarcNotes to Koha namespace
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27268 Aleisha Amohia changed: What|Removed |Added Blocks|3150| Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=3150 [Bug 3150] Move emails for sending cart and list contents into notices tool -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 27268] Move C4::Biblio::GetMarcNotes to Koha namespace
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27268 --- Comment #1 from Aleisha Amohia --- Created attachment 114504 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=114504=edit Bug 27268: Move GetMarcNotes to Koha namespace This patch moves C4::Biblio::GetMarcNotes to Koha::Biblio->get_marc_notes. This is so get_marc_notes can be used in templates and notices. To test: 1. Find a record that has a note (3xx for UNIMARC, 5xx for MARC21). Confirm the notes still show as normal under the Descriptions tab. 2. Add the record to the cart. 3. View your cart and click 'more details'. Confirm notes show as normal. 4. Log in to the OPAC. Find the record and add it to the cart 5. View the cart and click 'more details'. Confirm notes show as normal. 6. View the record detail page and confirm the notes show as normal under the Title Notes tab. 7. Confirm tests pass: - t/Biblio.t - t/db_dependent/Koha/Biblio.t Sponsored-by: Bibliotheksservice-Zentrum Baden-Württemberg (BSZ) -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 27268] Move C4::Biblio::GetMarcNotes to Koha namespace
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27268 Aleisha Amohia changed: What|Removed |Added Status|NEW |Needs Signoff -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 27268] Move C4::Biblio::GetMarcNotes to Koha namespace
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=27268 Aleisha Amohia changed: What|Removed |Added Assignee|koha-b...@lists.koha-commun |alei...@catalyst.net.nz |ity.org | Blocks||3150 Change sponsored?|--- |Sponsored Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=3150 [Bug 3150] Move emails for sending cart and list contents into notices tool -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
