Updated tag.php to use str_contains() instead of mb_strpos() where possible.

Fixed some return value issues in `htmldoc`.

Author: hexydec

.github/workflows/tests.yml

@@ -7,7 +7,7 @@ jobs:
     runs-on: ubuntu-latest
 
     steps:
-      - uses: actions/checkout@master
+      - uses: actions/checkout@v3
 
       - name: Cache Composer dependencies
         uses: actions/cache@v3

src/htmldoc.php

@@ -75,7 +75,6 @@ public function __get(string $var) : mixed {
 		} else {
 			return $this->children[0]->{$var};
 		}
-		return null;
 	}
 
 	/**
@@ -518,7 +517,7 @@ public function text() : string {
 	 * Adds the specified nodes to the htmldoc object
 	 *
 	 * @param array $nodes An array of nodes to add to the collection
-	 * @return void
+	 * @return htmldoc The current instance of this object
 	 */
 	protected function collection(array $nodes) : htmldoc {
 

src/tokens/tag.php

@@ -479,7 +479,7 @@ public function minify(array $minify) : void {
 				}
 
 				// use parent folders if it is shorter
-				if ($minify['urls']['parent'] && $dirs && \mb_strpos($attributes[$key], '/') === 0 && \mb_strpos($attributes[$key], '//') === false) {
+				if ($minify['urls']['parent'] && $dirs && \mb_strpos($attributes[$key], '/') === 0 && !\str_contains($attributes[$key], '//')) {
 					$isDir = \mb_strrpos($attributes[$key], '/') === \mb_strlen($attributes[$key])-1;
 					$compare = \explode('/', \trim($isDir ? $attributes[$key] : \dirname($attributes[$key]), '/'));
 					$update = false;
@@ -531,7 +531,7 @@ public function minify(array $minify) : void {
 					), '; ');
 
 				// trim classes
-				} elseif ($key === 'class' && $min['class'] && \mb_strpos($attributes[$key] ?? '', ' ') !== false) {
+				} elseif ($key === 'class' && $min['class'] && \str_contains($attributes[$key] ?? '', ' ')) {
 					$attributes[$key] = \trim(\preg_replace('/\s+/', ' ', $attributes[$key]));
 
 				// minify option tag, always capture the tag to prevent it being removed as a default
@@ -879,7 +879,7 @@ public function html(array $options = []) : string {
 					$html .= '='.$value;
 
 				// single quotes || swap when minimal and there are double quotes in the string
-				} elseif ($options['quotestyle'] === 'single' || ($options['quotestyle'] === 'minimal' && \mb_strpos($value, '"') !== false)) {
+				} elseif ($options['quotestyle'] === 'single' || ($options['quotestyle'] === 'minimal' && \str_contains($value, '"'))) {
 					$html .= "='".\str_replace(['&', "'", '<'], ['&amp;', '&#39;', '&lt;'], $value)."'";
 
 				// double quotes