corretgecom / PHPCI

PHPCI/Controller/BuildController.php

Doc Comments +1 added lines

@@ -233,6 +233,7 @@
 
     /**
      * Parse log for unix colours and replace with HTML.
+     * @param string $log
      */
     protected function cleanLog($log)
     {

Indentation +7 added lines, -7 removed lines

@@ -1,12 +1,12 @@
 <?php
 /**
- * PHPCI - Continuous Integration for PHP.
- *
- * @copyright    Copyright 2014, Block 8 Limited.
- * @license      https://github.com/Block8/PHPCI/blob/master/LICENSE.md
- *
- * @link         https://www.phptesting.org/
- */
+     * PHPCI - Continuous Integration for PHP.
+     *
+     * @copyright    Copyright 2014, Block 8 Limited.
+     * @license      https://github.com/Block8/PHPCI/blob/master/LICENSE.md
+     *
+     * @link         https://www.phptesting.org/
+     */
 
 namespace PHPCI\Controller;
 

Spacing +1 added lines, -1 removed lines

@@ -169,7 +169,7 @@
     protected function getBuildData(Build $build)
     {
         $data = array();
-        $data['status'] = (int) $build->getStatus();
+        $data['status'] = (int)$build->getStatus();
         $data['log'] = $this->cleanLog($build->getLog());
         $data['created'] = !is_null($build->getCreated()) ? $build->getCreated()->format('Y-m-d H:i:s') : null;
         $data['started'] = !is_null($build->getStarted()) ? $build->getStarted()->format('Y-m-d H:i:s') : null;

PHPCI/Model/Base/BuildBase.php

Doc Comments +6 added lines, -6 removed lines

@@ -425,7 +425,7 @@ 
     /**
      * Set the value of Log / log.
      *
-     * @param $value string
+     * @param string $value string
      */
     public function setLog($value)
     {
@@ -464,7 +464,7 @@ 
     /**
      * Set the value of Created / created.
      *
-     * @param $value \DateTime
+     * @param \DateTime $value \DateTime
      */
     public function setCreated($value)
     {
@@ -482,7 +482,7 @@ 
     /**
      * Set the value of Started / started.
      *
-     * @param $value \DateTime
+     * @param \DateTime $value \DateTime
      */
     public function setStarted($value)
     {
@@ -500,7 +500,7 @@ 
     /**
      * Set the value of Finished / finished.
      *
-     * @param $value \DateTime
+     * @param \DateTime $value \DateTime
      */
     public function setFinished($value)
     {
@@ -554,7 +554,7 @@ 
     /**
      * Set the value of Extra / extra.
      *
-     * @param $value string
+     * @param string $value string
      */
     public function setExtra($value)
     {
@@ -599,7 +599,7 @@ 
     /**
      * Set Project - Accepts an ID, an array representing a Project or a Project model.
      *
-     * @param $value mixed
+     * @param Model\Project $value mixed
      */
     public function setProject($value)
     {

PHPCI/Model/Base/BuildErrorBase.php

Doc Comments +2 added lines, -2 removed lines

@@ -434,7 +434,7 @@ 
      *
      * Must not be null.
      *
-     * @param $value \DateTime
+     * @param \DateTime $value \DateTime
      */
     public function setCreatedDate($value)
     {
@@ -480,7 +480,7 @@ 
     /**
      * Set Build - Accepts an ID, an array representing a Build or a Build model.
      *
-     * @param $value mixed
+     * @param Model\Build $value mixed
      */
     public function setBuild($value)
     {

PHPCI/Model/Base/ProjectBase.php

Doc Comments +6 added lines, -6 removed lines

@@ -372,7 +372,7 @@ 
      *
      * Must not be null.
      *
-     * @param $value string
+     * @param string $value string
      */
     public function setTitle($value)
     {
@@ -393,7 +393,7 @@ 
      *
      * Must not be null.
      *
-     * @param $value string
+     * @param string $value string
      */
     public function setReference($value)
     {
@@ -453,7 +453,7 @@ 
      *
      * Must not be null.
      *
-     * @param $value string
+     * @param string $value string
      */
     public function setType($value)
     {
@@ -472,7 +472,7 @@ 
     /**
      * Set the value of AccessInformation / access_information.
      *
-     * @param $value string
+     * @param string $value string
      */
     public function setAccessInformation($value)
     {
@@ -546,7 +546,7 @@ 
      *
      * Must not be null.
      *
-     * @param $value int
+     * @param integer $value int
      */
     public function setAllowPublicStatus($value)
     {
@@ -567,7 +567,7 @@ 
      *
      * Must not be null.
      *
-     * @param $value int
+     * @param boolean $value int
      */
     public function setArchived($value)
     {

PHPCI/Plugin/Codeception.php

Doc Comments +3 added lines, -3 removed lines

@@ -59,7 +59,7 @@ 
     /**
      * Try and find the codeception YML config file.
      *
-     * @param $buildPath
+     * @param string $buildPath
      *
      * @return null|string
      */
@@ -118,9 +118,9 @@ 
     /**
      * Run tests from a Codeception config file.
      *
-     * @param $configPath
+     * @param string $configPath
      *
-     * @return bool|mixed
+     * @return boolean
      *
      * @throws \Exception
      */

Indentation +7 added lines, -7 removed lines

@@ -1,12 +1,12 @@
 <?php
 /**
- * PHPCI - Continuous Integration for PHP.
- *
- * @copyright    Copyright 2014, Block 8 Limited.
- * @license      https://github.com/Block8/PHPCI/blob/master/LICENSE.md
- *
- * @link         https://www.phptesting.org/
- */
+     * PHPCI - Continuous Integration for PHP.
+     *
+     * @copyright    Copyright 2014, Block 8 Limited.
+     * @license      https://github.com/Block8/PHPCI/blob/master/LICENSE.md
+     *
+     * @link         https://www.phptesting.org/
+     */
 
 namespace PHPCI\Plugin;
 

Spacing +1 added lines, -1 removed lines

@@ -95,7 +95,7 @@
             $this->ymlConfigFile = $options['config'];
         }
         if (isset($options['args'])) {
-            $this->args = (string) $options['args'];
+            $this->args = (string)$options['args'];
         }
         if (isset($options['path'])) {
             $this->path = $options['path'];

PHPCI/Plugin/Irc.php

Doc Comments +3 added lines, -3 removed lines

@@ -94,9 +94,9 @@ 
 
     /**
      * @param resource $socket
-     * @param array    $commands
+     * @param string[]    $commands
      *
-     * @return bool
+     * @return boolean|null
      */
     private function executeIrcCommands($socket, array $commands)
     {
@@ -124,7 +124,7 @@ 
      * @param resource $socket
      * @param string   $command
      *
-     * @return bool
+     * @return boolean|null
      */
     private function executeIrcCommand($socket, $command)
     {

Indentation +7 added lines, -7 removed lines

@@ -1,12 +1,12 @@
 <?php
 /**
- * PHPCI - Continuous Integration for PHP.
- *
- * @copyright    Copyright 2014, Block 8 Limited.
- * @license      https://github.com/Block8/PHPCI/blob/master/LICENSE.md
- *
- * @link         https://www.phptesting.org/
- */
+     * PHPCI - Continuous Integration for PHP.
+     *
+     * @copyright    Copyright 2014, Block 8 Limited.
+     * @license      https://github.com/Block8/PHPCI/blob/master/LICENSE.md
+     *
+     * @link         https://www.phptesting.org/
+     */
 
 namespace PHPCI\Plugin;
 

PHPCI/Plugin/PhpCpd.php

Doc Comments +2 added lines, -2 removed lines

@@ -110,9 +110,9 @@
     /**
      * Process the PHPCPD XML report.
      *
-     * @param $xmlString
+     * @param string $xmlString
      *
-     * @return array
+     * @return integer
      *
      * @throws \Exception
      */

Indentation +7 added lines, -7 removed lines

@@ -1,12 +1,12 @@
 <?php
 /**
- * PHPCI - Continuous Integration for PHP.
- *
- * @copyright    Copyright 2014, Block 8 Limited.
- * @license      https://github.com/Block8/PHPCI/blob/master/LICENSE.md
- *
- * @link         https://www.phptesting.org/
- */
+     * PHPCI - Continuous Integration for PHP.
+     *
+     * @copyright    Copyright 2014, Block 8 Limited.
+     * @license      https://github.com/Block8/PHPCI/blob/master/LICENSE.md
+     *
+     * @link         https://www.phptesting.org/
+     */
 
 namespace PHPCI\Plugin;
 

Spacing +3 added lines, -3 removed lines

@@ -74,7 +74,7 @@ 
     {
         $ignore = '';
         if (count($this->ignore)) {
-            $map = function ($item) {
+            $map = function($item) {
                 // remove the trailing slash
                 $item = rtrim($item, DIRECTORY_SEPARATOR);
 
@@ -128,7 +128,7 @@ 
         $warnings = 0;
         foreach ($xml->duplication as $duplication) {
             foreach ($duplication->file as $file) {
-                $fileName = (string) $file['path'];
+                $fileName = (string)$file['path'];
                 $fileName = str_replace($this->phpci->buildPath, '', $fileName);
 
                 $message = <<<CPD
@@ -146,7 +146,7 @@ 
                     BuildError::SEVERITY_NORMAL,
                     $fileName,
                     $file['line'],
-                    (int) $file['line'] + (int) $duplication['lines']
+                    (int)$file['line'] + (int)$duplication['lines']
                 );
             }
 

PHPCI/Plugin/PhpMessDetector.php

Doc Comments +4 added lines, -4 removed lines

@@ -136,7 +136,7 @@ 
      * Override a default setting.
      *
      * @param $options
-     * @param $key
+     * @param string $key
      */
     protected function overrideSetting($options, $key)
     {
@@ -148,9 +148,9 @@ 
     /**
      * Process PHPMD's XML output report.
      *
-     * @param $xmlString
+     * @param string $xmlString
      *
-     * @return array
+     * @return integer
      *
      * @throws \Exception
      */
@@ -212,7 +212,7 @@ 
     /**
      * Execute PHP Mess Detector.
      *
-     * @param $binaryPath
+     * @param null|string $binaryPath
      */
     protected function executePhpMd($binaryPath)
     {

Indentation +7 added lines, -7 removed lines

@@ -1,12 +1,12 @@
 <?php
 /**
- * PHPCI - Continuous Integration for PHP.
- *
- * @copyright    Copyright 2014, Block 8 Limited.
- * @license      https://github.com/Block8/PHPCI/blob/master/LICENSE.md
- *
- * @link         https://www.phptesting.org/
- */
+     * PHPCI - Continuous Integration for PHP.
+     *
+     * @copyright    Copyright 2014, Block 8 Limited.
+     * @license      https://github.com/Block8/PHPCI/blob/master/LICENSE.md
+     *
+     * @link         https://www.phptesting.org/
+     */
 
 namespace PHPCI\Plugin;
 

Spacing +5 added lines, -5 removed lines

@@ -105,7 +105,7 @@ 
         }
 
         if (array_key_exists('allowed_warnings', $options)) {
-            $this->allowed_warnings = (int) $options['allowed_warnings'];
+            $this->allowed_warnings = (int)$options['allowed_warnings'];
         }
 
         foreach (array('rules', 'ignore', 'suffixes') as $key) {
@@ -166,7 +166,7 @@ 
         $warnings = 0;
 
         foreach ($xml->file as $file) {
-            $fileName = (string) $file['name'];
+            $fileName = (string)$file['name'];
             $fileName = str_replace($this->phpci->buildPath, '', $fileName);
 
             foreach ($file->violation as $violation) {
@@ -175,11 +175,11 @@ 
                 $this->build->reportError(
                     $this->phpci,
                     'php_mess_detector',
-                    (string) $violation,
+                    (string)$violation,
                     PHPCI\Model\BuildError::SEVERITY_HIGH,
                     $fileName,
-                    (int) $violation['beginline'],
-                    (int) $violation['endline']
+                    (int)$violation['beginline'],
+                    (int)$violation['endline']
                 );
             }
         }

PHPCI/Helper/MailerFactory.php

Indentation +7 added lines, -7 removed lines

@@ -1,12 +1,12 @@
 <?php
 /**
- * PHPCI - Continuous Integration for PHP.
- *
- * @copyright    Copyright 2014, Block 8 Limited.
- * @license      https://github.com/Block8/PHPCI/blob/master/LICENSE.md
- *
- * @link         https://www.phptesting.org/
- */
+     * PHPCI - Continuous Integration for PHP.
+     *
+     * @copyright    Copyright 2014, Block 8 Limited.
+     * @license      https://github.com/Block8/PHPCI/blob/master/LICENSE.md
+     *
+     * @link         https://www.phptesting.org/
+     */
 
 namespace PHPCI\Helper;