Merge pull request #42 from heiglandreas/addSessionize

Last touch on Sessionize
joindin · Feb 3, 2020 · b1f2971 · b1f2971
2 parents e3e8624 + c081334
commit b1f2971
Show file tree

Hide file tree

Showing 3 changed files with 17 additions and 10 deletions.
diff --git a/src/Subcommands/Sessionize/Parser/EntryParser.php b/src/Subcommands/Sessionize/Parser/EntryParser.php
@@ -18,6 +18,7 @@
 use Exception;
 use GuzzleHttp\Client;
 use InvalidArgumentException;
+use Throwable;
 
 class EntryParser
 {
@@ -34,7 +35,7 @@ public function __construct(Cfp $cfp, ServiceContainer $container)
         $this->geolocationService = $container->getGeolocationService();
     }
 
-    public function parse($uri)
+    public function parse($uri): Cfp
     {
         $cfp = clone($this->cfp);
         try {
@@ -97,8 +98,12 @@ public function parse($uri)
                 $cfp->eventEndDate = $cfp->eventStartDate;
             }
 
-            $iconUri = new IconUri($uri);
-            $cfp->iconUri = $iconUri->parse($dom, $xpath);
+            try {
+                $iconUri = new IconUri($uri);
+                $cfp->iconUri = $iconUri->parse($dom, $xpath);
+            } catch (Throwable $e) {
+                $cfp->iconUri = '';
+            }
 
             return $cfp;
         } catch (Exception $e) {

diff --git a/src/Subcommands/Sessionize/Parser/Location.php b/src/Subcommands/Sessionize/Parser/Location.php
@@ -28,7 +28,7 @@ public function parse(DOMDocument $dom, DOMXPath $xpath) : string
         if (! $locations || $locations->length == 0) {
             throw new \InvalidArgumentException('The Event does not seem to have a location');
         }
-        $location = [];
+        $locationArray = [];
         foreach ($locations as $item) {
             $location = trim($item->textContent);
             if ($location === '') {
@@ -37,9 +37,9 @@ public function parse(DOMDocument $dom, DOMXPath $xpath) : string
 
             return $location;
 
-            $location[] = $location;
+            $locationArray[] = $location;
         }
 
-        return implode(', ', array_unique($location));
+        return implode(', ', array_unique($locationArray));
     }
 }
diff --git a/src/Subcommands/Sessionize/Parser/Sessionize.php b/src/Subcommands/Sessionize/Parser/Sessionize.php
@@ -39,9 +39,9 @@ public function parse(WriterInterface $writer)
         $dom = new DOMDocument('1.0', 'UTF-8');
         $dom->load($uri);
         $dom->preserveWhiteSpace = false;
-
         $xpath = new DOMXPath($dom);
-        $nodes = $xpath->query("//url");
+        $xpath->registerNamespace('x', 'http://www.sitemaps.org/schemas/sitemap/0.9');
+        $nodes = $xpath->query("//x:url");
         if ($nodes->length < 1) {
             return $cfpList;
         }
@@ -62,9 +62,11 @@ public function parse(WriterInterface $writer)
             }
 
             try {
-                $cfpList->append($this->parser->parse($loc->textContent));
+                $cfp = $this->parser->parse($loc->textContent);
+                $writer->write($cfp, 'Sessionize');
+                $cfpList->append($cfp);
             } catch (\Exception $e) {
-                error_log($e->getMEssage());
+                //error_log($e->getMEssage());
             }
         }