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/

Reply via email to