UpdateDatabase/MysqlLinks.php

[Pman.Core] / UpdateDatabase / MysqlLinks.php

diff --git a/UpdateDatabase/MysqlLinks.php b/UpdateDatabase/MysqlLinks.php
index a506ac2..4af23dc 100644 (file)
--- a/UpdateDatabase/MysqlLinks.php
+++ b/UpdateDatabase/MysqlLinks.php
@@ -23,8 +23,17 @@
  *   -- {tablename}_trigger_{optional_string}_before_update_{column_name}(OLD.column, NEW.column}
  *   -- {tablename}_trigger_{optional_string}_before_insert_{column_name}(OLD.column}
  *
- *  
  *
+ * ------- Importing with triggers disabled.
+ *
+ *  SET @DISABLE_TRIGGER=1; (or anything you like except NULL) 
+ *  do imports
+ * SET @DISABLE_TRIGGER=NULL;
+ *
+ * ------ Call a method disabling a particular set of triggers
+ *  SET @DISABLE_TRIGGER_the_table_name=1; (or anything you like except NULL) 
+ *  do action
+ *  SET @DISABLE_TRIGGER_the_table_name=NULL;*
  */
 
 class Pman_Core_UpdateDatabase_MysqlLinks {
@@ -38,6 +47,10 @@ class Pman_Core_UpdateDatabase_MysqlLinks {
           
         $this->loadIniFiles();
         $this->updateTableComments();
+        
+        $this->updateCharacterSet();
+        $this->updateEngine();
+        
         $ff = HTML_FlexyFramework::get();
         if (!empty($ff->Pman['enable_trigger_tests'])) {
             
@@ -115,18 +128,11 @@ class Pman_Core_UpdateDatabase_MysqlLinks {
         foreach($map as $k=>$v) {
             $fks[] = "$k=$v";
         }
-        
         $fkstr = $tc . ' FK(' . implode("\n", $fks) .')';
-        
-        echo "$fkstr \n $tbl";
-        
-        exit;
         if ($q->TABLE_COMMENT == $fkstr) {
             return;
         }
         
-        
-        
         $q = DB_DAtaObject::factory('core_enum');
         $q->query("ALTER TABLE $tbl COMMENT = '{$q->escape($fkstr)}'");
         
@@ -192,8 +198,8 @@ class Pman_Core_UpdateDatabase_MysqlLinks {
                 BEFORE DELETE ON `{$target_table}`
             FOR EACH ROW
             BEGIN
-               DECLARE mid INT(11);
-             
+                DECLARE mid INT(11);
+                IF (@DISABLE_TRIGGER IS NULL AND @DISABLE_TRIGGER_{$target_table} IS NULL ) THEN  
                
             ";
             foreach($sources as $source=>$target) {
@@ -219,6 +225,7 @@ class Pman_Core_UpdateDatabase_MysqlLinks {
             }
             
             $trigger .= "
+                END IF;
             END 
            
             ";
@@ -255,7 +262,7 @@ class Pman_Core_UpdateDatabase_MysqlLinks {
             FOR EACH ROW
             BEGIN
                DECLARE mid INT(11);
-               
+                IF (@DISABLE_TRIGGER IS NULL AND @DISABLE_TRIGGER_{$tbl} IS NULL ) THEN 
                
             ";
             foreach($map as $source_col=>$target) {
@@ -297,6 +304,7 @@ class Pman_Core_UpdateDatabase_MysqlLinks {
             }
             
             $trigger .= "
+                END IF;
             END 
            
             ";
@@ -336,7 +344,7 @@ class Pman_Core_UpdateDatabase_MysqlLinks {
             FOR EACH ROW
             BEGIN
                DECLARE mid INT(11);
-               
+               IF (@DISABLE_TRIGGER IS NULL AND @DISABLE_TRIGGER_{$tbl} IS NULL ) THEN  
                
             ";
             foreach($map as $source_col=>$target) {
@@ -374,6 +382,7 @@ class Pman_Core_UpdateDatabase_MysqlLinks {
             }
             
             $trigger .= "
+                END IF;
             END 
            
             ";
@@ -439,6 +448,73 @@ class Pman_Core_UpdateDatabase_MysqlLinks {
         return $ret;
     }
         
+    function updateCharacterSet()
+    {
+        foreach (array_keys($this->schema) as $tbl){
+            
+            if(strpos($tbl, '__keys') !== false ){
+                continue;
+            }
+            
+            $ce = DB_DataObject::factory('core_enum');
+            
+            $ce->query("
+                SELECT
+                        CCSA.character_set_name csname,
+                        CCSA.collation_name collatename
+                FROM
+                        information_schema.`TABLES` T,
+                        information_schema.`COLLATION_CHARACTER_SET_APPLICABILITY` CCSA
+                WHERE
+                        CCSA.collation_name = T.table_collation
+                    AND
+                        T.table_schema = '{$ce->database()}' -- COLLATE utf8_general_ci
+                    AND
+                        T.table_name = '{$tbl}' -- COLLATE utf8_general_ci
+            ");
+                     
+            $ce->fetch();
+            
+            if($ce->csname == 'utf8' && $ce->collatename == 'utf8_general_ci'){
+                echo "$tbl is Already utf8 \n";
+                continue;
+            }
+            // this used to be utf8_unicode_ci
+            //as the default collation for stored procedure parameters is utf8_general_ci and you can't mix collations.
+            
+            $ce = DB_DataObject::factory('core_enum');
+            $ce->query("ALTER TABLE {$tbl} CONVERT TO CHARACTER SET  utf8 COLLATE utf8_general_ci");
+            echo "FIXED utf8 on {$tbl}\n";
+            
+        }
+    }
+    function updateEngine()
+    {
+        foreach (array_keys($this->schema) as $tbl){
+            
+            if(strpos($tbl, '__keys') !== false ){
+                continue;
+            }
+            
+            $ce = DB_DataObject::factory('core_enum');
+            
+            $ce->query("select engine from information_schema.tables where table_schema='hydra' and table_name = 'core_enum'");
+
+            $ce->fetch();
+            
+            if($ce->engine == 'InnoDB' ){
+                echo "SKIP engine on $tbl - already InnoDB\n";
+                continue;
+            }
+            // this used to be utf8_unicode_ci
+            //as the default collation for stored procedure parameters is utf8_general_ci and you can't mix collations.
+            
+            $ce = DB_DataObject::factory('core_enum');
+            $ce->query("ALTER TABLE $tbl ENGINE=InnoDB");
+            echo "FIXED engine on {$tbl}\n";
+            
+        }
+    }
     
 }