http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8365
--- Comment #9 from Katrin Fischer <katrin.fisc...@bsz-bw.de> --- Hi Jonathan, I read through your code and have some questions, mostly about documentation: 1) I was wondering if this line in the POD for CanBookBeRenewed makes sense: C<$dbh> is a DBI handle to the Koha database. Just a note, little typo a few lines below: C<$CanBookBeRenewed> returns a true value _iff_ the item may be renewed. The ... 2) POD for CalcDateDue misses $isrenewal 3) In the circulation matrix the new field is named "Renewals period" - "Renewal period" looks a bit better to me, but we are both not native speakers, so I will leave that to someone else. :) I also ran the test suite - so far it looks good. I will continue with testing the interface next. -- You are receiving this mail because: You are watching all bug changes. _______________________________________________ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/