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

src/Parser/Lanyrd/Tags.php

Doc Comments +4 added lines

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

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

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(
             "//strong[contains(@class, \"call-open\")]/following-sibling::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

@@ -38,6 +38,10 @@
         $this->timezone = new \DateTimezone($timezone);
     }
 
+    /**
+     * @param \DOMDocument $dom
+     * @param \DOMXPath $xpath
+     */
     public function parse($dom, $xpath)
     {
         $closingDate = $xpath->query("//span[text()='Closes on:']/following-sibling::strong");

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

@@ -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']

src/Parser/Lanyrd/EventStartDate.php

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

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']