Summary: | Patron Modification log redirects to circulation page | ||
---|---|---|---|
Product: | Koha | Reporter: | Abdou Yousuf <ayousef> |
Component: | Patrons | Assignee: | Jonathan Druart <jonathan.druart> |
Status: | CLOSED FIXED | QA Contact: | Testopia <testopia> |
Severity: | normal | ||
Priority: | P5 - low | CC: | 1joynelson, gmcharlt, jonathan.druart, josef.moravec, kyle.m.hall, lucas, martin.renvoize |
Version: | Main | ||
Hardware: | All | ||
OS: | All | ||
See Also: | https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21242 | ||
Change sponsored?: | --- | Patch complexity: | Trivial patch |
Documentation contact: | Documentation submission: | ||
Text to go in the release notes: | Version(s) released in: |
20.05.00, 19.11.04, 19.05.09
|
|
Circulation function: | |||
Bug Depends on: | 19080 | ||
Bug Blocks: | 25249 | ||
Attachments: |
Bug 19791: Keep user in circ when viewing patron's logs
Bug 19791: Keep user in circ when viewing patron's logs Bug 19791: Keep user in circ when viewing patron's logs |
Description
Abdou Yousuf
2017-12-11 10:54:05 UTC
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 |