[DomCrawler] Make ChoiceFormField::isDisabled return true for unchecked disabled checkboxes

MatTheCat · MatTheCat · commit 9807ce6058f4 · 2025-01-09T16:35:00.000+01:00
diff --git a/src/Symfony/Component/DomCrawler/Field/ChoiceFormField.php b/src/Symfony/Component/DomCrawler/Field/ChoiceFormField.php
@@ -45,6 +45,10 @@ public function hasValue(): bool
      */
     public function isDisabled(): bool
     {
+        if ('checkbox' === $this->type) {
+            return parent::isDisabled();
+        }
+
         if (parent::isDisabled() && 'select' === $this->type) {
             return true;
         }
diff --git a/src/Symfony/Component/DomCrawler/Tests/Field/ChoiceFormFieldTest.php b/src/Symfony/Component/DomCrawler/Tests/Field/ChoiceFormFieldTest.php
@@ -272,6 +272,17 @@ public function testCheckboxWithEmptyBooleanAttribute()
         $this->assertEquals('foo', $field->getValue());
     }
 
+    public function testCheckboxIsDisabled()
+    {
+        $node = $this->createNode('input', '', ['type' => 'checkbox', 'name' => 'name', 'disabled' => '']);
+        $field = new ChoiceFormField($node);
+
+        $this->assertTrue($field->isDisabled(), '->isDisabled() returns true when the checkbox is disabled, even if it is not checked');
+
+        $field->tick();
+        $this->assertTrue($field->isDisabled(), '->isDisabled() returns true when the checkbox is disabled, even if it is checked');
+    }
+
     public function testTick()
     {
         $node = $this->createSelectNode(['foo' => false, 'bar' => false]);

Original file line number	Diff line number	Diff line change
`@@ -45,6 +45,10 @@ public function hasValue(): bool`
`45`	`45`	`*/`
`46`	`46`	`public function isDisabled(): bool`
`47`	`47`	`{`
	`48`	`+ if ('checkbox' === $this->type) {`
	`49`	`+ return parent::isDisabled();`
	`50`	`+ }`
	`51`	`+`
`48`	`52`	`if (parent::isDisabled() && 'select' === $this->type) {`
`49`	`53`	`return true;`
`50`	`54`	`}`