Created attachment 114431 [details] [review] Bug 27246: Remove obsolete BEGIN code from C4::Context We don't longer enter the HTTP_USER_AGENT conditional in the BEGIN block of C4::Context. It was used when apache only was used. I am suggesting to remove it but keep the $CGI::VERSION < 4.08 condition to keep support of all version (not sure it's needed but won't hurt). I have not found a lot of occurrences of KOHA_BACKTRACES and I am assuming not much people are using it, we should prefer the dev_install flag instead (to set environment="development" when plack is started and enabled the starman trace) Test plan: Regression test what you think is useful Confirm the above statements
*** Bug 8389 has been marked as a duplicate of this bug. ***
Not worth a signoff but here is some random testing: applied patch restarted services (in koha-testing-docker) ran the GUI tests: prove -r t/db_dependent/selenium/ went to OPAC went to staff interface did a checkout, showed it in the patrons page (to test the related API call) restarted koha-testing-docker to see an eventual difference compared to applying when already started
Thumbs up for removing all that stuff
dcook, any hint of what I should test?
(In reply to Victor Grousset/tuxayo from comment #5) > dcook, any hint of what I should test? Honestly, I don't think that functionality has worked in a long time if ever. There's not really much to test except making sure that no typo was introduced, so just loading Koha is probably sufficient. Making sure C4::Context compiles.
Created attachment 120346 [details] [review] Bug 27246: Remove obsolete BEGIN code from C4::Context We don't longer enter the HTTP_USER_AGENT conditional in the BEGIN block of C4::Context. It was used when apache only was used. I am suggesting to remove it but keep the $CGI::VERSION < 4.08 condition to keep support of all version (not sure it's needed but won't hurt). I have not found a lot of occurrences of KOHA_BACKTRACES and I am assuming not much people are using it, we should prefer the dev_install flag instead (to set environment="development" when plack is started and enabled the starman trace) Test plan: Regression test what you think is useful Confirm the above statements Signed-off-by: Victor Grousset/tuxayo <victor@tuxayo.net> Signed-off-by: David Cook <dcook@prosentient.com.au>
Then we can sign it off :) Including David in it for the analysis ^^
Created attachment 120436 [details] [review] Bug 27246: Remove obsolete BEGIN code from C4::Context We don't longer enter the HTTP_USER_AGENT conditional in the BEGIN block of C4::Context. It was used when apache only was used. I am suggesting to remove it but keep the $CGI::VERSION < 4.08 condition to keep support of all version (not sure it's needed but won't hurt). I have not found a lot of occurrences of KOHA_BACKTRACES and I am assuming not much people are using it, we should prefer the dev_install flag instead (to set environment="development" when plack is started and enabled the starman trace) Test plan: Regression test what you think is useful Confirm the above statements Signed-off-by: Victor Grousset/tuxayo <victor@tuxayo.net> Signed-off-by: David Cook <dcook@prosentient.com.au> Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Nice bit of cleanup, no regressions spotted and I agree with David's conclusion that I don't think it's been used for quite some time. Passing QA
Pushed to master for 21.05, thanks to everybody involved!
Enhancement not pushed to 20.11.x