Dear all, - Showing patron Modification log from koha 17.11 doesn't show any system log for the patron, Instead it redirects to (http://domain_name/cgi-bin/koha/circ/circulation.pl?borrowernumber=) - Just try filling the (Librarian:) field with the borrower number instead of the (Object:) field, And leave the (Object:)field empty, You will see what I mean. - Also you can test it here (http://koha.adminkuhn.ch:8080/cgi-bin/koha/tools/viewlog.pl?do_it=1&modules=MEMBERS&modules=circulation&object=1&src=circ), It's a demo site for koha 17.11 provided by koha community here (https://koha-community.org/demo/) use (demo/demo) for logging in, Then test it for this user (Herr Demokrit Morgenstern (1)), Also the problem was not in koha 17.05, You can test it using Bywater demo version (koha17.05), You will see that it working fine in koha (17.05).
With bug 19080 we tried to redirect to the circulation home page when the borrowernumber did not exist (anymore). When you click on the "modification log" tab, the "object" parameter is automatically filled with the borrowernumber of the patron you are looking at. If you decide to empty this field or use a non-existent borrowernumber, you will get redirected to the circulation home page. I would suggest to: - Disable the "object" input and force the value to borrowernumber when we come from the circulation module and redirect to circ home page if non-existent - Make it editable when come from the tools module and display a "no logs" message if non-existent borrowernumber is used.
(In reply to Jonathan Druart from comment #1) > With bug 19080 we tried to redirect to the circulation home page when the > borrowernumber did not exist (anymore). > > When you click on the "modification log" tab, the "object" parameter is > automatically filled with the borrowernumber of the patron you are looking > at. > > If you decide to empty this field or use a non-existent borrowernumber, you > will get redirected to the circulation home page. > > I would suggest to: > - Disable the "object" input and force the value to borrowernumber when we > come from the circulation module and redirect to circ home page if > non-existent > - Make it editable when come from the tools module and display a "no logs" > message if non-existent borrowernumber is used. I don't think that it's beneficial to automatically fill (Object field) with the borrower number, On the contrary I think that the (Librarian:) field should be automatically filled by borrower number rather than (Object field), And it will be more beneficial, In fact I don't know why borrower number should be in (Object field) and not (Librarian field) when selecting "modification log" tab, What are the benefits? Also, You said that you tried to redirect to the circulation home page when the borrower number did not exist (anymore), How it can be? I think it's not logical even though it can be done programmatically. Regards.
(In reply to Abdou Yousuf from comment #2) > (In reply to Jonathan Druart from comment #1) > > With bug 19080 we tried to redirect to the circulation home page when the > > borrowernumber did not exist (anymore). > > > > When you click on the "modification log" tab, the "object" parameter is > > automatically filled with the borrowernumber of the patron you are looking > > at. > > > > If you decide to empty this field or use a non-existent borrowernumber, you > > will get redirected to the circulation home page. > > > > I would suggest to: > > - Disable the "object" input and force the value to borrowernumber when we > > come from the circulation module and redirect to circ home page if > > non-existent > > - Make it editable when come from the tools module and display a "no logs" > > message if non-existent borrowernumber is used. > > I don't think that it's beneficial to automatically fill (Object field) with > the borrower number, On the contrary I think that the (Librarian:) field > should be automatically filled by borrower number rather than (Object > field), And it will be more beneficial, In fact I don't know why borrower > number should be in (Object field) and not (Librarian field) when selecting > "modification log" tab, What are the benefits? The tabs on patron pages are all relevant to current patron, that is the reason why borrowernumber is in object - you want here to get information about this patron. > Also, You said that you tried to redirect to the circulation home page when > the > borrower number did not exist (anymore), How it can be? I think it's not > logical even though it can be done programmatically. It could be possible - for example one librarian deletes a patron and the second librarian wants to show some information about patron. > > Regards.
> I would suggest to: > - Disable the "object" input and force the value to borrowernumber when we > come from the circulation module and redirect to circ home page if > non-existent > - Make it editable when come from the tools module and display a "no logs" > message if non-existent borrowernumber is used. That would work for me. Also agree with Josef that using the borrowernumber of the patron in object is the expected behaviour.
Created attachment 98922 [details] [review] Bug 19791: Keep user in circ when viewing patron's logs When a librarian is viewing the log for a given patron (tab "modification log" from the circulation module), the "object" and "modules" parameter can be modified. It does not make sense, we should stay in the circulation module and avoid those fields to be modified. Test plan: - Modify several patrons (in order to have several modification logs) - Go to the modification log tab of one of them - Notice that the "Object" input is not a "Borrowernumber" labelled input and that you cannot modify it. Also notice that the "modules" is hidden. - Go to the log viewer tool and notice that the view is still the same as before this patch.
Created attachment 98967 [details] [review] Bug 19791: Keep user in circ when viewing patron's logs When a librarian is viewing the log for a given patron (tab "modification log" from the circulation module), the "object" and "modules" parameter can be modified. It does not make sense, we should stay in the circulation module and avoid those fields to be modified. Test plan: - Modify several patrons (in order to have several modification logs) - Go to the modification log tab of one of them - Notice that the "Object" input is not a "Borrowernumber" labelled input and that you cannot modify it. Also notice that the "modules" is hidden. - Go to the log viewer tool and notice that the view is still the same as before this patch.
Created attachment 99220 [details] [review] Bug 19791: Keep user in circ when viewing patron's logs When a librarian is viewing the log for a given patron (tab "modification log" from the circulation module), the "object" and "modules" parameter can be modified. It does not make sense, we should stay in the circulation module and avoid those fields to be modified. Test plan: - Modify several patrons (in order to have several modification logs) - Go to the modification log tab of one of them - Notice that the "Object" input is not a "Borrowernumber" labelled input and that you cannot modify it. Also notice that the "modules" is hidden. - Go to the log viewer tool and notice that the view is still the same as before this patch. Signed-off-by: Owen Leonard <oleonard@myacpl.org> Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Works as expected, no regressions found and QA scripts pass.. Passing QA
Nice work everyone! Pushed to master for 20.05
Pushed to 19.11.x branch for 19.11.04
backported to 19.05.x for 19.05.09