Created attachment 15549 [details] [review] Bug 9662 - ISSN search field not passed to Z3950Search Seems like simple omission when passing $pars hash to Z3950Search function. Test scenario: 1. select any Z39.50 search form and enter valid ISSN 2. verify that you don't get any results from remote servers 3. apply this patch and re-check that results are now availble
Created attachment 15565 [details] [review] Bug 9662 - ISSN search field not passed to Z3950Search Seems like simple omission when passing $pars hash to Z3950Search function. Test scenario: 1. select any Z39.50 search form and enter valid ISSN 2. verify that you don't get any results from remote servers 3. apply this patch and re-check that results are now availble Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
Created attachment 15566 [details] [review] Bug 9662: Followup for acquisition Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl> Trivial change corresponding with Dobrica's patch at cataloging side.
QA Comment: Two easy one line patches making lots of people happy searching on ISSN :) Looks good to me. (Probably forgot to add this field myself..) Have been so bold to add the second patch myself instead of harassing Dobrica.. Passed QA Dobrica: If you read this before Jared does, please feel free to check..
This patch has been pushed to master.
Does not apply on 3.10.x please submit a patch for it if you wish it.
(In reply to comment #6) > Does not apply on 3.10.x please submit a patch for it if you wish it. The patches are only relevant for master, not for 3.10.x or lower..