joindin / callingallpapers-cli

src/Writer/NullOutput.php

Doc Comments +2 added lines, -2 removed lines

@@ -144,7 +144,7 @@ 
     /**
      * Gets the decorated flag.
      *
-     * @return bool true if the output will decorate messages, false otherwise
+     * @return integer true if the output will decorate messages, false otherwise
      * @api
      */
     public function isDecorated()
@@ -166,7 +166,7 @@ 
     /**
      * Returns whether verbosity is quiet (-q).
      *
-     * @return bool true if verbosity is set to VERBOSITY_QUIET, false otherwise
+     * @return boolean|null true if verbosity is set to VERBOSITY_QUIET, false otherwise
      */
     public function isQuiet()
     {

src/Command/ParseEvents.php

Indentation +6 added lines, -6 removed lines

@@ -42,11 +42,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:
@@ -56,7 +56,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)

Spacing +2 added lines, -2 removed lines

@@ -67,11 +67,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');
         $writer = new ApiCfpWriter($config['event_api_url'], $config['event_api_token']);
         $writer->setOutput($output);
 

src/Parser/Lanyrd/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\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/Lanyrd/Uri.php

Doc Comments +4 added lines

@@ -32,6 +32,10 @@
 class Uri
 {
 
+    /**
+     * @param \DOMDocument $dom
+     * @param \DOMXPath $xpath
+     */
     public function parse($dom, $xpath)
     {
         $uri = $xpath->query(

Spacing +1 added lines, -1 removed lines

@@ -37,7 +37,7 @@
         $uri = $xpath->query(
             "//div[contains(@class, \"description\")]/following-sibling::p/a"
         );
-        if (! $uri || $uri->length == 0) {
+        if (!$uri || $uri->length == 0) {
             throw new \InvalidArgumentException('The CfP does not seem to have a CfP-Uri');
         }
 

src/Parser/Lanyrd/ClosingDate.php

Spacing +1 added lines, -1 removed lines

@@ -35,7 +35,7 @@
     public function parse($dom, $xpath)
     {
         $closingDate = $xpath->query("//span[text()='Closes on:']/following-sibling::strong");
-        if (! $closingDate || $closingDate->length == 0) {
+        if (!$closingDate || $closingDate->length == 0) {
             throw new \InvalidArgumentException('The CfP does not seem to have a closing date');
         }
 

Doc Comments +4 added lines

@@ -32,6 +32,10 @@
 class Uri
 {
 
+    /**
+     * @param \DOMDocument $dom
+     * @param \DOMXPath $xpath
+     */
     public function parse($dom, $xpath)
     {
         $uri = $xpath->query(

src/Parser/Lanyrd/Tags.php

Spacing +1 added lines, -1 removed lines

@@ -37,7 +37,7 @@
         $tags = $xpath->query(
             "//ul[contains(@class, \"tags\")]/li/a"
         );
-        if (! $tags || $tags->length == 0) {
+        if (!$tags || $tags->length == 0) {
             throw new \InvalidArgumentException('The CfP does not seem to have tags');
         }
 

Doc Comments +4 added lines

@@ -32,6 +32,10 @@
 class Uri
 {
 
+    /**
+     * @param \DOMDocument $dom
+     * @param \DOMXPath $xpath
+     */
     public function parse($dom, $xpath)
     {
         $uri = $xpath->query(

src/Parser/Lanyrd/EventEndDate.php

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

Doc Comments +4 added lines

@@ -32,6 +32,10 @@
 class Uri
 {
 
+    /**
+     * @param \DOMDocument $dom
+     * @param \DOMXPath $xpath
+     */
     public function parse($dom, $xpath)
     {
         $uri = $xpath->query(

src/Parser/Lanyrd/EventName.php

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

Doc Comments +4 added lines

@@ -32,6 +32,10 @@
 class Uri
 {
 
+    /**
+     * @param \DOMDocument $dom
+     * @param \DOMXPath $xpath
+     */
     public function parse($dom, $xpath)
     {
         $uri = $xpath->query(

src/Parser/Lanyrd/Location.php

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 = [];

Doc Comments +4 added lines

@@ -32,6 +32,10 @@
 class Uri
 {
 
+    /**
+     * @param \DOMDocument $dom
+     * @param \DOMXPath $xpath
+     */
     public function parse($dom, $xpath)
     {
         $uri = $xpath->query(