Bug 14421

Summary: Small error in example in SMS.pm
Product: Koha Reporter: Eivin Giske Skaaren <eivin>
Component: Staff interfaceAssignee: Eivin Giske Skaaren <eivin>
Status: CLOSED FIXED QA Contact: Testopia <testopia>
Severity: trivial    
Priority: P5 - low CC: chris, gmcharlt, liz, mtj, tomascohen
Version: Main   
Hardware: All   
OS: All   
Change sponsored?: --- Patch complexity: String patch
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Circulation function:
Attachments: Corrected example in SMS.pm to working version with hashref.
[SIGNED-OFF] Bug 14421: Corrected example in SMS.pm to working version with hashref.
Bug 14421: Corrected example in SMS.pm to working version with hashref.

Description Eivin Giske Skaaren 2015-06-19 11:07:05 UTC
The send_sms subroutine needs a hashref so {} should be added to the example.
Comment 1 Eivin Giske Skaaren 2015-06-19 11:14:16 UTC Comment hidden (obsolete)
Comment 2 Bernardo Gonzalez Kriegel 2015-06-19 14:36:33 UTC Comment hidden (obsolete)
Comment 3 Jonathan Druart 2015-06-19 16:01:07 UTC
Created attachment 40413 [details] [review]
Bug 14421: Corrected example in SMS.pm to working version with hashref.

Signed-off-by: Bernardo Gonzalez Kriegel <bgkriegel@gmail.com>

Test:
1) Apply patch
2) perldoc C4/SMS.pm
3) Check fixed argument in example

Argument is hashref, POD is now right
Added additional space on second arg
No errors

Signed-off-by: Jonathan Druart <jonathan.druart@koha-community.org>
Comment 4 Tomás Cohen Arazi (tcohen) 2015-06-22 14:41:58 UTC
Patch pushed to master.

Thanks Eivin, and congratulations for your first patch!
Comment 5 Chris Cormack 2015-06-29 20:34:23 UTC
Pushed to 3.20.x will be in 3.20.2
Comment 6 Liz Rea 2015-07-20 22:06:26 UTC
Pushed to 3.18.x will be in 3.18.09
Comment 7 Mason James 2015-07-25 10:42:53 UTC
Pushed to 3.16.x, will be in 3.16.13