Merge pull request #36 from jonasdekeukelaere/dropdown-required

Add shorthand to make dropdown required
forkcms · May 30, 2016 · f7a2fb8 · f7a2fb8
2 parents 6eb6865 + 11eeb3b
commit f7a2fb8
Show file tree

Hide file tree

Showing 2 changed files with 27 additions and 0 deletions.
diff --git a/spoon/form/dropdown.php b/spoon/form/dropdown.php
@@ -735,4 +735,16 @@ private function setValues(array $values = null)
 			}
 		}
 	}
+
+	/**
+	 * Marks a field as required
+	 *
+	 * @return self
+	 */
+	public function makeRequired()
+	{
+		$this->setAttribute('required', 'required');
+
+		return $this;
+	}
 }
diff --git a/spoon/tests/form/SpoonFormDropdownTest.php b/spoon/tests/form/SpoonFormDropdownTest.php
@@ -161,4 +161,19 @@ public function testParse()
 	{
 		$this->ddmDefaultElement->parse();
 	}
+
+	public function testRequired()
+	{
+		$this->ddmSingle->makeRequired();
+		$this->assertEquals(
+			'required',
+			$this->ddmSingle->getAttributes()['required']
+		);
+
+		$this->ddmMultiple->makeRequired();
+		$this->assertEquals(
+			'required',
+			$this->ddmMultiple->getAttributes()['required']
+		);
+	}
 }