Bug 13201

Summary: GetSuggestion takes suggestionid
Product: Koha Reporter: Jonathan Druart <jonathan.druart>
Component: Architecture, internals, and plumbingAssignee: Jonathan Druart <jonathan.druart>
Status: CLOSED FIXED QA Contact: Testopia <testopia>
Severity: normal    
Priority: P5 - low CC: chris, mtj, tomascohen
Version: Main   
Hardware: All   
OS: All   
Change sponsored?: --- Patch complexity: Trivial patch
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Attachments: Bug 13201: GetSuggestion takes suggestionid
[SIGNED OFF] Bug 13201: GetSuggestion takes suggestionid
[PASSED QA] Bug 13201: GetSuggestion takes suggestionid

Description Jonathan Druart 2014-11-05 11:41:09 UTC
The pod and the variable names are wrong!
This could be dangerous.
Comment 1 Jonathan Druart 2014-11-05 11:42:47 UTC Comment hidden (obsolete)
Comment 2 Mark Tompsett 2014-11-05 22:47:36 UTC Comment hidden (obsolete)
Comment 3 Katrin Fischer 2014-11-06 20:58:41 UTC
Created attachment 33349 [details] [review]
[PASSED QA] Bug 13201: GetSuggestion takes suggestionid

Having $ordernumber as the parameter in a function which is
handling suggestions and it is used as a suggestion id is
confusing to the coder and person trying to read the code.
This patch corrects the confusion.

REVISED TEST PLAN
-----------------
1) perldoc C4::Suggestions
   -- notice it says ordernumber for the GetSuggestions example.
2) Ensure you have CPL in your branches table.
3) prove -v t/db_dependent/Suggestions.t
   -- if you have more than 8 itemtypes in your database, two
      tests at the end will fail, but that is not relevant to
      the modified GetSuggestions code.
4) apply patch
5) perldoc C4::Suggestions
   -- notice it is corrected now.
6) prove -v t/db_dependent/Suggestions.t
   -- it should run with the same results as in step 3.
      That is, if you have 8 itemtypes, all pass, otherwise
      those two tests fail.
7) run koha qa test tool

Signed-off-by: Mark Tompsett <mtompset@hotmail.com>

Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Comment 4 Tomás Cohen Arazi 2015-02-09 18:57:03 UTC
Patch pushed to master.

Thanks Jonathan!
Comment 5 Chris Cormack 2015-02-12 22:47:28 UTC
Pushed to 3.18.x will be in 3.18.4
Comment 6 Mason James 2015-02-26 04:30:16 UTC
Pushed to 3.16.x, will be in 3.16.8