https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=43086 --- Comment #1 from Olivia Reynolds <olivia.reynolds@openfifth.co.uk> --- Created attachment 202006 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=202006&action=edit Bug 43086: Prefill unauthenticated ILL request patron information This patch uses the browser's sessionStorage to save the unauthenticated patron information from ILL request creation. This saves users time if they wish to make multiple requests. Test plan: 1. Enable the system preferences "ILLModule" and "ILLOpacUnauthenticatedRequest". 2. In a logged-out OPAC window, navigate to /cgi-bin/koha/opac-illrequests.pl?op=create 3. Choose the "Standard" provider for the new ILL request. 4. Make any valid request, noting the name, email address, and chosen library used. 5. In the same OPAC session, navigate again to /cgi-bin/koha/opac-illrequests.pl?op=create 6. Choose the "Standard" provider for the new ILL request. 7. Verify that the name, email address, and chosen library are automatically filled with the same values used before. 8. Change the name to a different value. 9. Change the type of request, causing a page load. 10. Verify the changed name is retained, rather than prefilling an incorrect one. Sponsored-by: NHS England -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=43086 David Nind <david@davidnind.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Needs Signoff |Signed Off -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=43086 --- Comment #2 from David Nind <david@davidnind.com> --- Created attachment 203393 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=203393&action=edit Bug 43086: Prefill unauthenticated ILL request patron information This patch uses the browser's sessionStorage to save the unauthenticated patron information from ILL request creation. This saves users time if they wish to make multiple requests. Test plan: 1. Enable the system preferences "ILLModule" and "ILLOpacUnauthenticatedRequest". 2. In a logged-out OPAC window, navigate to /cgi-bin/koha/opac-illrequests.pl?op=create 3. Choose the "Standard" provider for the new ILL request. 4. Make any valid request, noting the name, email address, and chosen library used. 5. In the same OPAC session, navigate again to /cgi-bin/koha/opac-illrequests.pl?op=create 6. Choose the "Standard" provider for the new ILL request. 7. Verify that the name, email address, and chosen library are automatically filled with the same values used before. 8. Change the name to a different value. 9. Change the type of request, causing a page load. 10. Verify the changed name is retained, rather than prefilling an incorrect one. Sponsored-by: NHS England Signed-off-by: David Nind <david@davidnind.com> -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=43086 Katrin Fischer <katrin.fischer@bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Attachment #203393|0 |1 is obsolete| | -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=43086 Katrin Fischer <katrin.fischer@bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |dcook@prosentient.com.au --- Comment #4 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- Hi Olivia, seeing this one assigned to me by our awesome QAM, I was a bit worried if I would remember how to test the unauthenticated requests, so thank you for the great test plan :) I understand the use case and that it could be quite helpful to users, but I wonder about the implications if this was used on a publicly available computer in the library. I am pondering if storing the patron's name and email plain text in the user's session is the best way to go here or if there could be data privacy issues, especially if the computer used was a public one. Maybe it could be remedied by a "remember me" checkbox or similar so it was an opt-in thing? We could also add a hint about not using it in a public environment. There is also no way to make it forget your credentials apart from closing the browser, (which people might not be aware of) if I understand this correctly. I am adding a sign-off for it working as intended an David Cook for another opinion on implementation. Another finding for a new bug, but maybe interesting for this use case as well: Once I have added my first request, I am stuck on the confirmation screen. A button to "place another request" might be helpful here. :) -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=43086 --- Comment #5 from Olivia Reynolds <olivia.reynolds@openfifth.co.uk> --- Thanks for taking a look Katrin, and glad the test plan was useful. There's still so much of Koha I have no idea about so I try to make them very step-by-step. Truthfully, I hadn't considered that the unauthenticated OPAC could be used on a shared session. In that case, I agree that we should avoid remembering user details by default. I'll get in touch with the sponsor and see what they think about that issue, if they'd be happy to make it opt-in, or whether we could do a system preference for this. Otherwise, this feature may best belong in a plugin. Looking at the OPAC form, none of these fields seem to be marked as `autocomplete="off"`, so I think the browser could save their values automatically and present them to future users. It might be worth disabling autocomplete on them whilst we're looking at this area. -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=43086 Olivia Reynolds <olivia.reynolds@openfifth.co.uk> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|Failed QA |Needs Signoff --- Comment #7 from Olivia Reynolds <olivia.reynolds@openfifth.co.uk> --- After talking with the sponsor, they're happy to have an opt-in checkbox for saving the unauthenticated patron details. I've submitted a patch to add that to the form. -- You are receiving this mail because: You are watching all bug changes.

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=43086 Lisette Scheer <lisette@bywatersolutions.com> changed: What |Removed |Added ---------------------------------------------------------------------------- QA Contact| |testopia@bugs.koha-communit | |y.org -- You are receiving this mail because: You are watching all bug changes.