Merge pull request #202 from Icinga/fix-template-form

Don't populate `UploadedFile` on `text/select` elements & vice versa
Icinga · Sep 1, 2023 · 062ba04 · 062ba04
2 parents 2e09144 + 69f9397
commit 062ba04
Showing 1 changed file with 9 additions and 0 deletions.
diff --git a/library/Reporting/Web/Forms/TemplateForm.php b/library/Reporting/Web/Forms/TemplateForm.php
@@ -247,6 +247,15 @@ protected function addColumnSettings($name, $label)
             ]
         ]);
 
+        $valueType = $this->getValue($type, 'none');
+        $populated = $this->getPopulatedValue($value);
+        if (
+            ($valueType === 'image' && ! $populated instanceof UploadedFile)
+            || ($valueType !== 'image' && $populated instanceof UploadedFile)
+        ) {
+            $this->clearPopulatedValue($value);
+        }
+
         switch ($this->getValue($type, 'none')) {
             case 'image':
                 $this->addElement('file', $value, [