Bugzilla – Attachment 179298 Details for
Bug 39307
console.error on circ/circulation.pl page
Home
|
New
|
Browse
|
Search
|
[?]
|
Reports
|
Help
|
New Account
|
Log In
[x]
|
Forgot Password
Login:
[x]
[patch]
Bug 39307: Check to make sure there is a document.referrer
Bug-39307-Check-to-make-sure-there-is-a-documentre.patch (text/plain), 2.02 KB, created by
Phil Ringnalda
on 2025-03-14 01:16:04 UTC
(
hide
)
Description:
Bug 39307: Check to make sure there is a document.referrer
Filename:
MIME Type:
Creator:
Phil Ringnalda
Created:
2025-03-14 01:16:04 UTC
Size:
2.02 KB
patch
obsolete
>From c70898c3f54f7858d177284a67a1c7418a7c87d9 Mon Sep 17 00:00:00 2001 >From: Lucas Gass <lucas@bywatersolutions.com> >Date: Tue, 11 Mar 2025 16:53:28 +0000 >Subject: [PATCH] Bug 39307: Check to make sure there is a document.referrer > >To test: >1. Navigate to circ/circulation.pl?borrowernumber=X >2. See the console error `document.referrer.split(...)[1] is undefined` >3. APPLY PATCH >4. JS error should be gone > >Signed-off-by: Phil Ringnalda <phil@chetcolibrary.org> >--- > .../prog/en/modules/circ/circulation.tt | 13 +++++++------ > 1 file changed, 7 insertions(+), 6 deletions(-) > >diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/circ/circulation.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/circ/circulation.tt >index 66cb852175..160a5a8ea4 100644 >--- a/koha-tmpl/intranet-tmpl/prog/en/modules/circ/circulation.tt >+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/circ/circulation.tt >@@ -1181,13 +1181,14 @@ > > // Handle checkout for fast cataloging > // Check the referrer to prevent csrf, fill and submit form >- if(document.referrer.split(window.location.origin + '/cgi-bin/koha/cataloguing/additem.pl?')[1].match(/frameworkcode=FA&circborrowernumber=/) ) { >- let urlParams = new URLSearchParams(window.location.search); >- let barcode = urlParams.get('barcode'); >- $('#barcode').val(barcode); >- $('#mainform').submit(); >+ if ( document.referrer.length ) { >+ if(document.referrer.split(window.location.origin + '/cgi-bin/koha/cataloguing/additem.pl?')[1].match(/frameworkcode=FA&circborrowernumber=/) ) { >+ let urlParams = new URLSearchParams(window.location.search); >+ let barcode = urlParams.get('barcode'); >+ $('#barcode').val(barcode); >+ $('#mainform').submit(); >+ } > } >- > $("#cancellation-reason").comboBox({ > displayProperty: 'name', > placeholder: _("Select or type a reason"), >-- >2.48.1
You cannot view the attachment while viewing its details because your browser does not support IFRAMEs.
View the attachment on a separate page
.
View Attachment As Diff
View Attachment As Raw
Actions:
View
|
Diff
|
Splinter Review
Attachments on
bug 39307
:
179173
|
179298
|
179638
|
179701
|
179702
|
179707