Bug 11891: (follow-up) set DBIC storage flags correctly

Since we're passing an existing DBI database handle to
DBIC, and that handle doesn't have RaiseError set to true
by default, don't let DBIC override that -- for now.

Test plan: verify that the DB-dependent test suite passes.

Signed-off-by: Galen Charlton <gmc@esilibrary.com>
Signed-off-by: Chris Cormack <chrisc@catalyst.net.nz>
Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>

diff --git a/Koha/Database.pm b/Koha/Database.pm
index 99888c6..92e83ec 100644 (file)
--- a/Koha/Database.pm
+++ b/Koha/Database.pm
@@ -45,7 +45,12 @@ __PACKAGE__->mk_accessors(qw( ));
 # database connection from the data given in the current context, and
 # returns it.
 sub _new_schema {
-    my $schema = Koha::Schema->connect( sub { C4::Context->dbh } );
+
+    my $context = C4::Context->new();
+
+    # we are letting C4::Context->dbh not set the RaiseError handle attribute
+    # for now for compatbility purposes
+    my $schema = Koha::Schema->connect( sub { C4::Context->dbh }, { unsafe => 1 } );
     return $schema;
 }