Bug 5754

Summary:	Enhancements to patron debarrment
Product:	Koha	Reporter:	Chris Cormack <chris>
Component:	Patrons	Assignee:	Paul Poulain <paul.poulain>
Status:	CLOSED DUPLICATE	QA Contact:	Bugs List <koha-bugs>
Severity:	enhancement
Priority:	P5 - low	CC:	chris, gmcharlt, nengard, paul.poulain
Version:	Main
Hardware:	All
OS:	All
Change sponsored?:	---	Patch complexity:	---
Documentation contact:		Documentation submission:
Text to go in the release notes:		Version(s) released in:
Circulation function:
Bug Depends on:
Bug Blocks:	5575
Attachments:	follow-up patch to display debarrement date & comment on opac

Description Chris Cormack 2011-02-14 02:44:09 UTC

Now when a user is debarred, you set a end date, and can put a comment.
This patch is the first, it need some improvement, but works globaly well.
If an overdue generate a debar, the debar date is set to 9999-12-31.

show debar date on member detail
This patch show debar date, if the member is debarred, and he have a valid date(not unlimited debar).

add debar date in circulation
This patch add the debar date if it's not unlimited, in circulation.pl

rework on member edition
This change the way to edit user debar date/comment, to follow nicolas' mockup.

Comment 1 Nicole C. Engard 2011-02-14 12:25:37 UTC

This may be a translation issue, but since you're working on this area, I'd like to suggest for US English that 'debar' be changed to 'restricted' or 'barred' - I had never heard the term 'debar' until I saw Koha and those I train also ask me what it means.

Comment 2 Paul Poulain 2011-04-29 17:54:17 UTC

chris, where's the patch for this one ?
(i've a small follow-up, i'll attach. the follow-up will work only if debarred is a date, and there is a borrowers.debarredcomment)

Comment 3 Paul Poulain 2011-04-29 17:55:13 UTC

Created attachment 4022 [details] [review]
follow-up patch to display debarrement date & comment on opac

Comment 4 Chris Cormack 2011-05-01 20:47:24 UTC

Hmm interesting it appears I never formatted a patch for this one.

It probably needs to be pulled out of the 5575 branch, fixed for TT (if needed) then attached

Comment 5 Chris Cormack 2011-05-01 20:47:56 UTC

Follow up patch needs to be reformatted for TT too, before it will apply

Comment 6 Paul Poulain 2011-05-11 14:57:04 UTC

solved by 6328

*** This bug has been marked as a duplicate of bug 6328 ***