Bug 7221

Summary: DelBiblio POD is incorrect
Product: Koha Reporter: Ian Walls <koha.sekjal>
Component: DocumentationAssignee: Joy Nelson <1joynelson>
Status: CLOSED FIXED QA Contact: Jonathan Druart <jonathan.druart>
Severity: minor    
Priority: PATCH-Sent (DO NOT USE) CC: 1joynelson, paul.poulain
Version: 3.6   
Hardware: All   
OS: All   
Change sponsored?: --- Patch complexity: ---
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Attachments: Patch file
Signed off patch

Description Ian Walls 2011-11-14 16:54:38 UTC
C4::Biblio::DelBiblio's in-line documentation says it takes two params, $dbh and $biblionumber, but in reality, it only takes $biblionumber.  Attempting to pass this extra param in a script will result in that script's failure.
Comment 1 Joy Nelson 2011-11-14 18:56:48 UTC Comment hidden (obsolete)
Comment 2 Ian Walls 2011-11-14 19:02:57 UTC
Created attachment 6305 [details] [review]
Signed off patch

Signed off patch... assigning to Jonathan for QA (since it's been just ByWater thus far in the writing/testing process)
Comment 3 Paul Poulain 2011-11-17 15:54:35 UTC
tiny patch, pushed. worth porting to 3.6

Joy, welcome into the list of commiters, you're 160th ! (and added in doc/history.txt file)
Comment 4 Joy Nelson 2011-11-17 20:02:59 UTC
Paul - 
I'm actually already listed in history.txt as #156 on 9/23/2011.  This is my second patch.  Still worthy of celebration!!
Thanks,
Joy

(In reply to comment #3)
> tiny patch, pushed. worth porting to 3.6
> 
> Joy, welcome into the list of commiters, you're 160th ! (and added in
> doc/history.txt file)
Comment 5 Paul Poulain 2011-11-21 13:21:26 UTC
(In reply to comment #4)
> Paul - 
> I'm actually already listed in history.txt as #156 on 9/23/2011.  This is my
> second patch.
oups... how could I miss you ? i've cancelled the commiter #160 line