Attachment 101584 Details for Bug 24754 – Bug 24754: Set userenv for ilsdi requests

[patch] Bug 24754: Set userenv for ilsdi requests

Bug-24754-Set-userenv-for-ilsdi-requests.patch (text/plain), 2.44 KB, created by Marcel de Rooy on 2020-03-24 13:58:38 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Marcel de Rooy

Created: 2020-03-24 13:58:38 UTC

Size: 2.44 KB

patch

obsolete

>From f8249b5123f744c5234dce4e56b04e61a7408289 Mon Sep 17 00:00:00 2001
>From: Nick Clemens <nick@bywatersolutions.com>
>Date: Fri, 28 Feb 2020 14:46:37 +0000
>Subject: [PATCH] Bug 24754: Set userenv for ilsdi requests
>Content-Type: text/plain; charset=utf-8
>
>Currently ILSDI scripts do not have a userenv set. This affects both
>action_logs and statistics.
>
>For action_logs we either get no user or, if there is an active session, we use that one. This can have unintended consequences and we should explicitly set the env so that system user (0) is recorded
>
>For statistics we need to specify a branch to record for actions like renewals. With bug 24759 we add an items method for this. We should take advantage of this method and use the 'opac' interface as the ilsdi scripts live there.
>
>To test:
>1 - Enable ILSDI in systempreferences
>2 - Checkout an item to a patron
>3 - Enable Renewal logs
>4 - Ensure you will be able to renew the item several times
>5 - Open an incognito/private browser
>6 - Renew item via ilsdi:
>    http://localhost:8080/cgi-bin/koha/ilsdi.pl?service=RenewLoan&patron_id=5&item_id=1128
>7 - Check action logs, interface may be random, user should be 0, if not make sure the user in step 9 is a different one
>8 - In private browser open a new tab
>9 - Sign in to staff client
>10 - Return to other tab
>11 - Refresh page to repeat renewal
>12 - Check action_logs
>13 - The second renewal indicates it was renewed via intranet by logged in staff member
>14 - Apply patch
>15 - restart_all
>16 - Ensure staff client still signed in
>17 - Repeate renewal again
>18 - Check logs, renewal now indicates opac again (as it should), user is 0 even if logged in on other tab
>
>Signed-off-by: David Nind <david@davidnind.com>
>
>Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
>---
> opac/ilsdi.pl | 8 ++++++++
> 1 file changed, 8 insertions(+)
>
>diff --git a/opac/ilsdi.pl b/opac/ilsdi.pl
>index a3e9566b42..4c0d5a5b51 100755
>--- a/opac/ilsdi.pl
>+++ b/opac/ilsdi.pl
>@@ -127,6 +127,14 @@ unless ( $cgi->param('service') ) {
>     exit 0;
> }
> 
>+# Set the userenv
>+C4::Context->_new_userenv(1);
>+C4::Context->set_userenv(
>+    undef, undef, undef, 'ILSDI', 'ILSDI', undef,
>+    undef, undef, undef, undef,  undef
>+);
>+C4::Context->interface('opac');
>+
> # If user requested a service description, then display it
> if ( scalar $cgi->param('service') eq "Describe" and any { scalar $cgi->param('verb') eq $_ } @services ) {
>     my ( $template, $loggedinuser, $cookie ) = get_template_and_user(
>-- 
>2.11.0

Actions: View | Diff | Splinter Review

Attachments on bug 24754: 99768 | 99772 | 99773 | 99778 | 100000 | 100001 | 100006 | 100035 | 100269 | 100305 | 101094 | 101584 | 101585