Fix parsing invalid href

See oscarotero/Embed#502
wishthis · Oct 10, 2022 · d29a853 · d29a853
1 parent f839fb1
commit d29a853
Show file tree

Hide file tree

Showing 2 changed files with 25 additions and 1 deletion.
diff --git a/vendor/embed/embed/src/Detectors/Languages.php b/vendor/embed/embed/src/Detectors/Languages.php
@@ -3,6 +3,8 @@
 
 namespace Embed\Detectors;
 
+use function Embed\isEmpty;
+
 class Languages extends Detector
 {
     /**
@@ -17,7 +19,7 @@ public function detect(): array
             $language = $node->getAttribute('hreflang');
             $href = $node->getAttribute('href');
 
-            if (!$language || !$href) {
+            if (isEmpty($language, $href)) {
                 continue;
             }
 

diff --git a/vendor/embed/embed/src/functions.php b/vendor/embed/embed/src/functions.php
@@ -132,3 +132,25 @@ function getDirectory(string $path, int $position): ?string
     $dirs = explode('/', $path);
     return $dirs[$position + 1] ?? null;
 }
+
+/**
+ * Determine whether at least one of the supplied variables is empty.
+ *
+ * @param mixed ...$values The values to check.
+ *
+ * @return boolean
+ */
+function isEmpty(mixed ...$values): bool
+{
+    $skipValues = array(
+        'undefined',
+    );
+
+    foreach ($values as $value) {
+        if (empty($value) || in_array($value, $skipValues)) {
+            return true;
+        }
+    }
+
+    return false;
+}