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/OpeningDate.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)
     {
         $openingDate = $xpath->query("//span[text()='Openend on:']/following-sibling::strong");

Spacing +1 added lines, -1 removed lines

@@ -41,7 +41,7 @@
     public function parse($dom, $xpath)
     {
         $openingDate = $xpath->query("//span[text()='Openend on:']/following-sibling::strong");
-        if (! $openingDate || $openingDate->length == 0) {
+        if (!$openingDate || $openingDate->length == 0) {
             throw new \UnexpectedValueException('No CfP-Open Date found');
         }
 

src/Parser/PapercallIo/ClosingDate.php

Spacing +1 added lines, -1 removed lines

@@ -50,7 +50,7 @@
     {
         $xpath = new DOMXPath($node->ownerDocument);
         $closingDate = $xpath->query(".//time/@datetime", $node);
-        if (! $closingDate || $closingDate->length == 0) {
+        if (!$closingDate || $closingDate->length == 0) {
             throw new InvalidArgumentException('The CfP does not seem to have a closing date');
         }
 

src/Parser/PapercallIoParser.php

Spacing +1 added lines, -1 removed lines

@@ -66,7 +66,7 @@
         do {
             $dom = new DOMDocument('1.0', 'UTF-8');
             libxml_use_internal_errors(true);
-            $uri = sprintf($this->uri, $i+1);
+            $uri = sprintf($this->uri, $i + 1);
             $dom->loadHTMLFile($uri);
             libxml_use_internal_errors(false);
             $dom->preserveWhiteSpace = false;

src/Parser/PapercallIo/EventEndDate.php

Unused Use Statements -1 removed lines

@@ -32,7 +32,6 @@
 use Callingallpapers\Entity\Cfp;
 use Callingallpapers\Parser\EventDetailParserInterface;
 use DateTimeImmutable;
-use DateTimeZone;
 use DOMDocument;
 use DOMNode;
 use DOMXPath;

Spacing +6 added lines, -6 removed lines

@@ -51,30 +51,30 @@
         $xpath = new DOMXPath($dom);
         $titlePath = $xpath->query("//h1[contains(@class, 'subheader__subtitle')]");
 
-        if (! $titlePath || $titlePath->length == 0) {
+        if (!$titlePath || $titlePath->length == 0) {
             return $cfp;
         }
 
         $locationTimeString = trim($titlePath->item(0)->textContent);
         $locationTime = explode(' - ', $locationTimeString);
 
-        if (! isset($locationTime[1])) {
+        if (!isset($locationTime[1])) {
             return $cfp;
         }
 
         $dates = explode(',', $locationTime[1]);
-        if (count($dates) % 2  !== 0) {
+        if (count($dates) % 2 !== 0) {
         var_Dump($dates);
             return $cfp;
         }
 
-        $datestring = $dates[0] . ', ' . $dates[1];
+        $datestring = $dates[0].', '.$dates[1];
 
         if (count($dates) >= 4) {
-            $datestring = $dates[2] . ', ' . $dates[3];
+            $datestring = $dates[2].', '.$dates[3];
         }
 
-        $endDate = new DateTimeImmutable($datestring . ' 00:00:00', $this->timezone );
+        $endDate = new DateTimeImmutable($datestring.' 00:00:00', $this->timezone);
         $cfp->eventEndDate = $endDate;
 
         return $cfp;

src/Parser/PapercallIo/Location.php

Spacing +1 added lines, -1 removed lines

@@ -43,7 +43,7 @@
         $xpath = new DOMXPath($dom);
         $titlePath = $xpath->query("//h1[contains(@class, 'subheader__subtitle')]");
 
-        if (! $titlePath || $titlePath->length == 0) {
+        if (!$titlePath || $titlePath->length == 0) {
             return $cfp;
         }
 

src/Parser/PapercallIo/Tags.php

Spacing +1 added lines, -1 removed lines

@@ -43,7 +43,7 @@
         $xpath = new DOMXPath($dom);
 
         $tagsPath = $xpath->query("//h1/following-sibling::a/following-sibling::span");
-        if (! $tagsPath || $tagsPath->length == 0) {
+        if (!$tagsPath || $tagsPath->length == 0) {
             throw new \InvalidArgumentException('The CfP does not seem to have tags');
         }
 

src/Parser/PapercallIo/EventName.php

Spacing +1 added lines, -1 removed lines

@@ -44,7 +44,7 @@
         $xpath = new DOMXPath($dom);
         $titlePath = $xpath->query("//h1[contains(@class, 'subheader__title')]");
 
-        if (! $titlePath || $titlePath->length == 0) {
+        if (!$titlePath || $titlePath->length == 0) {
             throw new \InvalidArgumentException('The CfP does not seem to have an eventname');
         }
 

src/Parser/PapercallIo/EventStartDate.php

Spacing +4 added lines, -4 removed lines

@@ -51,19 +51,19 @@ 
         $xpath = new DOMXPath($dom);
         $titlePath = $xpath->query("//h1[contains(@class, 'subheader__subtitle')]");
 
-        if (! $titlePath || $titlePath->length == 0) {
+        if (!$titlePath || $titlePath->length == 0) {
             return $cfp;
         }
 
         $location = trim($titlePath->item(0)->textContent);
         $location = explode(' - ', $location);
 
-        if (! isset($location[1])) {
+        if (!isset($location[1])) {
             return $cfp;
         }
 
         $dates = explode(',', $location[1]);
-        if (count($dates) % 2  !== 0) {
+        if (count($dates) % 2 !== 0) {
             return $cfp;
         }
 
@@ -71,7 +71,7 @@ 
             return $cfp;
         }
 
-        $startDate = new DateTimeImmutable($dates[0] . ', ' . $dates[1] . ' 00:00:00', $this->timezone );
+        $startDate = new DateTimeImmutable($dates[0].', '.$dates[1].' 00:00:00', $this->timezone);
         $cfp->eventStartDate = $startDate;
 
         return $cfp;