Bug 6807 - Add ISBN filter to advanced order search
Summary: Add ISBN filter to advanced order search
Alias: None
Product: Koha
Classification: Unclassified
Component: Acquisitions (show other bugs)
Version: unspecified
Hardware: All All
: PATCH-Sent (DO NOT USE) enhancement (vote)
Assignee: Amit Gupta
QA Contact: Bugs List
Depends on: 6721 7085
  Show dependency treegraph
Reported: 2011-08-30 05:03 UTC by Savitra Sirohi
Modified: 2012-10-25 23:10 UTC (History)
6 users (show)

See Also:
Change sponsored?: ---
Patch complexity: ---
Documentation contact:
Documentation submission:
Text to go in the release notes:
Version(s) released in:

Bug 6807 - Add ISBN filter to advanced order search. (5.09 KB, patch)
2011-09-08 05:48 UTC, Amit Gupta
Details | Diff | Splinter Review
[SIGNED-OFF] Bug 6807 - Add ISBN filter to advanced order search. (5.17 KB, patch)
2011-09-08 13:32 UTC, Nicole C. Engard
Details | Diff | Splinter Review

Note You need to log in before you can comment on or make changes to this bug.
Description Savitra Sirohi 2011-08-30 05:03:11 UTC
Add isbn search to acqui/histsearch.pl
Comment 1 Amit Gupta 2011-09-08 05:48:23 UTC Comment hidden (obsolete)
Comment 2 Nicole C. Engard 2011-09-08 13:32:28 UTC
Created attachment 5358 [details] [review]
[SIGNED-OFF] Bug 6807 - Add ISBN filter to advanced order search.

Signed-off-by: Nicole C. Engard <nengard@bywatersolutions.com>
Comment 3 Paul Poulain 2011-10-06 16:26:36 UTC
QA comment:
couldn't test this patch, I think i've found a bug introduced by 6721 that prevent this one to work.

Otherwise, should be QA OK
Comment 4 Ian Walls 2011-10-13 17:20:39 UTC
Adds new param very cleanly, and returns more information equally cleanly.  Marking as Passed QA.

One bit of followup I'd like to see for Academic and Special libraries:  ISSN support.  Many libraries primarily collect journals, so having the ISSN option, either as a separate field, as a syspref choice, or having it automatically searched at the same time as biblioitems.isbn, would be a very nice addition to round this out.
Comment 5 Chris Cormack 2011-10-13 22:07:08 UTC
Small string added, I think it is worth adding anyway
Comment 6 Katrin Fischer 2011-11-27 21:38:28 UTC
Successfully tested on current HEAD.