fix(section): duplication of conditions: bad itemtype

Signed-off-by: Thierry Bugier <tbugier@teclib.com>
pluginsGLPI · Jan 28, 2020 · cfcbea3 · cfcbea3
1 parent 73fca7b
commit cfcbea3
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 7 deletions.
diff --git a/inc/condition.class.php b/inc/condition.class.php
@@ -129,7 +129,6 @@ public static function import(PluginFormcreatorLinker $linker, $input = [], $con
          }
       }
       $input['plugin_formcreator_questions_id'] = $linked->getID();
-      $input['itemtype'] = $linked->getType();
 
       // Add or update condition
       $originalId = $input[$idKey];
@@ -177,7 +176,6 @@ public function export($remove_uuid = false) {
          $condition['plugin_formcreator_questions_id'] = $question->fields['uuid'];
       }
       unset($condition[$idToRemove]);
-      unset($condition['itemtype']);
 
       return $condition;
    }

diff --git a/inc/question.class.php b/inc/question.class.php
@@ -937,13 +937,13 @@ public static function import(PluginFormcreatorLinker $linker, $input = [], $con
          foreach ($input['_conditions'] as $condition) {
             PluginFormcreatorCondition::import($linker, $condition, $itemId);
          }
-         $field = PluginFormcreatorFields::getFieldInstance(
-            $input['fieldtype'],
-            $item
-         );
       }
 
       // Import parameters
+      $field = PluginFormcreatorFields::getFieldInstance(
+         $input['fieldtype'],
+         $item
+      );
       if (isset($input['_parameters'])) {
          $parameters = $field->getParameters();
          foreach ($parameters as $fieldName => $parameter) {

diff --git a/inc/section.class.php b/inc/section.class.php
@@ -316,7 +316,7 @@ public function export($remove_uuid = false) {
          }
       }
 
-      // get question conditions
+      // get section conditions
       $section['_conditions'] = [];
       $condition = new PluginFormcreatorCondition();
       $all_conditions = $condition->getConditionsFromItem($this);