Bug 6834

Summary: $template->{param_map} is deprecated with the introduction of Template::Toolkit
Product: Koha Reporter: Chris Cormack <chris>
Component: Architecture, internals, and plumbingAssignee: Chris Cormack <chris>
Status: CLOSED FIXED QA Contact: Bugs List <koha-bugs>
Severity: major    
Priority: PATCH-Sent (DO NOT USE) CC: dev_patches, koha.sekjal
Version: 3.6   
Hardware: All   
OS: All   
Change sponsored?: --- Patch complexity: ---
Who signed the patch off: Text to go in the release notes:
Version(s) released in:
Bug Depends on: 6738    
Bug Blocks: 5917    
Attachments: Bug 6834 : Removing last of the $template->{param_map}
this is a followup. since patch sent would leave 2 param_map

Description Chris Cormack 2011-09-02 03:27:20 UTC
Removing any instances of this
Comment 1 Chris Cormack 2011-09-02 03:32:03 UTC
Created attachment 5270 [details] [review]
Bug 6834 : Removing last of the $template->{param_map}
Comment 2 dev_patches 2011-09-02 07:28:48 UTC
GBSD#4 t├ęking that.
Comment 3 dev_patches 2011-09-02 08:38:03 UTC
grep -rli param_map **/*.pl
acqui/acqui-home.pl
acqui/basket.pl
Comment 4 dev_patches 2011-09-02 08:45:15 UTC
Created attachment 5276 [details] [review]
this is a followup. since patch sent would leave 2 param_map
Comment 5 Ian Walls 2011-09-02 11:40:35 UTC
Taken together, both patches encompass all cases of {param_map} I can find in Koha.  Signing off on the second, and marking both as Passed QA
Comment 6 Chris Cormack 2011-09-02 18:40:41 UTC
This bug was set to depend on 6738, which removes another instance of param_map, since its now been removed here, can someone please test and resolve 6738

Also, please do try to note the depends on status, and sign off bugs in that order if at all possible.

Pushed, please test
Comment 7 Chris Nighswonger 2011-10-26 18:02:28 UTC
The fix for this bug was published in the 3.4.5 release. If you were the reporter of this bug, please take time to verify the fix and update the status of this bug report accordingly. If the bug is fixed to your satisfaction, please close this report.