Summary: | searchResults uses SQL query rather than GetMarcFromKohaField | ||
---|---|---|---|
Product: | Koha | Reporter: | Jared Camins-Esakov <jcamins> |
Component: | Architecture, internals, and plumbing | Assignee: | Jared Camins-Esakov <jcamins> |
Status: | CLOSED FIXED | QA Contact: | Jonathan Druart <jonathan.druart> |
Severity: | normal | ||
Priority: | P5 - low | CC: | chris, jonathan.druart, julian.maurice, paul.poulain |
Version: | 3.8 | ||
Hardware: | All | ||
OS: | All | ||
GIT URL: | Change sponsored?: | --- | |
Patch complexity: | --- | Documentation contact: | |
Documentation submission: | Text to go in the release notes: | ||
Version(s) released in: | Circulation function: | ||
Bug Depends on: | |||
Bug Blocks: | 8649 | ||
Attachments: |
Bug 8648: searchResults uses SQL query rather than GetMarcFromKohaField
[SIGNED-OFF] Bug 8648: searchResults uses SQL query rather than GetMarcFromKohaField |
Description
Jared Camins-Esakov
2012-08-15 16:34:27 UTC
Created attachment 11629 [details] [review] Bug 8648: searchResults uses SQL query rather than GetMarcFromKohaField Around line 1470-something: my $sth = $dbh->prepare( "SELECT tagfield FROM marc_subfield_structure WHERE kohafield LIKE 'items.itemnumber'" ); $sth->execute; This patch replaces that with a call to GetMarcFromKohaField. To test: 1) Apply patch. 2) Do a search that returns both available and unavailable items. You'll know if the patch isn't working. Created attachment 11635 [details] [review] [SIGNED-OFF] Bug 8648: searchResults uses SQL query rather than GetMarcFromKohaField Simple change. Does not break anything. Save 1 query to database because result of GetMarcFromKohaField is cached. Signed off. QA Comment: use the routine GetMarcFromKohaField instead of execute an sql query. Marked as Passed QA Pushed to 3.8.x, will be in 3.8.5 |