Bug 12889

Summary: Improving documentation in C4::Members.pm
Product: Koha Reporter: shantanoo <shantanoo>
Component: Developer documentationAssignee: Galen Charlton <gmcharlt>
Status: CLOSED FIXED QA Contact: Testopia <testopia>
Severity: enhancement    
Priority: P5 - low CC: chris, rochellehealy.student, tomascohen
Version: unspecified   
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: Updated documentation message.
Diff using 'git format-patch'
Diff using 'git format-patch'
Updated pod for AddMember
[PASSED QA] Updated pod for AddMember

Description shantanoo 2014-09-08 16:02:38 UTC
In perldoc C4::Members.pm, AddMember does not explain what should be the keys for %borrower.

<snip>
   AddMember
         $borrowernumber = &AddMember(%borrower);

       insert new borrower into table Returns the borrowernumber upon success

       Returns as undef upon any db error without further processing
</snip>
Comment 1 shantanoo 2014-09-08 16:14:51 UTC Comment hidden (obsolete)
Comment 2 shantanoo 2014-09-08 17:23:11 UTC Comment hidden (obsolete)
Comment 3 shantanoo 2014-09-08 17:26:27 UTC
Created attachment 31468 [details]
Diff using 'git format-patch'
Comment 4 Rochelle Healy 2015-01-13 01:37:34 UTC Comment hidden (obsolete)
Comment 5 Katrin Fischer 2015-01-13 07:03:55 UTC
Created attachment 35139 [details] [review]
[PASSED QA] Updated pod for AddMember

Signed-off-by: Rochelle <Rochelle_healy@hotmail.com>

http://bugs.koha-community.org/show_bug.cgi?id=12889

Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Comment 6 Tomás Cohen Arazi 2015-01-15 00:49:51 UTC
Patch pushed to master.

Thanks and congratulations!
Comment 7 Chris Cormack 2015-01-18 02:33:15 UTC
Pushed to 3.18.x will be in 3.18.3