refactror(question): simplify code

Author: btry

inc/abstracttarget.class.php

@@ -217,11 +217,11 @@ protected function prepareTemplate($template, PluginFormcreatorFormAnswer $formA
    /**
     * Append fields data to input
     *
-    * @param PluginFormcreatorFormanswer $formanswer the source formanswer
     * @param array $input data of the generated target
-    * @return void
+    * @param PluginFormcreatorFormanswer $formanswer the source formanswer
+    * @return array
     */
-   protected function appendFieldsData(PluginFormcreatorFormanswer $formanswer, &$input): void {
+   protected function appendFieldsData(array$input, PluginFormcreatorFormanswer $formanswer): array {
       global $DB;
 
       //get all PluginFormcreatorAnswer
@@ -262,6 +262,8 @@ protected function appendFieldsData(PluginFormcreatorFormanswer $formanswer, &$i
          }
          $input['c_id'] = $blocks_field;
       }
+
+      return $input;
    }
 
 

inc/question.class.php

@@ -279,8 +279,6 @@ public function getRenderedHtml($domain, $canEdit = true, ?PluginFormcreatorForm
          return '';
       }
 
-      $html = '';
-
       $field = $this->getSubField();
       if (!$field->isPrerequisites()) {
          return '';
@@ -292,7 +290,7 @@ public function getRenderedHtml($domain, $canEdit = true, ?PluginFormcreatorForm
       $x = $this->fields['col'];
       $width = $this->fields['width'];
       $hiddenAttribute = $isVisible ? '' : 'hidden=""';
-      $html .= '<div'
+      $html = '<div'
          . ' gs-x="' . $x . '"'
          . ' gs-w="' . $width . '"'
          . ' data-itemtype="' . self::class . '"'

inc/targetticket.class.php

@@ -893,7 +893,7 @@ public function save(PluginFormcreatorFormAnswer $formanswer): ?CommonDBTM {
 
       $data = $this->prepareUploadedFiles($data, $formanswer);
 
-      $this->appendFieldsData($formanswer, $data);
+      $data = $this->appendFieldsData($data, $formanswer);
 
       // Cleanup actors array
       $data = $this->cleanActors($data);