Summary: | Prevent renewing for restricted patrons | ||
---|---|---|---|
Product: | Koha | Reporter: | David Cook <dcook> |
Component: | Circulation | Assignee: | David Cook <dcook> |
Status: | CLOSED FIXED | QA Contact: | |
Severity: | normal | ||
Priority: | P5 - low | CC: | gmcharlt, koha, kyle.m.hall |
Version: | Main | ||
Hardware: | All | ||
OS: | All | ||
See Also: | http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8236 | ||
Change sponsored?: | --- | Patch complexity: | Small patch |
Documentation contact: | Documentation submission: | ||
Text to go in the release notes: | Version(s) released in: | ||
Attachments: | Bug 10089 - Prevent renewing for restricted patrons |
Description
David Cook
2013-04-22 03:00:57 UTC
Created attachment 17570 [details] [review] Bug 10089 - Prevent renewing for restricted patrons (In reply to comment #0) > In the OPAC patron screen, they are informed that their account is frozen, > but...they are still able to renew books. I do not see how they are able to renew books. The table of checkouts shows the count of renewals, but there are no controls on the page for renewing. > 1) Prevent renewing of checked out items by patrons/staff I think this is a good improvement but I think it's confusing that the "override renewal limit" checkbox still works to enable the renew checkboxes in the table of checkouts. If the user is restricted there should probably only be a button which says "Check in checked items." I'm setting to "Failed QA" because I'd like clarification on the first issue at the very least in order to enable proper testing. It's arguable that the second issue isn't enough to block this. (In reply to comment #2) > (In reply to comment #0) > > In the OPAC patron screen, they are informed that their account is frozen, > > but...they are still able to renew books. > > I do not see how they are able to renew books. The table of checkouts shows > the count of renewals, but there are no controls on the page for renewing. > It appears that you are correct! I'm not sure why I said that patrons could still renew books through the OPAC. I would've sworn that I tested it and found that to be the case, but now that I'm looking...there are no controls as you say. It's probably worth dropping the opac-user.pl/.tt changes from this patch then. > > 1) Prevent renewing of checked out items by patrons/staff > > I think this is a good improvement but I think it's confusing that the > "override renewal limit" checkbox still works to enable the renew checkboxes > in the table of checkouts. If the user is restricted there should probably > only be a button which says "Check in checked items." > > I'm setting to "Failed QA" because I'd like clarification on the first issue > at the very least in order to enable proper testing. It's arguable that the > second issue isn't enough to block this. Mmm, I didn't see that "override renewal limit" checkbox before. I also agree that there should only be a "Check in checked items" button. I'll take another look at this when I have some time. I would have add Lyon3 right away in the list copy of this bug , I 'm confused. I didn't see you had already worked on it and also treated the restricted problem in the patch I proposed yesterday for bug 8236 (unlike I was supposed to do, I changed my mind). Olivier Crouzet (In reply to comment #4) > I would have add Lyon3 right away in the list copy of this bug , I 'm > confused. I didn't see you had already worked on it and also treated the > restricted problem in the patch I proposed yesterday for bug 8236 (unlike I > was supposed to do, I changed my mind). > > Olivier Crouzet Olivier, I added a link to this bug on bug 8236 (see comment #32) when I opened this one. I thought it might be rude to add Lyon3's email to the CC list, but I'll keep it in mind for next time ;). |