joindin / callingallpapers-cli

src/Command/ParseEvents.php

Unused Use Statements -1 removed lines

@@ -31,7 +31,6 @@
 
 use Callingallpapers\Parser\JoindinCfpParser;
 use Callingallpapers\Parser\Lanyrd\LanyrdCfpParser;
-use Callingallpapers\Parser\PapercallIoParser;
 use Callingallpapers\Parser\PapercallIoParserFactory;
 use Callingallpapers\Service\TimezoneService;
 use Callingallpapers\Writer\ApiCfpWriter;

Indentation +8 added lines, -8 removed lines

@@ -47,11 +47,11 @@ 
     protected function configure()
     {
         $this->setName("parseCfPs")
-             ->setDescription("Retrieve CfPs and parse them")
-             ->setDefinition(array(
-                 new InputOption('start', 's', InputOption::VALUE_OPTIONAL, 'What should be the first date to be taken into account?', ''),
-             ))
-             ->setHelp(<<<EOT
+                ->setDescription("Retrieve CfPs and parse them")
+                ->setDefinition(array(
+                    new InputOption('start', 's', InputOption::VALUE_OPTIONAL, 'What should be the first date to be taken into account?', ''),
+                ))
+                ->setHelp(<<<EOT
 Get details about CfPs from different sources
 
 Usage:
@@ -61,7 +61,7 @@ 
 If you ommit the date the current date will be used instead
 <info>callingallpapers parseCfPs<env></info>
 EOT
-             );
+                );
     }
 
     protected function execute(InputInterface $input, OutputInterface $output)
@@ -81,9 +81,9 @@ 
 
         $timezoneService = new TimezoneService(new Client(), $config['timezonedb_token']);
         $parser = new LanyrdCfpParser($timezoneService);
-       // $parser->parse($writer);
+        // $parser->parse($writer);
         $parser = new JoindinCfpParser();
-       // $parser->parse($writer);
+        // $parser->parse($writer);
         $factory = new PapercallIoParserFactory($timezoneService);
         $parser = $factory();
         $parser->parse($writer);

src/Parser/PapercallIo/Description.php

Doc Comments +4 added lines

@@ -32,6 +32,10 @@
 class Description
 {
 
+    /**
+     * @param \DOMDocument $dom
+     * @param \DOMXPath $xpath
+     */
     public function parse($dom, $xpath)
     {
         $result = $xpath->query('//div[contains(@class, "description")]');

src/Parser/PapercallIo/Entry.php

Doc Comments +3 added lines

@@ -150,6 +150,9 @@
         return [$location->lat, $location->lon];
     }
 
+    /**
+     * @param \DOMXPath $xpath
+     */
     public function getEventPage($xpath)
     {
         $confPath = $xpath->query("//h3/a[contains(@class, 'summary')]");

Spacing +4 added lines, -4 removed lines

@@ -56,7 +56,7 @@ 
 
             $content = file_Get_contents($uri);
             $content = mb_convert_encoding($content, 'UTF-8');
-            $dom->loadHTML('<?xml version="1.0" charset="UTF-8" ?>' . $content);
+            $dom->loadHTML('<?xml version="1.0" charset="UTF-8" ?>'.$content);
             $dom->preserveWhiteSpace = false;
 
             $timezone = 'UTC';
@@ -154,15 +154,15 @@ 
     {
         $confPath = $xpath->query("//h3/a[contains(@class, 'summary')]");
 
-        if (! $confPath || $confPath->length == 0) {
+        if (!$confPath || $confPath->length == 0) {
             throw new \InvalidArgumentException('We can\'t find an EventPage');
         }
 
         $dom = new \DOMDocument('1.0', 'UTF-8');
 
-        $content = file_get_contents('https://lanyrd.com' . $confPath->item(0)->attributes->getNamedItem('href')->textContent);
+        $content = file_get_contents('https://lanyrd.com'.$confPath->item(0)->attributes->getNamedItem('href')->textContent);
         $content = mb_convert_encoding($content, 'UTF-8');
-        $dom->loadHTML('<?xml version="1.0" charset="UTF-8" ?>' . $content);
+        $dom->loadHTML('<?xml version="1.0" charset="UTF-8" ?>'.$content);
         $dom->preserveWhiteSpace = false;
 
         return $dom;

src/Parser/PapercallIo/EventEndDate.php

Doc Comments +4 added lines

@@ -38,6 +38,10 @@
         $this->timezone = new \DateTimezone($timezone);
     }
 
+    /**
+     * @param \DOMDocument $dom
+     * @param \DOMXPath $xpath
+     */
     public function parse($dom, $xpath)
     {
         $endDate = $xpath->query("//div[contains(@class, 'vevent')]/*/abbr[contains(@class, 'dtend')]"); ///a/abbr[class='dtstart']

Spacing +1 added lines, -1 removed lines

@@ -35,7 +35,7 @@
     public function parse($dom, $xpath)
     {
         $endDate = $xpath->query("//div[contains(@class, 'vevent')]/*/abbr[contains(@class, 'dtend')]"); ///a/abbr[class='dtstart']
-        if (! $endDate || $endDate->length == 0) {
+        if (!$endDate || $endDate->length == 0) {
             throw new \InvalidArgumentException('The Event does not seem to have an end date');
         }
 

src/Parser/PapercallIo/EventName.php

Doc Comments +4 added lines

@@ -32,6 +32,10 @@
 class EventName
 {
 
+    /**
+     * @param \DOMDocument $dom
+     * @param \DOMXPath $xpath
+     */
     public function parse($dom, $xpath)
     {
         $confPath = $xpath->query("//h3/a[contains(@class, 'summary')]");

Spacing +1 added lines, -1 removed lines

@@ -36,7 +36,7 @@
     {
         $confPath = $xpath->query("//h3/a[contains(@class, 'summary')]");
 
-        if (! $confPath || $confPath->length == 0) {
+        if (!$confPath || $confPath->length == 0) {
             throw new \InvalidArgumentException('The CfP does not seem to have an eventname');
         }
 

src/Parser/PapercallIo/EventStartDate.php

Doc Comments +4 added lines

@@ -38,6 +38,10 @@
         $this->timezone = new \DateTimezone($timezone);
     }
 
+    /**
+     * @param \DOMDocument $dom
+     * @param \DOMXPath $xpath
+     */
     public function parse($dom, $xpath)
     {
         $startDate = $xpath->query("//div[contains(@class, 'vevent')]/*/abbr[contains(@class, 'dtstart')]"); ///a/abbr[class='dtstart']

Spacing +1 added lines, -1 removed lines

@@ -35,7 +35,7 @@
     public function parse($dom, $xpath)
     {
         $startDate = $xpath->query("//div[contains(@class, 'vevent')]/*/abbr[contains(@class, 'dtstart')]"); ///a/abbr[class='dtstart']
-        if (! $startDate || $startDate->length == 0) {
+        if (!$startDate || $startDate->length == 0) {
             throw new \InvalidArgumentException('The Event does not seem to have a start date');
         }
 

src/Parser/PapercallIo/EventUri.php

Doc Comments +4 added lines

@@ -32,6 +32,10 @@
 class EventUri
 {
 
+    /**
+     * @param \DOMDocument $dom
+     * @param \DOMXPath $xpath
+     */
     public function parse($dom, $xpath)
     {
         $uriPath = $xpath->query("//a[contains(@title, 'visit their website')]");

Spacing +1 added lines, -1 removed lines

@@ -36,7 +36,7 @@
     {
         $uriPath = $xpath->query("//a[contains(@title, 'visit their website')]");
 
-        if (! $uriPath || $uriPath->length == 0) {
+        if (!$uriPath || $uriPath->length == 0) {
             throw new \InvalidArgumentException('The CfP does not seem to have an EventUri');
         }
 

src/Parser/PapercallIo/LanyrdCfpParser.php

Unused Use Statements -1 removed lines

@@ -30,7 +30,6 @@
 namespace Callingallpapers\Parser\Lanyrd;
 
 use Callingallpapers\Entity\Cfp;
-use Callingallpapers\Parser\Lanyrd\LanyrdEntryParser;
 use Callingallpapers\Parser\ParserInterface;
 use Callingallpapers\Service\TimezoneService;
 use Callingallpapers\Writer\WriterInterface;

Spacing +2 added lines, -2 removed lines

@@ -46,7 +46,7 @@ 
 
         do {
             $dom = new \DOMDocument('1.0', 'UTF-8');
-            $dom->loadHTMLFile($uri . '?page=' . ($i+1));
+            $dom->loadHTMLFile($uri.'?page='.($i + 1));
             $dom->preserveWhiteSpace = false;
 
             $xpath = new \DOMXPath($dom);
@@ -77,7 +77,7 @@ 
 
                     $eventPageUrl = $links->item(0)->attributes->getNamedItem('href')->textContent;
                     error_log($eventPageUrl);
-                    if (! $eventPageUrl) {
+                    if (!$eventPageUrl) {
                         continue;
                     }
                     $writer->write($lanyrdEntryParser->parse($eventPageUrl), 'lanyrd.com');

src/Parser/PapercallIo/Location.php

Doc Comments +4 added lines

@@ -32,6 +32,10 @@
 class Location
 {
 
+    /**
+     * @param \DOMDocument $dom
+     * @param \DOMXPath $xpath
+     */
     public function parse($dom, $xpath)
     {
         $locations = $xpath->query("//div[contains(@class, 'vevent')]/p[contains(@class, 'location')]/a"); ///a/abbr[class='dtstart']

Spacing +1 added lines, -1 removed lines

@@ -35,7 +35,7 @@
     public function parse($dom, $xpath)
     {
         $locations = $xpath->query("//div[contains(@class, 'vevent')]/p[contains(@class, 'location')]/a"); ///a/abbr[class='dtstart']
-        if (! $locations || $locations->length == 0) {
+        if (!$locations || $locations->length == 0) {
             throw new \InvalidArgumentException('The Event does not seem to have an end date');
         }
         $location = [];