Summary: | Lost patron card tracking | ||
---|---|---|---|
Product: | Koha | Reporter: | Jane Wagner <jwagner> |
Component: | Patrons | Assignee: | Jane Wagner <jwagner> |
Status: | RESOLVED MOVED | QA Contact: | Bugs List <koha-bugs> |
Severity: | enhancement | ||
Priority: | PATCH-Sent (DO NOT USE) | CC: | chris, m.de.rooy, veron |
Version: | Main | ||
Hardware: | All | ||
OS: | All | ||
See Also: | https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=3820 | ||
Change sponsored?: | Sponsored | Patch complexity: | --- |
Documentation contact: | Documentation submission: | ||
Text to go in the release notes: | Version(s) released in: | ||
Circulation function: | |||
Attachments: |
Lost card screenshot
Squashed commit Follow-up corrections |
Description
Chris Cormack
2010-05-21 01:11:02 UTC
Can't find patch to apply. Please attach. Resetting patch status. Created attachment 3202 [details] [review] Squashed commit (In reply to comment #1) > Can't find patch to apply. Please attach. Resetting patch status. Found it altogether under branch origin/new_features_ptfs_lost_cards. Can still be merged with current master, or use squashed commit from previous comment. - Merge error? Patron search results table duplicated (/cgi-bin/koha/members/member.pl) - moremember.pl refers to a non-existent granular permission, "CAN_user_borrowers_edit_borrowers" I see that one of the commit messages refers to "Added granular permission check to previous cardnumber feature," but there isn't any code included to add a new permission. Line 125 of Stats.pm needs a correction for this to work. It should be: my $member = C4::Members::GetMember( borrowernumber => $borrowernumber ); Created attachment 3205 [details] [review] Follow-up corrections This patch makes the necessary correction to Stats.pm and removes the check for the granular edit permission. I would prefer for the edit permission to be properly added, but at least this patch gets it working. |