joindin / callingallpapers-cli

src/Parser/Lanyrd/EventUri.php

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');
         }
 

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')]");

src/Parser/JoindinCfpParser.php

Spacing +3 added lines, -3 removed lines

@@ -47,14 +47,14 @@ 
         $content = json_decode($content, true);
 
         foreach ($content['events'] as $event) {
-            if (! $event['cfp_url']) {
+            if (!$event['cfp_url']) {
                 continue;
             }
             $info = new Cfp();
             $info->conferenceName = $event['name'];
             $info->conferenceUri = $event['href'];
             if ($event['icon']) {
-                $info->iconUri = 'https://joind.in/inc/img/event_icons/' . $event['icon'];
+                $info->iconUri = 'https://joind.in/inc/img/event_icons/'.$event['icon'];
             }
             $info->eventEndDate = new \DateTimeImmutable($event['end_date']);
             $info->eventStartDate = new \DateTimeImmutable($event['start_date']);
@@ -66,7 +66,7 @@ 
             $info->longitude = $event['longitude'];
             $info->tags = $event['tags'];
             $info->uri = $event['cfp_url'];
-            $info->timezone = $event['tz_continent'] . '/' . $event['tz_place'];
+            $info->timezone = $event['tz_continent'].'/'.$event['tz_place'];
 
             $writer->write($info, 'joind.in');
         }

src/Parser/Lanyrd/LanyrdCfpParser.php

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');

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;

src/Service/WriterFactory.php

Spacing +2 added lines, -2 removed lines

@@ -29,7 +29,7 @@ 
 
 class WriterFactory
 {
-    public $configFile = __DIR__ . '/../../config/callingallpapers.ini';
+    public $configFile = __DIR__.'/../../config/callingallpapers.ini';
 
     public function getWriter()
     {
@@ -42,7 +42,7 @@ 
             throw new UnknownWriterException('The requested writer could not be created');
         }
         
-        if (! $writer instanceof WriterInterface) {
+        if (!$writer instanceof WriterInterface) {
             throw new InvalidWriterException('The requested writer is not of the expected type');
         }
 

src/Service/TimezoneService.php

Spacing +1 added lines, -1 removed lines

@@ -63,7 +63,7 @@
 
         $values = json_decode($result->getBody()->getContents(), true);
 
-        if (! isset($values['status']) || $values['status'] !== 'OK') {
+        if (!isset($values['status']) || $values['status'] !== 'OK') {
             return 'UTC';
         }
 

src/Reader/ApiCfpReader.php

Spacing +2 added lines, -2 removed lines

@@ -92,7 +92,7 @@ 
             return $list;
         }
 
-        if (! $items) {
+        if (!$items) {
             return $list;
         }
 
@@ -110,7 +110,7 @@ 
             $cfp->longitude      = $item['longitude'];
             $cfp->iconUri        = $item['iconUri'];
             $cfp->uri            = $item['uri'];
-            $cfp->tags           = array_filter($item['tags'], function ($item) {
+            $cfp->tags           = array_filter($item['tags'], function($item) {
                 return (bool) $item;
             });
             $cfp->timezone       = $item['timezone'];

src/Command/NotifyClosingCfps.php

Indentation +6 added lines, -6 removed lines

@@ -45,18 +45,18 @@
     protected function configure()
     {
         $this->setName("notifyClosingCfps")
-             ->setDescription("Notify about CfPs that close within 24 hours")
-             ->setDefinition(array(
-                 new InputOption('start', 's', InputOption::VALUE_OPTIONAL, 'What should be the first date to be taken into account?', ''),
-             ))
-             ->setHelp(<<<EOT
+                ->setDescription("Notify about CfPs that close within 24 hours")
+                ->setDefinition(array(
+                    new InputOption('start', 's', InputOption::VALUE_OPTIONAL, 'What should be the first date to be taken into account?', ''),
+                ))
+                ->setHelp(<<<EOT
 Notify about CfPs that are closing within 24 hours
 
 Usage:
 
 <info>callingallpapers notifyClosingCfp</info>
 EOT
-             );
+                );
     }
 
     protected function execute(InputInterface $input, OutputInterface $output)

Spacing +2 added lines, -2 removed lines

@@ -66,11 +66,11 @@
 
         $start = new \DateTime($input->getOption('start'));
 
-        if (! $start instanceof \DateTime) {
+        if (!$start instanceof \DateTime) {
             throw new \InvalidArgumentException('The given date could not be parsed');
         }
 
-        $config = parse_ini_file(__DIR__ . '/../../config/callingallpapers.ini');
+        $config = parse_ini_file(__DIR__.'/../../config/callingallpapers.ini');
         $reader = new ApiCfpReader($config['event_api_url'], $config['event_api_token']);
         $cfps = $reader->getCfpsEndingWithinInterval(
             new \DateInterval('PT1H'),

src/Entity/Cfp.php

Spacing +1 added lines, -1 removed lines

@@ -105,7 +105,7 @@
 
     public function addTag($tag)
     {
-        if (! in_array($tag, $this->tags)) {
+        if (!in_array($tag, $this->tags)) {
             $this->tags[] = $tag;
         }
     }

src/Parser/Lanyrd/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');
         }