trivago / chapi

src/Entity/DatePeriod/Iso8601Entity.php

Doc Comments +1 added lines, -1 removed lines

@@ -47,7 +47,7 @@
     }
 
     /**
-     * @return array
+     * @return string[]
      */
     private function parseIsoString()
     {

Indentation +7 added lines, -7 removed lines

@@ -1,12 +1,12 @@
 <?php
 /**
- * @package: chapi
- *
- * @author:  msiebeneicher
- * @since:   2015-09-25
- *
- * @link:    https://github.com/msiebeneicher/chapi/issues/36
- */
+     * @package: chapi
+     *
+     * @author:  msiebeneicher
+     * @since:   2015-09-25
+     *
+     * @link:    https://github.com/msiebeneicher/chapi/issues/36
+     */
 
 
 namespace Chapi\Entity\DatePeriod;

src/BusinessCase/JobManagement/StoreJobBusinessCase.php

Braces +6 added lines, -12 removed lines

@@ -85,8 +85,7 @@ 
                         'Job "%s" successfully added to chronos',
                         $_sJobName
                     ));
-                }
-                else
+                } else
                 {
                     $this->oLogger->error(sprintf(
                         'Failed to add job "%s" to chronos',
@@ -109,8 +108,7 @@ 
                         'Job "%s" successfully removed from chronos',
                         $_sJobName
                     ));
-                }
-                else
+                } else
                 {
                     $this->oLogger->error(sprintf(
                         'Failed to remove job "%s" from chronos',
@@ -134,8 +132,7 @@ 
                         'Job "%s" successfully updated in chronos',
                         $_sJobName
                     ));
-                }
-                else
+                } else
                 {
                     $this->oLogger->error(sprintf(
                         'Failed to update job "%s" in chronos',
@@ -154,8 +151,7 @@ 
         if (empty($aJobNames))
         {
             $_aChronosJobs = $this->oJobRepositoryChronos->getJobs();
-        }
-        else
+        } else
         {
             $_aChronosJobs = [];
             foreach ($aJobNames as $_sJobName)
@@ -177,8 +173,7 @@ 
                         'Job "%s" successfully stored in local repository',
                         $_oJobEntity->name
                     ));
-                }
-                else
+                } else
                 {
                     $this->oLogger->error(sprintf(
                         'Failed to store job "%s" in local repository',
@@ -209,8 +204,7 @@ 
                         'Job "%s" successfully updated in local repository',
                         $_oJobEntity->name
                     ));
-                }
-                else
+                } else
                 {
                     $this->oLogger->error(sprintf(
                         'Failed to update job "%s" in local repository',

Indentation +6 added lines, -6 removed lines

@@ -1,11 +1,11 @@
 <?php
 /**
- * @package: Chapi
- *
- * @author:  msiebeneicher
- * @since:   2015-07-31
- *
- */
+     * @package: Chapi
+     *
+     * @author:  msiebeneicher
+     * @since:   2015-07-31
+     *
+     */
 
 
 namespace Chapi\Service\JobIndex;

src/ChapiApplication.php

Indentation +6 added lines, -6 removed lines

@@ -1,11 +1,11 @@
 <?php
 /**
- * @package: Chapi
- *
- * @author:  msiebeneicher
- * @since:   2015-07-28
- *
- */
+     * @package: Chapi
+     *
+     * @author:  msiebeneicher
+     * @since:   2015-07-28
+     *
+     */
 namespace Chapi;
 
 use Symfony\Component\Console\Application;

src/Commands/AbstractCommand.php

Indentation +22 added lines, -22 removed lines

@@ -1,11 +1,11 @@ 
 <?php
 /**
- * @package: chapi
- *
- * @author:  msiebeneicher
- * @since:   2015-07-21
- *
- */
+     * @package: chapi
+     *
+     * @author:  msiebeneicher
+     * @since:   2015-07-21
+     *
+     */
 
 
 namespace Chapi\Commands;
@@ -44,22 +44,22 @@ 
     private static $sHomeDir = '';
 
     /**
-    * Executes the current command.
-    *
-    * This method is not abstract because you can use this class
-    * as a concrete class. In this case, instead of defining the
-    * execute() method, you set the code to execute by passing
-    * a Closure to the setCode() method.
-    *
-    * @param InputInterface $oInput An InputInterface instance
-    * @param OutputInterface $oOutput An OutputInterface instance
-    *
-    * @return integer null or 0 if everything went fine, or an error code
-    *
-    * @throws \LogicException When this abstract method is not implemented
-    *
-    * @see setCode()
-    */
+     * Executes the current command.
+     *
+     * This method is not abstract because you can use this class
+     * as a concrete class. In this case, instead of defining the
+     * execute() method, you set the code to execute by passing
+     * a Closure to the setCode() method.
+     *
+     * @param InputInterface $oInput An InputInterface instance
+     * @param OutputInterface $oOutput An OutputInterface instance
+     *
+     * @return integer null or 0 if everything went fine, or an error code
+     *
+     * @throws \LogicException When this abstract method is not implemented
+     *
+     * @see setCode()
+     */
     protected function execute(InputInterface $oInput, OutputInterface $oOutput)
     {
         $this->oInput = $oInput;

Braces +3 added lines, -1 removed lines

@@ -125,12 +125,14 @@
         if (
             !file_exists($this->getHomeDir() . DIRECTORY_SEPARATOR . 'parameters.yml')
             AND !file_exists(getcwd() . DIRECTORY_SEPARATOR . '.chapiconfig')
-        ) // one file have to exist
+        ) {
+            // one file have to exist
         {
             $this->oOutput->writeln(sprintf(
                 '<error>%s</error>',
                 'No parameter file found. Please run "configure" command for initial setup or add a local `.chapiconfig` to your working directory.'
             ));
+        }
             return false;
         }
 

src/Commands/DiffCommand.php

Braces +3 added lines, -6 removed lines

@@ -37,8 +37,7 @@ 
         if (!empty($_sJobName))
         {
             $this->printJobDiff($_sJobName);
-        }
-        else
+        } else
         {
             $_aLocalJobUpdates = $_oJobComparisonBusinessCase->getLocalJobUpdates();
             if (!empty($_aLocalJobUpdates))
@@ -76,12 +75,10 @@ 
                 if ($_sDiffSign == '+')
                 {
                     $this->oOutput->writeln(sprintf("<info>%s\t%s: %s</info>", $_sDiffSign, $_sProperty, substr($_sDiffLine, 1)));
-                }
-                elseif ($_sDiffSign == '-')
+                } elseif ($_sDiffSign == '-')
                 {
                     $this->oOutput->writeln(sprintf("<fg=red>%s\t%s: %s</>", $_sDiffSign, $_sProperty, substr($_sDiffLine, 1)));
-                }
-                else
+                } else
                 {
                     $this->oOutput->writeln(sprintf("\t%s: %s", $_sProperty, $_sDiffLine));
                 }

Indentation +6 added lines, -6 removed lines

@@ -1,11 +1,11 @@
 <?php
 /**
- * @package: chapi
- *
- * @author:  msiebeneicher
- * @since:   2015-07-30
- *
- */
+     * @package: chapi
+     *
+     * @author:  msiebeneicher
+     * @since:   2015-07-30
+     *
+     */
 namespace Chapi\Component\Comparison;
 
 

src/Commands/InfoCommand.php

Braces +1 added lines, -2 removed lines

@@ -49,8 +49,7 @@
                 $_mValue = (!empty($_mValue))
                     ? '[ ' . implode(', ', $_mValue) . ' ]'
                     : '[ ]';
-            }
-            elseif (is_bool($_mValue))
+            } elseif (is_bool($_mValue))
             {
                 $_mValue = (true === $_mValue)
                     ? 'true'

Indentation +6 added lines, -6 removed lines

@@ -1,11 +1,11 @@
 <?php
 /**
- * @package: chapi
- *
- * @author:  msiebeneicher
- * @since:   2015-07-30
- *
- */
+     * @package: chapi
+     *
+     * @author:  msiebeneicher
+     * @since:   2015-07-30
+     *
+     */
 namespace Chapi\Component\Comparison;
 
 

src/Commands/ListCommand.php

Braces +2 added lines, -4 removed lines

@@ -50,12 +50,10 @@
                 {
 
                     $this->oOutput->writeln(sprintf("\t<fg=red>%s</>", $_oJobEntity->name));
-                }
-                elseif ($_oJobEntity->errorCount > 0)
+                } elseif ($_oJobEntity->errorCount > 0)
                 {
                     $this->oOutput->writeln(sprintf("\t<comment>%s</comment>", $_oJobEntity->name));
-                }
-                else
+                } else
                 {
                     $this->oOutput->writeln(sprintf("\t<info>%s</info>", $_oJobEntity->name));
                 }

Indentation +6 added lines, -6 removed lines

@@ -1,11 +1,11 @@
 <?php
 /**
- * @package: chapi
- *
- * @author:  msiebeneicher
- * @since:   2015-07-28
- *
- */
+     * @package: chapi
+     *
+     * @author:  msiebeneicher
+     * @since:   2015-07-28
+     *
+     */
 
 namespace Chapi\Service\JobRepository;
 

src/Commands/PullCommand.php

Indentation +6 added lines, -6 removed lines

@@ -1,11 +1,11 @@
 <?php
 /**
- * @package: chapi
- *
- * @author:  msiebeneicher
- * @since:   2015-08-02
- *
- */
+     * @package: chapi
+     *
+     * @author:  msiebeneicher
+     * @since:   2015-08-02
+     *
+     */
 
 
 namespace Chapi\Commands;

src/Commands/ResetCommand.php

Braces +1 added lines, -2 removed lines

@@ -49,8 +49,7 @@
         if (JobUtils::isWildcard($aJobNames))
         {
             $_oJobIndexService->resetJobIndex();
-        }
-        else
+        } else
         {
             $_oJobIndexService->removeJobs($aJobNames);
         }

Indentation +6 added lines, -6 removed lines

@@ -1,11 +1,11 @@
 <?php
 /**
- * @package: chapi
- *
- * @author:  msiebeneicher
- * @since:   2015-07-31
- *
- */
+     * @package: chapi
+     *
+     * @author:  msiebeneicher
+     * @since:   2015-07-31
+     *
+     */
 
 
 namespace Chapi\Service\JobRepository;