After executing Z39.50 search, the result table is not populated with publication dates. This is the result of code refactoring made by Bug 30813. The removed function _add_rowdata treated in special way the publication date putting it in $row in under special, non MARC variant dependent key 'date'--since the z3950_search.tt looks under breeding_loo.date. But now the $row->{date} is not defined. A patch will follow.
Created attachment 148195 [details] [review] Bug 33231 (Bug 30813 follow-up) No publication date in Z39.50 biblio search results After executing Z39.50 search, the result table is not populated with publication dates. This is the result of code refactoring made by Bug 30813. The removed function _add_rowdata treated in special way the publication date putting it in $row in under special, non MARC variant dependent key 'date'--since the z3950_search.tt looks under breeding_loo.date. Test plan: ========= 1. Have a Koha instance with some Z39.50 servers defined. 2. In Cataloging, perform a Z39.50 search for any term. 3. In the result table you would not get the publication dates. 4. Apply the patch. 5. Repeat the search. 6. You should see the publication dates (according to the current mapping, i.e. for MARC 21 coming from 260 $c or 264 $c subfield).
Created attachment 148196 [details] [review] Bug 33231 (Bug 30813 follow-up) No publication date nor edition statements in Z39.50 biblio search results After executing Z39.50 search, the result table is not populated with publication dates. This is the result of code refactoring made by Bug 30813. The removed function _add_rowdata treated in special way the publication date putting it in $row in under special, non MARC variant dependent key 'date'--since the z3950_search.tt looks under breeding_loo.date. Same effect (no data in the result table) with edition statement. Reason: editionstatement coming from TransformMarcToKoha vs edition expected by z3950_search.tt. Test plan: ========== 1. Have a Koha instance with some Z39.50 servers defined. 2. In Cataloging, perform a Z39.50 search for any term. 3. In the result table you would not get the publication dates nor edition statement (if present in the record). 4. Apply the patch. 5. Repeat the search. 6. You should see the publication dates (according to the current mapping, i.e. for MARC 21 coming from 260 $c or 264 $c subfield) and edition statements from the records found.
Created attachment 149142 [details] [review] Bug 33231 (Bug 30813 follow-up) No publication date nor edition statements in Z39.50 biblio search results After executing Z39.50 search, the result table is not populated with publication dates. This is the result of code refactoring made by Bug 30813. The removed function _add_rowdata treated in special way the publication date putting it in $row in under special, non MARC variant dependent key 'date'--since the z3950_search.tt looks under breeding_loo.date. Same effect (no data in the result table) with edition statement. Reason: editionstatement coming from TransformMarcToKoha vs edition expected by z3950_search.tt. Test plan: ========== 1. Have a Koha instance with some Z39.50 servers defined. 2. In Cataloging, perform a Z39.50 search for any term. 3. In the result table you would not get the publication dates nor edition statement (if present in the record). 4. Apply the patch. 5. Repeat the search. 6. You should see the publication dates (according to the current mapping, i.e. for MARC 21 coming from 260 $c or 264 $c subfield) and edition statements from the records found. Signed-off-by: David Nind <david@davidnind.com>
Testing notes (using KTD): 1. Searching for the title "Perl best practices" returns a result with the publication year and an edition statement.
Created attachment 149612 [details] [review] Bug 33231: (Bug 30813 follow-up) No publication date nor edition statements in Z39.50 biblio search results After executing Z39.50 search, the result table is not populated with publication dates. This is the result of code refactoring made by Bug 30813. The removed function _add_rowdata treated in special way the publication date putting it in $row in under special, non MARC variant dependent key 'date'--since the z3950_search.tt looks under breeding_loo.date. Same effect (no data in the result table) with edition statement. Reason: editionstatement coming from TransformMarcToKoha vs edition expected by z3950_search.tt. Test plan: ========== 1. Have a Koha instance with some Z39.50 servers defined. 2. In Cataloging, perform a Z39.50 search for any term. 3. In the result table you would not get the publication dates nor edition statement (if present in the record). 4. Apply the patch. 5. Repeat the search. 6. You should see the publication dates (according to the current mapping, i.e. for MARC 21 coming from 260 $c or 264 $c subfield) and edition statements from the records found. Signed-off-by: David Nind <david@davidnind.com> Signed-off-by: Nick Clemens <nick@bywatersolutions.com>
Created attachment 149613 [details] [review] Bug 33231: (QA follow-up) Simplify conditional for date display This patch simply checks if we have a value for copyrighydate and displays publicationyear if not. Even if copyrightdate is requested (MARC21) but isn't populated, the publicatoinyear won't replace it because we haven't transformed that field. I think this read a bit easier, but RM can weigh in Signed-off-by: Nick Clemens <nick@bywatersolutions.com>
Fast moving patch, I like it :)
Pushed to master for 23.05. Nice work everyone, thanks!
Nice work everyone! Pushed to stable for 22.11.x
Missing 22.05.x dependencies. NO backport.