Bug 9455

Summary: Embedded mysql quotes in sql string
Product: Koha Reporter: Colin Campbell <colin.campbell>
Component: AcquisitionsAssignee: Colin Campbell <colin.campbell>
Status: CLOSED FIXED QA Contact: Katrin Fischer <katrin.fischer>
Severity: trivial    
Priority: P5 - low    
Version: Main   
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:
Attachments: Proposed patch
[SIGNED-OFF] Bug 9455: Remove unnecessary quoting from sql
[PASSED QA] Bug 9455: Remove unnecessary quoting from sql

Description Colin Campbell 2013-01-23 11:53:01 UTC
The sql in C4::Acquisitions::GetBasketgroups contains mysql specific backtick quotes. We have been removing these from sql files. They should be removed from the code for consistency
Comment 1 Colin Campbell 2013-01-23 12:01:30 UTC Comment hidden (obsolete)
Comment 2 Owen Leonard 2013-01-28 15:14:09 UTC Comment hidden (obsolete)
Comment 3 Katrin Fischer 2013-02-17 18:11:10 UTC
Created attachment 15480 [details] [review]
[PASSED QA] Bug 9455: Remove unnecessary quoting from sql

Wa've removed the mysql specific back quotes from sql files
But this case remained of them embedded in an sql string
Also removed the unnecessary call to finish
and temporary variable in the subroutine return

Testing: Functionality should remain unchanged. Retrieval of
basketgroups in acquisitions should not be affected

Signed-off-by: Owen Leonard <oleonard@myacpl.org>

Works according to test instructions: Basketgroups
functionality appears to work normally.

Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de>
Checked SQL produces the same results as before and did some
tests on the basket group functionality.
Comment 4 Jared Camins-Esakov 2013-02-20 13:23:25 UTC
This patch has been pushed to master.