Merge pull request #664 from devmes/bugfix/663

sebastianstein · web-flow · commit cd0ba959c73d · 2026-02-12T15:15:48.000+01:00
[BUGFIX] Support multiple values in SelectViewHelper
diff --git a/Classes/ViewHelpers/Form/SelectViewHelper.php b/Classes/ViewHelpers/Form/SelectViewHelper.php
@@ -32,7 +32,7 @@ class SelectViewHelper extends AbstractFormFieldViewHelper
     public function initialize(): void
     {
         parent::initialize();
-        $this->tag->addAttribute('data-selected-value', $this->getSelectedValue());
+        $this->tag->addAttribute('data-selected-value', is_array($this->getSelectedValue()) ? implode(',', $this->getSelectedValue()) : $this->getSelectedValue());
     }
 
     public function initializeArguments(): void
@@ -313,7 +313,6 @@ protected function isSelected(mixed $value): bool
      */
     protected function getSelectedValue()
     {
-        $selectedValues = null;
         $this->setRespectSubmittedDataValue(true);
         $value = $this->getValueAttribute();
         if (!is_array($value) && !$value instanceof \Traversable) {
@@ -322,6 +321,10 @@ protected function getSelectedValue()
                 return $selectedValues;
             }
         }
+        $selectedValues = [];
+        foreach ($value as $selectedValueElement) {
+            $selectedValues[] = $this->getOptionValueScalar($selectedValueElement);
+        }
 
         // set preselection from TypoScript
         if (! $this->renderingContext instanceof RenderingContext) {

Original file line number	Diff line number	Diff line change
`@@ -32,7 +32,7 @@ class SelectViewHelper extends AbstractFormFieldViewHelper`
`32`	`32`	`public function initialize(): void`
`33`	`33`	`{`
`34`	`34`	`parent::initialize();`
`35`		`- $this->tag->addAttribute('data-selected-value', $this->getSelectedValue());`
	`35`	`+ $this->tag->addAttribute('data-selected-value', is_array($this->getSelectedValue()) ? implode(',', $this->getSelectedValue()) : $this->getSelectedValue());`
`36`	`36`	`}`
`37`	`37`
`38`	`38`	`public function initializeArguments(): void`
`@@ -313,7 +313,6 @@ protected function isSelected(mixed $value): bool`
`313`	`313`	`*/`
`314`	`314`	`protected function getSelectedValue()`
`315`	`315`	`{`
`316`		`- $selectedValues = null;`
`317`	`316`	`$this->setRespectSubmittedDataValue(true);`
`318`	`317`	`$value = $this->getValueAttribute();`
`319`	`318`	`if (!is_array($value) && !$value instanceof \Traversable) {`
`@@ -322,6 +321,10 @@ protected function getSelectedValue()`
`322`	`321`	`return $selectedValues;`
`323`	`322`	`}`
`324`	`323`	`}`
	`324`	`+ $selectedValues = [];`
	`325`	`+ foreach ($value as $selectedValueElement) {`
	`326`	`+ $selectedValues[] = $this->getOptionValueScalar($selectedValueElement);`
	`327`	`+ }`
`325`	`328`
`326`	`329`	`// set preselection from TypoScript`
`327`	`330`	`if (! $this->renderingContext instanceof RenderingContext) {`