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

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

Attachments
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
Show Obsolete (1) View All Add an attachment (proposed patch, testcase, etc.)

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.