Binding null values (#298)

* fix: binding null value does nothing on Element and Placeholder binds

* test: null binds by key

* test: null binds by data

* test: null binds by table

* test: null binds by list

* stan: fix type annotations

* build: update dependencies

composer.json

@@ -9,8 +9,8 @@
 		"phpgt/dom": "^3.0.0"
 	},
 	"require-dev": {
-		"phpunit/phpunit": "9.*",
-		"phpstan/phpstan": "1.*"
+		"phpunit/phpunit": "~9.5",
+		"phpstan/phpstan": "~1.1"
 	},
 
 	"autoload": {

src/DocumentBinder.php

@@ -4,6 +4,7 @@
 use Gt\Dom\Attr;
 use Gt\Dom\Document;
 use Gt\Dom\Element;
+use Gt\Dom\XPathResult;
 
 class DocumentBinder {
 	private ElementBinder $elementBinder;
@@ -129,11 +130,11 @@ public function bindListCallback(
 	}
 
 	public function cleanDatasets():void {
+		/** @var XPathResult<Attr> $xpathResult */
 		$xpathResult = $this->document->evaluate(
 			"//*/@*[starts-with(name(), 'data-bind')] | //*/@*[starts-with(name(), 'data-template')]"
 		);
 		foreach($xpathResult as $item) {
-			/** @var Attr $item */
 			$item->ownerElement->removeAttribute($item->name);
 		}
 	}

src/ElementBinder.php

@@ -34,8 +34,8 @@ public function bind(
 		mixed $value,
 		Element $context
 	):void {
+		/** @var Element $element */
 		foreach($this->htmlAttributeCollection->find($context) as $element) {
-			/** @var Element $element */
 			$this->htmlAttributeBinder->expandAttributes($element);
 			$this->htmlAttributeBinder->bind($key, $value, $element);
 		}

src/HTMLAttributeBinder.php

@@ -14,6 +14,10 @@ public function bind(
 		mixed $value,
 		Document|Element $element
 	):void {
+		if(is_null($value)) {
+			return;
+		}
+
 		if($element instanceof Document) {
 			$element = $element->documentElement;
 		}

src/ListBinder.php

@@ -73,6 +73,10 @@ public function bindListData(
 				);
 			}
 
+			if(is_null($listItem)) {
+				continue;
+			}
+
 			if($this->isKVP($listItem)) {
 				$elementBinder->bind(null, $listKey, $t);
 

src/PlaceholderBinder.php

@@ -12,6 +12,10 @@ public function bind(
 		mixed $value,
 		Node|Document $context
 	):void {
+		if(is_null($value)) {
+			return;
+		}
+
 		if($context instanceof Document) {
 			$context = $context->documentElement;
 		}
@@ -31,10 +35,10 @@ public function bind(
 			/** @var Text|Attr $text */
 			$text = $attributeOrText;
 			if($text instanceof Attr) {
+				/** @var Text $text */
 				$text = $text->firstChild;
 			}
 
-			/** @var Text $text */
 			$placeholder = $text->splitText(
 				strpos($text->data, "{{")
 			);

src/TableBinder.php

@@ -37,9 +37,8 @@ public function bindTableData(
 		}
 
 		$headerRow = array_shift($tableData);
+		/** @var HTMLTableElement $table */
 		foreach($tableArray as $table) {
-			/** @var HTMLTableElement $table */
-
 			$allowedHeaders = $headerRow;
 
 			$tHead = $table->tHead;
@@ -95,7 +94,7 @@ public function bindTableData(
 					}
 
 					$cellElement = $tr->ownerDocument->createElement($cellTypeToCreate);
-					$cellElement->textContent = $columnValue;
+					$cellElement->textContent = $columnValue ?? "";
 					$tr->appendChild($cellElement);
 				}
 			}

src/TemplateCollection.php

@@ -35,8 +35,8 @@ public function get(
 
 	private function extractTemplates(Document $document):void {
 		$dataTemplateArray = [];
+		/** @var Element $element */
 		foreach($document->querySelectorAll("[data-template]") as $element) {
-			/** @var Element $element */
 			$nodePath = (string)(new NodePathCalculator($element));
 			$templateElement = new TemplateElement($element);
 			$key = $templateElement->getTemplateName() ?? $nodePath;