fix(condition): allow longer texts

btry · btry · commit eecdf8a2a8f6 · 2023-01-02T13:35:11.000+01:00
diff --git a/install/mysql/plugin_formcreator_2.13.4_empty.sql b/install/mysql/plugin_formcreator_2.13.4_empty.sql
@@ -149,7 +149,7 @@ CREATE TABLE IF NOT EXISTS `glpi_plugin_formcreator_conditions` (
 	`items_id`                        int unsigned NOT NULL DEFAULT '0' COMMENT 'item ID of the item affected by the condition',
 	`plugin_formcreator_questions_id` int unsigned NOT NULL DEFAULT '0' COMMENT 'question to test for the condition',
 	`show_condition`                  int(11)      NOT NULL DEFAULT '0',
-	`show_value`                      varchar(255) NULL DEFAULT NULL,
+	`show_value`                      mediumtext   NULL DEFAULT NULL,
 	`show_logic`                      int(11)      NOT NULL DEFAULT '1',
 	`order`                           int(11)      NOT NULL DEFAULT '1',
 	`uuid`                            varchar(255) NULL DEFAULT NULL,
diff --git a/install/upgrade_to_2.13.4.php b/install/upgrade_to_2.13.4.php
@@ -41,6 +41,7 @@ public function isResyncIssuesRequired() {
     */
    public function upgrade(Migration $migration) {
       $this->migration = $migration;
+      $this->updateConditions();
       $this->addServiceCatalogHopepage();
    }
 
@@ -53,4 +54,8 @@ public function addServiceCatalogHopepage() {
          'condition' => 'WHERE `entities_id` = 0'
       ]);
    }
+
+   public function updateConditions() {
+      $this->migration->changeField('glpi_plugin_formcreator_conditions', 'show_value', 'show_value', 'mediumtext');
+   }
 }

Original file line number	Diff line number	Diff line change
`@@ -41,6 +41,7 @@ public function isResyncIssuesRequired() {`
`41`	`41`	`*/`
`42`	`42`	`public function upgrade(Migration $migration) {`
`43`	`43`	`$this->migration = $migration;`
	`44`	`+ $this->updateConditions();`
`44`	`45`	`$this->addServiceCatalogHopepage();`
`45`	`46`	`}`
`46`	`47`
`@@ -53,4 +54,8 @@ public function addServiceCatalogHopepage() {`
`53`	`54`	'condition' => 'WHERE `entities_id` = 0'
`54`	`55`	`]);`
`55`	`56`	`}`
	`57`	`+`
	`58`	`+ public function updateConditions() {`
	`59`	`+ $this->migration->changeField('glpi_plugin_formcreator_conditions', 'show_value', 'show_value', 'mediumtext');`
	`60`	`+ }`
`56`	`61`	`}`