feature: allow binding directly to data-element

PhpGt · Nov 21, 2024 · 459836d · 459836d
1 parent 1db4484
commit 459836d
Show file tree

Hide file tree

Showing 5 changed files with 60 additions and 6 deletions.
diff --git a/src/DocumentBinder.php b/src/DocumentBinder.php
@@ -160,23 +160,29 @@ public function bindListCallback(
 
 	public function cleanupDocument():void {
 		$xpathResult = $this->document->evaluate(
-			"//*/@*[starts-with(name(), 'data-bind')] | //*/@*[starts-with(name(), 'data-list')] | //*/@*[starts-with(name(), 'data-template')] | //*/@*[starts-with(name(), 'data-table-key')]"
+			"//*/@*[starts-with(name(), 'data-bind')] | //*/@*[starts-with(name(), 'data-list')] | //*/@*[starts-with(name(), 'data-template')] | //*/@*[starts-with(name(), 'data-table-key')] | //*/@*[starts-with(name(), 'data-element')]"
 		);
 
 		$elementsToRemove = [];
 		/** @var Attr $item */
 		foreach($xpathResult as $item) {
-			if($item->ownerElement->hasAttribute("data-element")) {
-				array_push($elementsToRemove, $item->ownerElement);
+			$ownerElement = $item->ownerElement;
+			if($ownerElement->hasAttribute("data-element")) {
+				if(!$ownerElement->hasAttribute("data-bound")) {
+					array_push($elementsToRemove, $ownerElement);
+				}
 				continue;
 			}
 
-			$item->ownerElement->removeAttribute($item->name);
+			$ownerElement->removeAttribute($item->name);
 		}
 
 		foreach($this->document->querySelectorAll("[data-element]") as $dataElement) {
 			$dataElement->removeAttribute("data-element");
 		}
+		foreach($this->document->querySelectorAll("[data-bound]") as $dataBound) {
+			$dataBound->removeAttribute("data-bound");
+		}
 
 		foreach($elementsToRemove as $element) {
 			$element->remove();

diff --git a/src/HTMLAttributeBinder.php b/src/HTMLAttributeBinder.php
@@ -41,6 +41,12 @@ public function bind(
 		foreach($element->attributes as $attrName => $attr) {
 			$attrValue = $attr->value;
 
+			if($attrName === "data-element") {
+				if($attr->value === $key && $value) {
+					$element->setAttribute("data-bound", true);
+				}
+			}
+
 			if(!str_starts_with($attrName, "data-bind")) {
 				continue;
 			}
@@ -81,6 +87,7 @@ public function bind(
 				$value,
 				$modifier
 			);
+			$element->setAttribute("data-bound", true);
 
 			if(!$attr->ownerElement->hasAttribute("data-rebind")) {
 				array_push($attributesToRemove, $attrName);

diff --git a/src/HTMLAttributeCollection.php b/src/HTMLAttributeCollection.php
@@ -7,7 +7,7 @@
 class HTMLAttributeCollection {
 	public function find(Element $context):XPathResult {
 		return $context->ownerDocument->evaluate(
-			"descendant-or-self::*[@*[starts-with(name(), 'data-bind')]]",
+			"descendant-or-self::*[@*[starts-with(name(), 'data-bind')] or (@data-element and @data-element != '')]",
 			$context
 		);
 	}

diff --git a/test/phpunit/DocumentBinderTest.php b/test/phpunit/DocumentBinderTest.php
@@ -1245,7 +1245,29 @@ public function test_keepsElementWhenBound():void {
 		$sut->setDependencies(...$this->documentBinderDependencies($document));
 		$sut->bindKeyValue("error", "Example error!");
 		$sut->cleanupDocument();
-		self::assertStringContainsStringIgnoringCase("error", (string)$document);
+		$errorDiv = $document->querySelector("form>div");
+		self::assertNotNull($errorDiv);
+		self::assertSame("Example error!", $errorDiv->textContent);
+	}
+
+	public function test_bindElementWithBindValue():void {
+		$document = new HTMLDocument(HTMLPageContent::HTML_REMOVE_UNBOUND_BIND_VALUE);
+		$sut = new DocumentBinder($document);
+		$sut->setDependencies(...$this->documentBinderDependencies($document));
+		$sut->bindKeyValue("error", true);
+		$sut->cleanupDocument();
+		$errorDiv = $document->querySelector("form>div");
+		self::assertNotNull($errorDiv);
+		self::assertSame("There has been an error!", $errorDiv->textContent);
+	}
+
+	public function test_bindElementIsRemovedWhenNotBound():void {
+		$document = new HTMLDocument(HTMLPageContent::HTML_REMOVE_UNBOUND_BIND_VALUE);
+		$sut = new DocumentBinder($document);
+		$sut->setDependencies(...$this->documentBinderDependencies($document));
+		$sut->cleanupDocument();
+		$errorDiv = $document->querySelector("form>div");
+		self::assertNull($errorDiv);
 	}
 
 	public function test_bindData_withList_dataBindList():void {

diff --git a/test/phpunit/TestHelper/HTMLPageContent.php b/test/phpunit/TestHelper/HTMLPageContent.php
@@ -1093,6 +1093,25 @@ class HTMLPageContent {
 	
 	<button name="do" value="login">Log in!</button>
 </form>
+HTML;
+
+	const HTML_REMOVE_UNBOUND_BIND_VALUE = <<<HTML
+<!doctype html>
+<h1>Log in to the system</h1>
+<form method="post">
+	<div data-element="error">There has been an error!</div>
+	
+	<label>
+		<span>Your email address:</span>
+		<input name="email" type="email" required />
+	</label>
+	<label>
+		<span>Your password:</span>
+		<input name="password" type="password" required />
+	</label>
+	
+	<button name="do" value="login">Log in!</button>
+</form>
 HTML;
 
 	const HTML_DATA_BIND_LIST = <<<HTML