diff options
author | Dylan William Hardison <dylan@hardison.net> | 2018-09-13 22:37:55 +0200 |
---|---|---|
committer | GitHub <noreply@github.com> | 2018-09-13 22:37:55 +0200 |
commit | d3e00529333fb2eadac9c22a45f20c1050401952 (patch) | |
tree | 85634ffdbd491ea1fe7fcbcc68ad416f8904caf1 /xt | |
parent | 5ab05f16aaa44042666c69065dc1c7576eb0de51 (diff) | |
download | bugzilla-d3e00529333fb2eadac9c22a45f20c1050401952.tar.gz bugzilla-d3e00529333fb2eadac9c22a45f20c1050401952.tar.xz |
Bug 1490708 - Ensure we always call DBIx::Connector->dbh before any DBI method (#744)
The code didn't allow a way of doing this without a lot of work.
So I had to take the following approach:
The 'dbh' attribute is now a method that delegates to DBIx::Connector's dbh
method. Per the docs, ->dbh() "Returns the connection's database handle. It will
use a an existing handle if there is one, if the process has not been forked or
a new thread spawned, and if the database is pingable. Otherwise, it will
instantiate, cache, and return a new handle."
Then there is the matter of the 'handles' on dbh. I've used Package::Stash to
insert proxy methods into the class when it is loaded.
Diffstat (limited to 'xt')
0 files changed, 0 insertions, 0 deletions