From ec02ab6f68efa15b79a0892de324fc45259a3f98 Mon Sep 17 00:00:00 2001 From: Sean Harvey Date: Sun, 25 Mar 2012 20:38:29 +1300 Subject: [PATCH] BUGFIX Fixed FulltextSearchable checking DB::getConn() unncessarily for MySQL, as Database::requireTable() will only apply create_table_options to the database adapter they're set for. Replace Object::add_static_var usage (deprecated) with the new config system instead. --- search/FulltextSearchable.php | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/search/FulltextSearchable.php b/search/FulltextSearchable.php index 07b96423a..a69c13436 100644 --- a/search/FulltextSearchable.php +++ b/search/FulltextSearchable.php @@ -52,9 +52,7 @@ class FulltextSearchable extends DataExtension { if(!class_exists($class)) continue; if(isset($defaultColumns[$class])) { - if(DB::getConn()->getDatabaseServer() == 'mysql') { - Object::add_static_var($class, 'create_table_options', array('MySQLDatabase' => 'ENGINE=MyISAM'), true); - } + Config::inst()->update($class, 'create_table_options', array('MySQLDatabase' => 'ENGINE=MyISAM')); Object::add_extension($class, "FulltextSearchable('{$defaultColumns[$class]}')"); } else { throw new Exception("FulltextSearchable::enable() I don't know the default search columns for class '$class'");