Created attachment 107700 [details] opac When a logged in patron "reports a problem", there is a place for the URL of the page the patron is on to be filled in on the line: 'Problem found on page'. Currently, no matter what page on the OPAC, the Report a Problem is clicked on the , the URL does not display. This is what it will display: Problem found on page: / From reading the bug 4461, this should display the URL.
Hi Kelly, can you confirm you have OPACBaseURL set correctly?
Hi Jonathan, yes, I have the OPACbaseURL set up. It is working and directing staff to OPAC via bib records.
Created attachment 107704 [details] [review] Bug 26127: local_referer should not compare with OPACBaseURL case insensitive In our test data we have OPACBaseURL that is filled with http://kohadev.myDNSname.org:8080, but the CGI referer in local_referer is lowercase We should compare it without case sensitivity
Can you test this patch? It's just a guess but I do recreate on koha-testing-docker and it fixes the problem.
This is still not working when testing in Sandboxes. Adding screenshots.
Created attachment 107710 [details] url
I could see this problem when my OpacBaseURL system preference didn't match my actual OPAC URL. In my case it was because the OpacBaseURL from the sample data doesn't match my test environment. Is the same true of the sandboxes?
Sandboxes should have the same case sensitive problem my patch is supposed to fix.
Created attachment 107744 [details] [review] Bug 26127: local_referer should not compare with OPACBaseURL case insensitive In our test data we have OPACBaseURL that is filled with http://kohadev.myDNSname.org:8080, but the CGI referer in local_referer is lowercase We should compare it without case sensitivity Test plan: - Use koha-testing-docker - Notice the value of OPACBaseURL => http://kohadev.myDNSname.org:8080 - Use the "reports a problem" feature at the OPAC - Notice that the "Problem found on page" is correctly filled with the referer
The original problem is also due to an issue in koha-sandboxes-docker, reported at https://github.com/bywatersolutions/koha-sandboxes-docker/issues/37
Created attachment 108094 [details] [review] Bug 26127: local_referer should not compare with OPACBaseURL case insensitive In our test data we have OPACBaseURL that is filled with http://kohadev.myDNSname.org:8080, but the CGI referer in local_referer is lowercase We should compare it without case sensitivity Test plan: - Use koha-testing-docker - Notice the value of OPACBaseURL => http://kohadev.myDNSname.org:8080 - Use the "reports a problem" feature at the OPAC - Notice that the "Problem found on page" is correctly filled with the referer Signed-off-by: Kelly McElligott <kelly@bywatersolutions.com>
Created attachment 108317 [details] [review] Bug 26127: local_referer should not compare with OPACBaseURL case insensitive In our test data we have OPACBaseURL that is filled with http://kohadev.myDNSname.org:8080, but the CGI referer in local_referer is lowercase We should compare it without case sensitivity Test plan: - Use koha-testing-docker - Notice the value of OPACBaseURL => http://kohadev.myDNSname.org:8080 - Use the "reports a problem" feature at the OPAC - Notice that the "Problem found on page" is correctly filled with the referer Signed-off-by: Kelly McElligott <kelly@bywatersolutions.com> Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Pushed to master for 20.11, thanks to everybody involved!
backported to 20.05.x for 20.05.04
missing dependencies, not backported to 19.11.x
*** Bug 27468 has been marked as a duplicate of this bug. ***