BaseControl::$disabled is bool, added $disabledChoices

Author: dg

src/Forms/Controls/BaseControl.php

@@ -44,9 +44,7 @@ abstract class BaseControl extends Nette\ComponentModel\Component implements Con
 	protected mixed $value = null;
 	protected Html $control;
 	protected Html $label;
-
-	/** @var bool|bool[] */
-	protected bool|array $disabled = false;
+	protected bool $disabled = false;
 
 	/** @var callable[][]  extension methods */
 	private static array $extMethods = [];
@@ -197,7 +195,7 @@ public function setDisabled(bool $state = true): static
 	 */
 	public function isDisabled(): bool
 	{
-		return $this->disabled === true;
+		return $this->disabled;
 	}
 
 

src/Forms/Controls/CheckboxList.php

@@ -59,7 +59,7 @@ public function getControl(): Html
 				array_merge($input->attrs, [
 					'id' => null,
 					'checked?' => $this->value,
-					'disabled:' => $this->disabled,
+					'disabled:' => $this->disabled ?: $this->disabledChoices,
 					'required' => null,
 					'data-nette-rules:' => [array_key_first($items) => $input->attrs['data-nette-rules']],
 				]),
@@ -82,7 +82,7 @@ public function getControlPart($key = null): Html
 		return parent::getControl()->addAttributes([
 			'id' => $this->getHtmlId() . '-' . $key,
 			'checked' => in_array($key, (array) $this->value, strict: true),
-			'disabled' => is_array($this->disabled) ? isset($this->disabled[$key]) : $this->disabled,
+			'disabled' => $this->disabled || isset($this->disabledChoices[$key]),
 			'required' => null,
 			'value' => $key,
 		]);

src/Forms/Controls/ChoiceControl.php

@@ -20,6 +20,8 @@
  */
 abstract class ChoiceControl extends BaseControl
 {
+	/** @var bool[] */
+	protected array $disabledChoices = [];
 	private bool $checkDefaultValue = true;
 	private array $items = [];
 
@@ -68,7 +70,7 @@ public function setValue($value): static
 	public function getValue(): mixed
 	{
 		return array_key_exists($this->value, $this->items)
-			&& !isset($this->disabled[$this->value])
+			&& !isset($this->disabledChoices[$this->value])
 			? $this->value
 			: null;
 	}
@@ -127,12 +129,11 @@ public function getSelectedItem(): mixed
 	public function setDisabled(bool|array $value = true): static
 	{
 		if (!is_array($value)) {
+			$this->disabledChoices = [];
 			return parent::setDisabled($value);
 		}
-
-		parent::setDisabled(false);
-		$this->disabled = array_fill_keys($value, value: true);
-		return $this;
+		$this->disabledChoices = array_fill_keys($value, value: true);
+		return parent::setDisabled(false);
 	}
 
 

src/Forms/Controls/MultiChoiceControl.php

@@ -20,6 +20,8 @@
  */
 abstract class MultiChoiceControl extends BaseControl
 {
+	/** @var bool[] */
+	protected array $disabledChoices = [];
 	private bool $checkDefaultValue = true;
 	private array $items = [];
 
@@ -116,10 +118,7 @@ public function getItems(): array
 	 */
 	public function getSelectedItems(): array
 	{
-		$selected = array_intersect_key($this->items, array_flip($this->value));
-		return is_array($this->disabled)
-			? array_diff_key($selected, $this->disabled)
-			: $selected;
+		return array_diff_key(array_intersect_key($this->items, array_flip($this->value)), $this->disabledChoices);
 	}
 
 
@@ -129,12 +128,11 @@ public function getSelectedItems(): array
 	public function setDisabled(bool|array $value = true): static
 	{
 		if (!is_array($value)) {
+			$this->disabledChoices = [];
 			return parent::setDisabled($value);
 		}
-
-		parent::setDisabled(false);
-		$this->disabled = array_fill_keys($value, value: true);
-		return $this;
+		$this->disabledChoices = array_fill_keys($value, value: true);
+		return parent::setDisabled(false);
 	}
 
 

src/Forms/Controls/MultiSelectBox.php

@@ -65,7 +65,7 @@ public function getControl(): Nette\Utils\Html
 		return Nette\Forms\Helpers::createSelectBox(
 			$items,
 			[
-				'disabled:' => is_array($this->disabled) ? $this->disabled : null,
+				'disabled:' => $this->disabledChoices,
 			] + $this->optionAttributes,
 			$this->value,
 		)->addAttributes(parent::getControl()->attrs)->multiple(true);

src/Forms/Controls/RadioList.php

@@ -57,7 +57,7 @@ public function getControl(): Html
 				array_merge($input->attrs, [
 					'id:' => $ids,
 					'checked?' => $this->value,
-					'disabled:' => $this->disabled,
+					'disabled:' => $this->disabled ?: $this->disabledChoices,
 					'data-nette-rules:' => [key($items) => $input->attrs['data-nette-rules']],
 				]),
 				['for:' => $ids] + $this->itemLabel->attrs,
@@ -79,7 +79,7 @@ public function getControlPart($key = null): Html
 		return parent::getControl()->addAttributes([
 			'id' => $this->getHtmlId() . '-' . $key,
 			'checked' => in_array($key, (array) $this->value, strict: true),
-			'disabled' => is_array($this->disabled) ? isset($this->disabled[$key]) : $this->disabled,
+			'disabled' => $this->disabled || isset($this->disabledChoices[$key]),
 			'value' => $key,
 		]);
 	}

src/Forms/Controls/SelectBox.php

@@ -95,7 +95,7 @@ public function getControl(): Nette\Utils\Html
 		}
 
 		$attrs = $this->optionAttributes;
-		$attrs['disabled:'] = is_array($this->disabled) ? $this->disabled : [];
+		$attrs['disabled:'] = $this->disabledChoices;
 
 		$selected = $this->value;
 		if ($this->prompt !== false) {