Fixed bug in htmldoc::children() where it selected the root nodes instead of their children.

Updated `htmldoc::collection` to return itself instead of `void`.
Added `htmldoc::parent()` to get the parents of each item in the collection.
Added tests and updated `children()` test as this was incorrect.

Author: hexydec

src/htmldoc.php

@@ -72,6 +72,8 @@ public function __get(string $var) : mixed {
 			return $this->config;
 		} elseif ($var === 'length') {
 			return \count($this->children);
+		} else {
+			return $this->children[0]->{$var};
 		}
 		return null;
 	}
@@ -123,8 +125,7 @@ public function offsetUnset(mixed $i) : void {
 	public function offsetGet(mixed $i) : mixed { // return reference so you can set it like an array
 		if (isset($this->children[$i])) {
 			$obj = new htmldoc($this->config);
-			$obj->collection([$this->children[$i]]);
-			return $obj;
+			return $obj->collection([$this->children[$i]]);
 		}
 		return null;
 	}
@@ -137,8 +138,7 @@ public function offsetGet(mixed $i) : mixed { // return reference so you can set
 	public function current() : mixed {
 		if (isset($this->children[$this->pointer])) {
 			$obj = new htmldoc($this->config);
-			$obj->collection([$this->children[$this->pointer]]);
-			return $obj;
+			return $obj->collection([$this->children[$this->pointer]]);
 		}
 		return null;
 	}
@@ -357,16 +357,17 @@ public function get(int $index = null) : tag|array|null {
 		// return all children if no index
 		if ($index === null) {
 			return $children;
-		}
+		} else {
 
-		// check if index is minus
-		if ($index < 0) {
-			$index = \count($children) + $index;
-		}
+			// check if index is minus
+			if ($index < 0) {
+				$index = \count($children) + $index;
+			}
 
-		// return index if set
-		if (isset($children[$index])) {
-			return $children[$index];
+			// return index if set
+			if (isset($children[$index])) {
+				return $children[$index];
+			}
 		}
 		return null;
 	}
@@ -443,7 +444,23 @@ public function eq(int $index) : htmldoc {
 	 * @return htmldoc A new htmldoc object
 	 */
 	public function children() : htmldoc {
-		return $this->find('>*');
+		return $this->find('*>*');
+	}
+
+	/**
+	 * Generate a new htmldoc object containing all the child tags of the parents
+	 *
+	 * @return htmldoc A new htmldoc object
+	 */
+	public function parent() : htmldoc {
+		$doc = new htmldoc($this->config);
+		$nodes = [];
+		foreach ($this->children AS $item) {
+			if (\get_class($item) === 'hexydec\\html\\tag' && ($parent = $this->children[0]->parent) !== null) {
+				$nodes[] = $parent;
+			}
+		}
+		return $doc->collection($nodes);
 	}
 
 	/**
@@ -489,7 +506,7 @@ public function text() : string {
 	 * @param array $nodes An array of nodes to add to the collection
 	 * @return void
 	 */
-	protected function collection(array $nodes) : void {
+	protected function collection(array $nodes) : htmldoc {
 
 		// only store unique nodes as some find operations can produce the same node multiple times
 		$unique = [];
@@ -499,6 +516,7 @@ protected function collection(array $nodes) : void {
 			}
 		}
 		$this->children = $unique;
+		return $this;
 	}
 
 	/**

tests/findHtmldocTest.php

@@ -81,11 +81,12 @@ public function testCanTraverseElements() {
 
 			$this->assertEquals('<div id="first" class="first">First</div>', $doc->find('.positions > *')->first()->html(), 'Can find first element');
 			$this->assertEquals('<div class="last">Last</div>', $doc->find('.positions > *')->last()->html(), 'Can find last element');
-			$this->assertEquals('<div class="find"><h1 class="find__heading">Heading</h1><p class="find__paragraph" title="This is a paragraph">Paragraph</p><a class="find__anchor" href="https://github.com/hexydec/htmldoc/">Anchor</a></div>', $doc->find('.positions > *')->eq(1)->html(), 'Can specific element');
-			$this->assertEquals('<div class="find"><h1 class="find__heading">Heading</h1><p class="find__paragraph" title="This is a paragraph">Paragraph</p><a class="find__anchor" href="https://github.com/hexydec/htmldoc/">Anchor</a></div>', $doc->find('.positions > *')->eq(-2)->html(), 'Can specific element');
-			$this->assertEquals('<div class="find"><h1 class="find__heading">Heading</h1><p class="find__paragraph" title="This is a paragraph">Paragraph</p><a class="find__anchor" href="https://github.com/hexydec/htmldoc/">Anchor</a></div>', $doc->find('.find')->children()->html(), 'Can specific element');
+			$this->assertEquals('<div class="find"><h1 class="find__heading">Heading</h1><p class="find__paragraph" title="This is a paragraph">Paragraph</p><a class="find__anchor" href="https://github.com/hexydec/htmldoc/">Anchor</a></div>', $doc->find('.positions > *')->eq(1)->html(), 'Can find specific element');
+			$this->assertEquals('<div class="find"><h1 class="find__heading">Heading</h1><p class="find__paragraph" title="This is a paragraph">Paragraph</p><a class="find__anchor" href="https://github.com/hexydec/htmldoc/">Anchor</a></div>', $doc->find('.positions > *')->eq(-2)->html(), 'Can find specific element');
+			$this->assertEquals('<h1 class="find__heading">Heading</h1><p class="find__paragraph" title="This is a paragraph">Paragraph</p><a class="find__anchor" href="https://github.com/hexydec/htmldoc/">Anchor</a>', $doc->find('.find')->children()->html(), 'Can extract children of an element');
+			$this->assertEquals('<div class="find"><h1 class="find__heading">Heading</h1><p class="find__paragraph" title="This is a paragraph">Paragraph</p><a class="find__anchor" href="https://github.com/hexydec/htmldoc/">Anchor</a></div>', $doc->find('.find__heading')->parent()->html(), 'Can specific element');
 
-			$this->assertEquals(3, count($doc->find('.positions > *')->get()));
+			$this->assertEquals(3, \count($doc->find('.positions > *')->get()));
 			$this->assertEquals('<div class="last">Last</div>', $doc->find('.positions > *')->get(2)->html());
 			$this->assertEquals('<div class="last">Last</div>', $doc->find('.positions > *')->get(-1)->html());