Summary: | have mysql returning errors | ||
---|---|---|---|
Product: | Koha | Reporter: | Paul Poulain <paul.poulain> |
Component: | Architecture, internals, and plumbing | Assignee: | Paul Poulain <paul.poulain> |
Status: | CLOSED FIXED | QA Contact: | Ian Walls <koha.sekjal> |
Severity: | enhancement | ||
Priority: | PATCH-Sent (DO NOT USE) | ||
Version: | 3.8 | ||
Hardware: | All | ||
OS: | All | ||
GIT URL: | Change sponsored?: | --- | |
Patch complexity: | --- | Documentation contact: | |
Documentation submission: | Text to go in the release notes: | ||
Version(s) released in: | Circulation function: | ||
Bug Depends on: | |||
Bug Blocks: | 7119 | ||
Attachments: | Bug 7184 raise mySQL error if debug is set |
Description
Paul Poulain
2011-11-06 06:44:14 UTC
silly me (& ian) : ->preference needs a db handler, that is not still open, so it calls new_dbh, that need ->preference ... => forever loop => boom ! We will have to use DEBUG pragma (in koha-httpd.conf) Patch coming soon ! Created attachment 6238 [details] [review] Bug 7184 raise mySQL error if debug is set Before this patch, if a mySQL occured in Koha (any error), then the user was seeing nothing. With this patch, if DEBUG is set in Koha VirtualHost, any mySQL error will make Koha die and display the SQL error Step to test: === BEFORE PATCH === go anywhere, and change a mySQL statement to make it invalid. Launch a page using the SQL you've made wrong, you'll see nothing, except you won't see the expected results === AFTER PATCH === Modify koha-httpd.conf, and add the following line, in the virtualhost (if it does not already exist) SetEnv DEBUG 1 restart/reload Apache Launch the same page, you'll get a Perl statement saying something like: DBD::mysql::st execute failed: <<blablabla SQL error>> at <<where the error was raised line <<error line>> Simple patch, implements more feedback information using the Environment variable instead of the system preference. Followup will become necessary: modify etc/koha-httpd.conf to include default values for the DEBUG environment variable. The choice of this default value should be consistent with the level of debugging already defaulted to in Koha with the syspref. This will come in very handy whenever atomic update files are not applied along with their companion code; a missing column or system preference value will become very clear very quickly. Marking as Passed QA patch pushed, please test. QMaint: I think this patch should be kept for 3.8: if it's activated on 3.6, there may be unexpected errors. I'll propose on koha-devel to have all tester setting DEBUG to 1, but I think it should not be done for 3.6 yet. There have been no further reports of problems so I am marking this bug resolved. |