fix: make getSoleValue return null instead of throwing

phanan · May 24, 2024 · 8c14629 · 8c14629
1 parent 618963b
commit 8c14629
Showing 1 changed file with 19 additions and 15 deletions.
diff --git a/src/Poddle.php b/src/Poddle.php
@@ -76,24 +76,28 @@ public function getEpisodes(): EpisodeCollection
      */
     private function getSoleValue(string ...$queries): ?string
     {
-        foreach ($queries as $query) {
-            if (!Str::startsWith('/rss/channel/', $query)) {
-                $query = '/rss/channel/'.ltrim($query, '/');
+        try {
+            foreach ($queries as $query) {
+                if (!Str::startsWith('/rss/channel/', $query)) {
+                    $query = '/rss/channel/' . ltrim($query, '/');
+                }
+
+                if (Str::contains($query, '@')) {
+                    [$query, $attribute] = explode('@', $query, 2);
+                    $value = $this->xmlReader->xpathElement($query)->first()?->getAttribute($attribute); // @phpstan-ignore-line
+                } else {
+                    $value = $this->xmlReader->xpathValue($query)->first();
+                }
+
+                if ($value) {
+                    return $value;
+                }
             }
 
-            if (Str::contains($query, '@')) {
-                [$query, $attribute] = explode('@', $query, 2);
-                $value = $this->xmlReader->xpathElement($query)->first()?->getAttribute($attribute); // @phpstan-ignore-line
-            } else {
-                $value = $this->xmlReader->xpathValue($query)->first();
-            }
-
-            if ($value) {
-                return $value;
-            }
+            return null;
+        } catch (Throwable) {
+            return null;
         }
-
-        return null;
     }
 
     /**