[Koha-bugs] [Bug 10863] Add 2 circulation permissions: force_checkout and lift_restriction
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10863 Galen Charlton changed: What|Removed |Added Status|Passed QA |Pushed to Master --- Comment #38 from Galen Charlton --- Pushed to master, along with a follow-up to make it clear to a circ operator who lacks the force_checkouts permission how to proceed if they cannot make the loan. Thanks, Jonathan! -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 10863] Add 2 circulation permissions: force_checkout and lift_restriction
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10863 --- Comment #37 from Galen Charlton --- (In reply to mathieu saby from comment #31) > And 13.00.XXX should now be 15.00.XXX in updatedatabase As a general note from the RM: I don't care at all what the stub DBrev number is set to so long as it contains XXX as expected by CheckVersion in updatedatabase.pl -- it's going to get changed anyway when the patch is pushed to master. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 10863] Add 2 circulation permissions: force_checkout and lift_restriction
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10863 Kyle M Hall changed: What|Removed |Added Status|Signed Off |Passed QA -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 10863] Add 2 circulation permissions: force_checkout and lift_restriction
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10863 Kyle M Hall changed: What|Removed |Added Attachment #23254|0 |1 is obsolete|| --- Comment #36 from Kyle M Hall --- Created attachment 23540 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=23540&action=edit Bug 10863: Add 2 circulation permissions: force_checkout and lift_restriction This patch adds 2 circulation permissions: force_checkout and lift_restriction. Both are added to existing patrons having circulation flags. force_checkout allows a librarian to force a checkout if a limitation occurred. lift_restriction allows a librarian to lift a restriction for a patron. Test plan: 1/ Find a debarred patron and go on the checkout page. The "Lift restriction" button should be present only if the logged librarian has the lift_restriction permission. 2/ If the force_checkout permission is set, a librarian should be allowed to check out in several cases: - age restriction - the item is issued to another patron - the item is not for loan - the patron has overdue items - the item is lost - the item is a high demand item - the item is reserved - another case ? Signed-off-by: Cedric Vita Bug 10863: The force checkout permission should not affect high holds - typo checkout => check out. - the force checkout permission does not affect high holds. Signed-off-by: Cedric Vita Bug 10863: Follow-up: Adding missing permissions to translated files This patch adds the 2 new permissions to all remaining sample files. If the permissions don't get installed, the problem is bigger than having an English description. Also adds back the question "Check out anyway?" for high demand items (HIGHHOLDS). Test xt/permissions.t passes now. Bug 10863: Rename lift_restriction with manage_restrictions The pref manage_restrictions now takes into account "Lost card" and "Gone no address" restrictions. Test plan: - log in with a user with manage_restrictions permission - verify you can set/unset restrictions when editing a patron - log in with a user without manage_restrictions permission - verify you cannot set/unset restrictions when editing a patron Signed-off-by: Mathieu Saby Note: The AgeRestrictionOverride pref has to be set to "Allow" if you want to override the age restriction, even if the new permission is set. Signed-off-by: Kyle M Hall Passes koha-qa.pl, works as advertised -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 10863] Add 2 circulation permissions: force_checkout and lift_restriction
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10863 mathieu saby changed: What|Removed |Added Status|Needs Signoff |Signed Off -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 10863] Add 2 circulation permissions: force_checkout and lift_restriction
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10863 mathieu saby changed: What|Removed |Added Attachment #22717|0 |1 is obsolete|| --- Comment #35 from mathieu saby --- Created attachment 23254 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=23254&action=edit [PATCH][SIGNED OFF] Bug 10863: Add 2 circulation permissions: force_checkout and lift_restriction Ok, so I sign off -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 10863] Add 2 circulation permissions: force_checkout and lift_restriction
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10863 Jonathan Druart changed: What|Removed |Added Status|Failed QA |Needs Signoff --- Comment #34 from Jonathan Druart --- (In reply to mathieu saby from comment #33) > All these operations were possible with the permission (and with some > choices in sysprefs), and not possible without, except one operation : > checking out a document marked "PEGI 12" to a 4 year child > (I mapped biblioitems.agerestricted with 301a field, set the preference > AgeRestrictionMarker with "PEGI", and AgeRestrictionOverride to "Don't > allow") Thanks for testing Mathieu! The AgeRestrictionOverride pref has to be set to "Allow" if you want to override the age restriction, even if the new permission is set. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 10863] Add 2 circulation permissions: force_checkout and lift_restriction
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10863 mathieu saby changed: What|Removed |Added Status|Needs Signoff |Failed QA --- Comment #33 from mathieu saby --- For force_checkout permission : I tried to play with some Circulation syprefs ( AllowFineOverride , IssueLostItem, OverduesBlockCirc ) and to make checkouts - of a lost item - of a not for loan item - of an item checked out to an other patron - of a document marked "PEGI 12" to a 4 year child - with a return date choosen in the past with the datepicker - to a patron with overdues - to a patron with fines exceding limit - to a patron with already the maximum number of documents on his card All these operations were possible with the permission (and with some choices in sysprefs), and not possible without, except one operation : checking out a document marked "PEGI 12" to a 4 year child (I mapped biblioitems.agerestricted with 301a field, set the preference AgeRestrictionMarker with "PEGI", and AgeRestrictionOverride to "Don't allow") I'm failing QA just for the last point. Maybe I misunderstood the expected behavior, but could you take a look at that please? Mathieu -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 10863] Add 2 circulation permissions: force_checkout and lift_restriction
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10863 --- Comment #32 from mathieu saby --- It's not the purpose of this patch, but 2 UX remarks : - it could maybe be interesting to manage the 2 flags "lost card" and "gone no adress" as 2 kinds of restrictions, using the new restriction system. By the way, I don't know if "flag" is meaningful in english in this context, but in French the translation has always be difficult. - when a librarian has the rights to manage restrictions, he must click on restriction tab, or on "View restrictions" button to manage them, but Koha does not tell him that he will have the possibility to manage the restriction. Maybe the button "View restriction" should be pĥrased "Manage restriction" when the librarian has the manage_restriction permission. Mathieu -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 10863] Add 2 circulation permissions: force_checkout and lift_restriction
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10863 mathieu saby changed: What|Removed |Added CC||mathieu.saby@univ-rennes2.f ||r --- Comment #31 from mathieu saby --- I'm testing the permission for restriction (with the different types of restrictions). It's working well, but I remarked that the description of the manage_restriction need to be updated in updatedatabase "Lift restriction for restricted accounts" (and its name could also be changed in the title of the patch, but it's cosmetic) And 13.00.XXX should now be 15.00.XXX in updatedatabase Now I'm testing the force_checkout permission. Mathieu -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 10863] Add 2 circulation permissions: force_checkout and lift_restriction
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10863 --- Comment #30 from Koha Team AMU --- I applied the patch on a sandbox and had the following message : The sandbox you've requested is not ready. Some problems occurred applying patches from bug 10863: Something went wrong !Applying: Bug 10863: Add 2 circulation permissions: force_checkout and lift_restriction Using index info to reconstruct a base tree... Falling back to patching base and 3-way merge... Auto-merging installer/data/mysql/updatedatabase.pl CONFLICT (content): Merge conflict in installer/data/mysql/updatedatabase.pl Auto-merging koha-tmpl/intranet-tmpl/prog/en/includes/borrower_debarments.inc Auto-merging koha-tmpl/intranet-tmpl/prog/en/modules/circ/circulation.tt Auto-merging koha-tmpl/intranet-tmpl/prog/en/modules/members/memberentrygen.tt Failed to merge in the changes. Patch failed at 0001 Bug 10863: Add 2 circulation permissions: force_checkout and lift_restriction When you have resolved this problem run git bz apply --continue. If you would prefer to skip this patch, instead run git bz apply --skip. To restore the original branch and stop patching run git bz apply --abort. Bug 10863 - Add 2 circulation permissions: force_checkout and lift_restriction 22717 - Bug 10863: Add 2 circulation permissions: force_checkout and lift_restriction Apply? [(y)es, (n)o, (i)nteractive] Patch left in tmpBug-10863-Add-2-circulation-permissions-forcecheck-Vi1MDW.patch . -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 10863] Add 2 circulation permissions: force_checkout and lift_restriction
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10863 Koha Team AMU changed: What|Removed |Added CC||koha.aixmarsei...@gmail.com -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 10863] Add 2 circulation permissions: force_checkout and lift_restriction
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10863 Nicole C. Engard changed: What|Removed |Added CC||neng...@gmail.com -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 10863] Add 2 circulation permissions: force_checkout and lift_restriction
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10863 Jonathan Druart changed: What|Removed |Added Depends on||2720 -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 10863] Add 2 circulation permissions: force_checkout and lift_restriction
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10863 --- Comment #29 from Jonathan Druart --- Bug 2720 breaks existing patches. I squashed them and rewrite a part of the original patches. Back to needs signoff. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 10863] Add 2 circulation permissions: force_checkout and lift_restriction
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10863 Jonathan Druart changed: What|Removed |Added Attachment #21247|0 |1 is obsolete|| Attachment #21248|0 |1 is obsolete|| Attachment #21249|0 |1 is obsolete|| Attachment #21542|0 |1 is obsolete|| Attachment #21661|0 |1 is obsolete|| --- Comment #28 from Jonathan Druart --- Created attachment 22717 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=22717&action=edit Bug 10863: Add 2 circulation permissions: force_checkout and lift_restriction This patch adds 2 circulation permissions: force_checkout and lift_restriction. Both are added to existing patrons having circulation flags. force_checkout allows a librarian to force a checkout if a limitation occurred. lift_restriction allows a librarian to lift a restriction for a patron. Test plan: 1/ Find a debarred patron and go on the checkout page. The "Lift restriction" button should be present only if the logged librarian has the lift_restriction permission. 2/ If the force_checkout permission is set, a librarian should be allowed to check out in several cases: - age restriction - the item is issued to another patron - the item is not for loan - the patron has overdue items - the item is lost - the item is a high demand item - the item is reserved - another case ? Signed-off-by: Cedric Vita Bug 10863: The force checkout permission should not affect high holds - typo checkout => check out. - the force checkout permission does not affect high holds. Signed-off-by: Cedric Vita Bug 10863: Follow-up: Adding missing permissions to translated files This patch adds the 2 new permissions to all remaining sample files. If the permissions don't get installed, the problem is bigger than having an English description. Also adds back the question "Check out anyway?" for high demand items (HIGHHOLDS). Test xt/permissions.t passes now. Bug 10863: Rename lift_restriction with manage_restrictions The pref manage_restrictions now takes into account "Lost card" and "Gone no address" restrictions. Test plan: - log in with a user with manage_restrictions permission - verify you can set/unset restrictions when editing a patron - log in with a user without manage_restrictions permission - verify you cannot set/unset restrictions when editing a patron -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 10863] Add 2 circulation permissions: force_checkout and lift_restriction
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10863 I'm just a bot changed: What|Removed |Added When did the bot|2013-10-02 00:00:00 |2013-10-28 last check this|| --- Comment #27 from I'm just a bot --- Patch applied cleanly, go forth and signoff -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 10863] Add 2 circulation permissions: force_checkout and lift_restriction
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10863 I'm just a bot changed: What|Removed |Added CC||git...@bugs.koha-community. ||org When did the bot||2013-10-02 last check this|| --- Comment #26 from I'm just a bot --- Patch applied cleanly, go forth and signoff -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 10863] Add 2 circulation permissions: force_checkout and lift_restriction
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10863 --- Comment #25 from Jonathan Druart --- (In reply to Katrin Fischer from comment #23) > I am not sure about including the gone no adress and lost flag - but I like > manage_restrictions. Maybe we can have a quick poll on IRC? Oops, I did not wait for your answer. I think it makes sense to include 3 restrictions, but maybe I'm wrong. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 10863] Add 2 circulation permissions: force_checkout and lift_restriction
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10863 --- Comment #24 from Jonathan Druart --- Created attachment 21661 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=21661&action=edit Bug 10863: Rename lift_restriction with manage_restrictions The pref manage_restrictions now takes into account "Lost card" and "Gone no address" restrictions. Test plan: - log in with a user with manage_restrictions permission - verify you can set/unset restrictions when editing a patron - log in with a user without manage_restrictions permission - verify you cannot set/unset restrictions when editing a patron -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 10863] Add 2 circulation permissions: force_checkout and lift_restriction
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10863 Jonathan Druart changed: What|Removed |Added Status|Failed QA |Needs Signoff -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 10863] Add 2 circulation permissions: force_checkout and lift_restriction
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10863 --- Comment #23 from Katrin Fischer --- I am not sure about including the gone no adress and lost flag - but I like manage_restrictions. Maybe we can have a quick poll on IRC? -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 10863] Add 2 circulation permissions: force_checkout and lift_restriction
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10863 --- Comment #22 from Jonathan Druart --- (In reply to Jonathan Druart from comment #21) > Katrin, > In fact I completely forgot a staff can change the flags on editing a patron. > The goal of this patch was to hide/show the "Lift restriction" button on top > of the circulation page. > But it is not enough. Looking at the patch, I did not forget :) What do you think about changing the lift_restriction permission to manage_restrictions (including Restricted, Lost and Gone no address)? -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 10863] Add 2 circulation permissions: force_checkout and lift_restriction
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10863 --- Comment #21 from Jonathan Druart --- (In reply to Katrin Fischer from comment #20) > Hi Jonathan, > > I have one bigger and some smaller issues with "Lift restriction for > restricted accounts" > > I think either the description is wrong, or it does not work like it should. > We have 3 possible restrictions on patron accounts: > > - Restricted > - Lost > - Gone no address > > It would be nice if it was clearer that it only affected "Restricted", but > that is ok. (1) > > What I have a problem with is, that you can also not restrict someone if you > are missing the permission - which means it's not only for 'lifting' but > also for 'setting' the restriction. Is this by design? (2) Katrin, In fact I completely forgot a staff can change the flags on editing a patron. The goal of this patch was to hide/show the "Lift restriction" button on top of the circulation page. But it is not enough. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 10863] Add 2 circulation permissions: force_checkout and lift_restriction
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10863 Katrin Fischer changed: What|Removed |Added Status|Signed Off |Failed QA --- Comment #20 from Katrin Fischer --- Hi Jonathan, I have one bigger and some smaller issues with "Lift restriction for restricted accounts" I think either the description is wrong, or it does not work like it should. We have 3 possible restrictions on patron accounts: - Restricted - Lost - Gone no address It would be nice if it was clearer that it only affected "Restricted", but that is ok. (1) What I have a problem with is, that you can also not restrict someone if you are missing the permission - which means it's not only for 'lifting' but also for 'setting' the restriction. Is this by design? (2) If you don't set a date for the restriction, which you can to indicate an unlimited restriction, the message shown without permission is: "Restricted: Yes, until" (3) Failed for (2) - please verify what the expected behaviour is and provide a follow-up. Thx! -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 10863] Add 2 circulation permissions: force_checkout and lift_restriction
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10863 --- Comment #19 from Katrin Fischer --- Created attachment 21542 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=21542&action=edit Bug 10863: Follow-up: Adding missing permissions to translated files This patch adds the 2 new permissions to all remaining sample files. If the permissions don't get installed, the problem is bigger than having an English description. Also adds back the question "Check out anyway?" for high demand items (HIGHHOLDS). Test xt/permissions.t passes now. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 10863] Add 2 circulation permissions: force_checkout and lift_restriction
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10863 --- Comment #18 from Katrin Fischer --- Fails xt/permissions.t for missing entries in various translated permissions.sql files, I will provide a follow-up. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 10863] Add 2 circulation permissions: force_checkout and lift_restriction
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10863 Katrin Fischer changed: What|Removed |Added CC||katrin.fisc...@bsz-bw.de QA Contact|testo...@bugs.koha-communit |katrin.fisc...@bsz-bw.de |y.org | -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 10863] Add 2 circulation permissions: force_checkout and lift_restriction
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10863 --- Comment #17 from Katrin Fischer --- Working on this now... -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 10863] Add 2 circulation permissions: force_checkout and lift_restriction
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10863 Jonathan Druart changed: What|Removed |Added Attachment #21201|0 |1 is obsolete|| --- Comment #14 from Jonathan Druart --- Created attachment 21247 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=21247&action=edit Bug 10863: Add 2 circulation permissions: force_checkout and lift_restriction This patch adds 2 circulation permissions: force_checkout and lift_restriction. Both are added to existing patrons having circulation flags. force_checkout allows a librarian to force a checkout if a limitation occurred. lift_restriction allows a librarian to lift a restriction for a patron. Test plan: 1/ Find a debarred patron and go on the checkout page. The "Lift restriction" button should be present only if the logged librarian has the lift_restriction permission. 2/ If the force_checkout permission is set, a librarian should be allowed to check out in several cases: - age restriction - the item is issued to another patron - the item is not for loan - the patron has overdue items - the item is lost - the item is a high demand item - the item is reserved - another case ? Signed-off-by: Cedric Vita -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 10863] Add 2 circulation permissions: force_checkout and lift_restriction
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10863 Jonathan Druart changed: What|Removed |Added Attachment #21204|0 |1 is obsolete|| --- Comment #16 from Jonathan Druart --- Created attachment 21249 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=21249&action=edit Bug 10863: The force checkout permission should not affect high holds - typo checkout => check out. - the force checkout permission does not affect high holds. Signed-off-by: Cedric Vita -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 10863] Add 2 circulation permissions: force_checkout and lift_restriction
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10863 Jonathan Druart changed: What|Removed |Added Attachment #21203|0 |1 is obsolete|| --- Comment #15 from Jonathan Druart --- Created attachment 21248 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=21248&action=edit Bug 10863: The lift restriction button should be hide on the moremember page This patchs hides the lift restriction button on the moremember page is needed. Signed-off-by: Cedric Vita -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 10863] Add 2 circulation permissions: force_checkout and lift_restriction
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10863 Pierre Angot changed: What|Removed |Added Status|Needs Signoff |Signed Off CC||tredok.pie...@gmail.com -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 10863] Add 2 circulation permissions: force_checkout and lift_restriction
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10863 --- Comment #13 from cedric.v...@dracenie.com --- Signing patches on sandbox (test1) did not work. Thank you to "Sign-off" on. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 10863] Add 2 circulation permissions: force_checkout and lift_restriction
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10863 --- Comment #12 from Jonathan Druart --- (In reply to cedric.vita from comment #11) > All works except Hold found. It is impossible to hold confimr. I do not know > if the problem is the patch or not. > Message: > Software error: > Could not find a letter called Expired 'HOLD_PRINT' in the 'reserved' Module > at / home/koha/src/C4/Reserves.pm line 1880. > For help, please send mail to the webmaster (webmaster @ > koha36-sandbox-matrix), giving this error messages and the time and date of > the error. > > Set permission added work. Curious to see what kind of permission You must create a HOLD_PRINT notice (in tools/letters.pl). -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 10863] Add 2 circulation permissions: force_checkout and lift_restriction
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10863 cedric.v...@dracenie.com changed: What|Removed |Added CC||cedric.v...@dracenie.com --- Comment #11 from cedric.v...@dracenie.com --- All works except Hold found. It is impossible to hold confimr. I do not know if the problem is the patch or not. Message: Software error: Could not find a letter called Expired 'HOLD_PRINT' in the 'reserved' Module at / home/koha/src/C4/Reserves.pm line 1880. For help, please send mail to the webmaster (webmaster @ koha36-sandbox-matrix), giving this error messages and the time and date of the error. Set permission added work. Curious to see what kind of permission -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 10863] Add 2 circulation permissions: force_checkout and lift_restriction
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10863 --- Comment #10 from Jonathan Druart --- Created attachment 21204 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=21204&action=edit Bug 10863: The force checkout permission should not affect high holds - typo checkout => check out. - the force checkout permission does not affect high holds. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 10863] Add 2 circulation permissions: force_checkout and lift_restriction
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10863 Jonathan Druart changed: What|Removed |Added Attachment #20948|0 |1 is obsolete|| Attachment #21042|0 |1 is obsolete|| Attachment #21043|0 |1 is obsolete|| --- Comment #8 from Jonathan Druart --- Created attachment 21201 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=21201&action=edit Bug 10863: Add 2 circulation permissions: force_checkout and lift_restriction This patch adds 2 circulation permissions: force_checkout and lift_restriction. Both are added to existing patrons having circulation flags. force_checkout allows a librarian to force a checkout if a limitation occurred. lift_restriction allows a librarian to lift a restriction for a patron. Test plan: 1/ Find a debarred patron and go on the checkout page. The "Lift restriction" button should be present only if the logged librarian has the lift_restriction permission. 2/ If the force_checkout permission is set, a librarian should be allowed to check out in several cases: - age restriction - the item is issued to another patron - the item is not for loan - the patron has overdue items - the item is lost - the item is a high demand item - the item is reserved - another case ? -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 10863] Add 2 circulation permissions: force_checkout and lift_restriction
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10863 --- Comment #9 from Jonathan Druart --- Created attachment 21203 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=21203&action=edit Bug 10863: The lift restriction button should be hide on the moremember page This patchs hides the lift restriction button on the moremember page is needed. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 10863] Add 2 circulation permissions: force_checkout and lift_restriction
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10863 Jonathan Druart changed: What|Removed |Added Status|Failed QA |Needs Signoff -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 10863] Add 2 circulation permissions: force_checkout and lift_restriction
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10863 Jonathan Druart changed: What|Removed |Added Attachment #20981|0 |1 is obsolete|| --- Comment #6 from Jonathan Druart --- Created attachment 21042 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=21042&action=edit Bug 10863: The lift restriction button should be hide on the moremember page This patchs hides the lift restriction button on the moremember page is needed. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 10863] Add 2 circulation permissions: force_checkout and lift_restriction
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10863 --- Comment #7 from Jonathan Druart --- Created attachment 21043 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=21043&action=edit Bug 10863: The force checkout permission should not affect high holds - typo checkout => check out. - the force checkout permission does not affect high holds. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 10863] Add 2 circulation permissions: force_checkout and lift_restriction
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10863 Owen Leonard changed: What|Removed |Added Status|Needs Signoff |Failed QA --- Comment #5 from Owen Leonard --- Testing this again, I realize that there is a bigger problem with the checkout of high hold items: If the staff member doesn't have force_checkout permission they can't check out an item with high holds at all. There should be a "check out/don't check out" option but there isn't. I would also like to suggest a change in wording for the permission. Instead of "Force checkout if a limitation exists," how about "Override blocked checkouts." -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 10863] Add 2 circulation permissions: force_checkout and lift_restriction
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10863 Jonathan Druart changed: What|Removed |Added Status|Failed QA |Needs Signoff --- Comment #4 from Jonathan Druart --- (In reply to Owen Leonard from comment #2) > This definitely needs (1) fixed, (2) while you're at it. (3) I could be > argued about if I misunderstand. Owen, Yes you are right on these 3 points. Thank you to catch them! -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 10863] Add 2 circulation permissions: force_checkout and lift_restriction
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10863 --- Comment #3 from Jonathan Druart --- Created attachment 20981 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=20981&action=edit Bug 10863: The lift restriction button should be hide on the moremember page This followup fixes 3 issues: - hide the lift restriction button on the moremember page is needed. - typo checkout => check out. - the force checkout permission does not affect high holds. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 10863] Add 2 circulation permissions: force_checkout and lift_restriction
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10863 Owen Leonard changed: What|Removed |Added Status|Needs Signoff |Failed QA --- Comment #2 from Owen Leonard --- A couple of issues: 1. When the logged-in user lacks the permission to lift restrictions the "lift restriction" button is gone from the circulation page but not from the patron detail page (moremember.pl). 2. "Cannot checkout" should be "Cannot check out" 3. I don't think the "high holds" warning should fall into the category of actions needing force_checkout permission. I think in this case the staff member is not overriding something which should normally be blocked. In this case the warning is informational: "Do you still want to check this out even though the loan period has been shortened?" This definitely needs (1) fixed, (2) while you're at it. (3) I could be argued about if I misunderstand. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 10863] Add 2 circulation permissions: force_checkout and lift_restriction
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10863 Jonathan Druart changed: What|Removed |Added Blocks||10859 -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 10863] Add 2 circulation permissions: force_checkout and lift_restriction
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10863 Jonathan Druart changed: What|Removed |Added Status|NEW |Needs Signoff Patch complexity|--- |Small patch Assignee|koha-b...@lists.koha-commun |jonathan.dru...@biblibre.co |ity.org |m -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
[Koha-bugs] [Bug 10863] Add 2 circulation permissions: force_checkout and lift_restriction
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=10863 --- Comment #1 from Jonathan Druart --- Created attachment 20948 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=20948&action=edit Bug 10863: Add 2 circulation permissions: force_checkout and lift_restriction This patch adds 2 circulation permissions: force_checkout and lift_restriction. Both are added to existing patrons having circulation flags. force_checkout allows a librarian to force a checkout if a limitation occurred. lift_restriction allows a librarian to lift a restriction for a patron. Test plan: 1/ Find a debarred patron and go on the checkout page. The "Lift restriction" button should be present only if the logged librarian has the lift_restriction permission. 2/ If the force_checkout permission is set, a librarian should be allowed to check out in several cases: - age restriction - the item is issued to another patron - the item is not for loan - the patron has overdue items - the item is lost - the item is a high demand item - the item is reserved - another case ? -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/