| Summary: | z3950_search.pl ISSN search field not passed to Z3950Search | ||
|---|---|---|---|
| Product: | Koha | Reporter: | Dobrica Pavlinusic <dpavlin> |
| Component: | Cataloging | Assignee: | Dobrica Pavlinusic <dpavlin> |
| Status: | CLOSED FIXED | QA Contact: | Marcel de Rooy <m.de.rooy> |
| Severity: | major | ||
| Priority: | P5 - low | CC: | chris, m.de.rooy |
| Version: | Main | ||
| Hardware: | All | ||
| OS: | All | ||
| Change sponsored?: | --- | Patch complexity: | Trivial patch |
| Documentation contact: | Documentation submission: | ||
| Text to go in the release notes: | Version(s) released in: | ||
| Circulation function: | |||
| Attachments: |
Bug 9662 - ISSN search field not passed to Z3950Search
Bug 9662 - ISSN search field not passed to Z3950Search Bug 9662: Followup for acquisition |
||
|
Description
Dobrica Pavlinusic
2013-02-19 14:55:33 UTC
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.. |