[Koha-bugs] [Bug 8365] Renewal duration in issuingules

[bugzilla-daemon at bugs.koha-community.org]

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

--- Comment #9 from Katrin Fischer <katrin.fischer at 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.