Merge pull request #25174 from seamuslee001/ref_create_alter_column_f…

…unction [REF] Create Alter Column function in upgrade base class
civicrm · Dec 16, 2022 · e9e87cd · e9e87cd
2 parents 59d1d50 + f2cbacf
commit e9e87cd
Show file tree

Hide file tree

Showing 2 changed files with 18 additions and 9 deletions.
diff --git a/CRM/Upgrade/Incremental/Base.php b/CRM/Upgrade/Incremental/Base.php
@@ -585,4 +585,21 @@ public static function rebuildMultilingalSchema($ctx, $version = NULL) {
     return TRUE;
   }
 
+  public static function alterColumn($ctx, $table, $column, $properties, $localizable = FALSE): bool {
+    $locales = CRM_Core_I18n::getMultilingual();
+    $queries = [];
+    if ($localizable && $locales) {
+      foreach ($locales as $locale) {
+        $queries[] = "ALTER TABLE `$table` CHANGE `{$column}_{$locale}` `{$column}_{$locale}` $properties";
+      }
+    }
+    else {
+      $queries[] = "ALTER TABLE `$table` CHANGE `$column` `$column` $properties";
+    }
+    foreach ($queries as $query) {
+      CRM_Core_DAO::executeQuery($query, [], TRUE, NULL, FALSE, FALSE);
+    }
+    return TRUE;
+  }
+
 }
diff --git a/CRM/Upgrade/Incremental/php/FiveFiftyEight.php b/CRM/Upgrade/Incremental/php/FiveFiftyEight.php
@@ -33,15 +33,7 @@ public function upgrade_5_58_alpha1($rev): void {
   }
 
   public static function addOptionGroupDescriptions($ctx): bool {
-    $locales = CRM_Core_I18n::getMultilingual();
-    if ($locales) {
-      foreach ($locales as $locale) {
-        CRM_Core_DAO::executeQuery("ALTER TABLE `civicrm_option_group` CHANGE `description_{$locale}` `description_{$locale}` TEXT COMMENT 'Option group description.'", [], TRUE, NULL, FALSE, FALSE);
-      }
-    }
-    else {
-      CRM_Core_DAO::executeQuery('ALTER TABLE `civicrm_option_group` MODIFY COLUMN `description` TEXT', [], TRUE, NULL, FALSE, FALSE);
-    }
+    CRM_Upgrade_Incremental_Base::alterColumn($ctx, 'civicrm_option_group', 'description', "TEXT COMMENT 'Option group description.'", TRUE);
     $values = [
       [
         'group' => 'gender',