[Koha-bugs] [Bug 8236] Prevent renewing if overdue or restriction

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

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

Koha Team Lyon 3 <koha at univ-lyon3.fr> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #11958|0                           |1
        is obsolete|                            |

--- Comment #12 from Koha Team Lyon 3 <koha at univ-lyon3.fr> ---
Created attachment 12362
  -->
http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=12362&action=edit
Block renewal for overdue documents

I reworked entirely the initial patch, added some parts that were obviously
missing (Detail circulation tab was forgotten had different behaviour from
Checkout one) and tight all with a new syspref (RenewIfOverdue).
As for the updatedatabase.pl part and specially the version number I doubt it
would be correct, cause I don't know the way to set it, I just increment the
last version number present in our updatedatabase.pl file.

The syspref offers a triple choice when any of a patron's document is late :
=> allow renewal anyhow (if the document has renewable type of course) which is
today the default action.
=> block only the overdue items
=> block all patron's checked out documents  (the big punishment...).

In the circulation interface, the renewal remains possible through the Override
renewal button. the Renew all one is hidden if blocking options are choosen.
In the patron's interfaces (Patron's file and the Webservice apis), the same
logic is working.
Beside the object of this patch, I also noted 2 things that would diserve to
have a look :
1) the comments of the IsMemberBlocked routine announces a control on the fines
status of the borrower, but the control is done on his restriction status.
2) In case of restriction, the patron's file is frozen (so renewal is not
possible but through the webservice interface (RenewLoan in Services.pm) it is. 

The present patch will correct that last point if validated (an other story)
but I think it should be corrected anyhow.

-- 
You are receiving this mail because:
You are watching all bug changes.