| 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 | ||
| GIT URL: | Initiative type: | --- | |
| Sponsorship status: | --- | Crowdfunding goal: | 0 | 
| 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..  |