Created attachment 61338 [details] renew buttons circled in red Hi, Some of our staff users reports that when students are doing a search in the OPAC, the buttons "Renew selected" and "Renew all" are confusing for them (circled in red). They think they are able to renew their books but in fact they aren't allowed to do this. The problem is when the "OpacRenewalAllowed" is disabled, these buttons stay there and/or keep the same appearance. Thanks for helping, Alexandre from CCSR
Hi Alexandre, I can not reproduce on current master. What Koha version is it? Marc
Hi Marc, We use v3.22, we will soon pass to 16.05 however. Thanks for helping,
Very sorry for this second post. After a discussion with my collegue Marjorie, it's appears that this bug is in 16.11 too.
I just tested it in 16.11 too, they still there. We can also observe an error message (in yellow) when we try the "renew all" button (see second picture): " Your loan renewal failed because of the following reason(s): You have renewed this item the maximum number of times allowed. You have renewed this item the maximum number of times allowed." I must admit that can be very confusing for a student. These buttons let the student think that they can renew their books by themself and at home (OPAC) but they will realise just in the last days of their loans that they can't... This message don't help them to understand the situation. Marc, which version did you use ? Maybe it was corrected for 17.05 but I didn't see any bug on this.
Created attachment 61400 [details] error message with renew button in 16.11
(In reply to Alexandre from comment #4) (...) > Marc, which version did you use ? Maybe it was corrected for 17.05 but I > didn't see any bug on this. Current master (16.12.00.013)
I don't know if it's the only condition which creates the problem, but I see that having OverduesBlockRenewing set to 'block' or 'blockitem' can lead to a situation where the renew links are displayed but no checkboxes appear.
Still valid in 18.05 Marjorie
Created attachment 86853 [details] [review] Bug 18306: "Renew selected" and "renew all" in OPAC can be confusing for users not allowed to do it This patch add additional checks to opac-user.pl defining the ability to renew some or all items. The template can now better check whether it should show the renewal controls. To test, apply the patch and log in to the OPAC as a user with multiple checkouts. Test the following conditions: - All checkouts can be renewed - "Renew selected" and "Renew all" should appear at the top and bottom of the table of checkouts. - No checkouts can be renewed - No renewal controls should appear. - Some checkouts can be renewed - "Renew selected" should appear but not "Renew all" Test with various settings of the OverduesBlockRenewing system preference.
(In reply to Owen Leonard from comment #9) > Created attachment 86853 [details] [review] [review] > Bug 18306: "Renew selected" and "renew all" in OPAC can be confusing for > users not allowed to do it > > This patch add additional checks to opac-user.pl defining the ability to > renew some or all items. The template can now better check whether it > should show the renewal controls. > > To test, apply the patch and log in to the OPAC as a user with multiple > checkouts. Test the following conditions: > > - All checkouts can be renewed > - "Renew selected" and "Renew all" should appear at the top and > bottom of the table of checkouts. > - No checkouts can be renewed > - No renewal controls should appear. > - Some checkouts can be renewed > - "Renew selected" should appear but not "Renew all" > > Test with various settings of the OverduesBlockRenewing system > preference. Hi Owen, I tested it on sandbox http://catalogue.user05-koha.sandbox.biblibre.eu/cgi-bin/koha/opac-user.pl and link Renew selected still appears even if all my checkout are not renewable. OverduesBlockRenewing is set on allow renewing Regards, Marjorie
Created attachment 86855 [details] not renewable
(In reply to Marjorie Barry-Vila from comment #10) > even if all my checkout are not renewable. Why are your checkouts not renewable?
(In reply to Owen Leonard from comment #12) > (In reply to Marjorie Barry-Vila from comment #10) > > > even if all my checkout are not renewable. > > Why are your checkouts not renewable? I used all the renewals available according to the circulation rules. Marjorie
I see this too, renew selected appears even if there are no items available to renew. It seems to work correctly when OverduesBlockRenewing is set to block. Sorry Owen. :(