corretgecom / PHPCI

PHPCI/Model/Base/BuildBase.php

Doc Comments +5 added lines, -5 removed lines

@@ -421,7 +421,7 @@ 
     /**
     * Set the value of Log / log.
     *
-    * @param $value string
+    * @param string $value string
     */
     public function setLog($value)
     {
@@ -459,7 +459,7 @@ 
     /**
     * Set the value of Created / created.
     *
-    * @param $value \DateTime
+    * @param \DateTime $value \DateTime
     */
     public function setCreated($value)
     {
@@ -477,7 +477,7 @@ 
     /**
     * Set the value of Started / started.
     *
-    * @param $value \DateTime
+    * @param \DateTime $value \DateTime
     */
     public function setStarted($value)
     {
@@ -495,7 +495,7 @@ 
     /**
     * Set the value of Finished / finished.
     *
-    * @param $value \DateTime
+    * @param \DateTime $value \DateTime
     */
     public function setFinished($value)
     {
@@ -549,7 +549,7 @@ 
     /**
     * Set the value of Extra / extra.
     *
-    * @param $value string
+    * @param string $value string
     */
     public function setExtra($value)
     {

Indentation +127 added lines, -127 removed lines

@@ -15,23 +15,23 @@ 
 class BuildBase extends Model
 {
     /**
-    * @var array
-    */
+     * @var array
+     */
     public static $sleepable = array();
 
     /**
-    * @var string
-    */
+     * @var string
+     */
     protected $tableName = 'build';
 
     /**
-    * @var string
-    */
+     * @var string
+     */
     protected $modelName = 'Build';
 
     /**
-    * @var array
-    */
+     * @var array
+     */
     protected $data = array(
         'id' => null,
         'project_id' => null,
@@ -48,8 +48,8 @@ 
     );
 
     /**
-    * @var array
-    */
+     * @var array
+     */
     protected $getters = array(
         // Direct property getters:
         'id' => 'getId',
@@ -70,8 +70,8 @@ 
     );
 
     /**
-    * @var array
-    */
+     * @var array
+     */
     protected $setters = array(
         // Direct property setters:
         'id' => 'setId',
@@ -92,8 +92,8 @@ 
     );
 
     /**
-    * @var array
-    */
+     * @var array
+     */
     public $columns = array(
         'id' => array(
             'type' => 'int',
@@ -161,8 +161,8 @@ 
     );
 
     /**
-    * @var array
-    */
+     * @var array
+     */
     public $indexes = array(
             'PRIMARY' => array('unique' => true, 'columns' => 'id'),
             'project_id' => array('columns' => 'project_id'),
@@ -170,8 +170,8 @@ 
     );
 
     /**
-    * @var array
-    */
+     * @var array
+     */
     public $foreignKeys = array(
             'build_ibfk_1' => array(
                 'local_col' => 'project_id',
@@ -183,10 +183,10 @@ 
     );
 
     /**
-    * Get the value of Id / id.
-    *
-    * @return int
-    */
+     * Get the value of Id / id.
+     *
+     * @return int
+     */
     public function getId()
     {
         $rtn    = $this->data['id'];
@@ -195,10 +195,10 @@ 
     }
 
     /**
-    * Get the value of ProjectId / project_id.
-    *
-    * @return int
-    */
+     * Get the value of ProjectId / project_id.
+     *
+     * @return int
+     */
     public function getProjectId()
     {
         $rtn    = $this->data['project_id'];
@@ -207,10 +207,10 @@ 
     }
 
     /**
-    * Get the value of CommitId / commit_id.
-    *
-    * @return string
-    */
+     * Get the value of CommitId / commit_id.
+     *
+     * @return string
+     */
     public function getCommitId()
     {
         $rtn    = $this->data['commit_id'];
@@ -219,10 +219,10 @@ 
     }
 
     /**
-    * Get the value of Status / status.
-    *
-    * @return int
-    */
+     * Get the value of Status / status.
+     *
+     * @return int
+     */
     public function getStatus()
     {
         $rtn    = $this->data['status'];
@@ -231,10 +231,10 @@ 
     }
 
     /**
-    * Get the value of Log / log.
-    *
-    * @return string
-    */
+     * Get the value of Log / log.
+     *
+     * @return string
+     */
     public function getLog()
     {
         $rtn    = $this->data['log'];
@@ -243,10 +243,10 @@ 
     }
 
     /**
-    * Get the value of Branch / branch.
-    *
-    * @return string
-    */
+     * Get the value of Branch / branch.
+     *
+     * @return string
+     */
     public function getBranch()
     {
         $rtn    = $this->data['branch'];
@@ -255,10 +255,10 @@ 
     }
 
     /**
-    * Get the value of Created / created.
-    *
-    * @return \DateTime
-    */
+     * Get the value of Created / created.
+     *
+     * @return \DateTime
+     */
     public function getCreated()
     {
         $rtn    = $this->data['created'];
@@ -271,10 +271,10 @@ 
     }
 
     /**
-    * Get the value of Started / started.
-    *
-    * @return \DateTime
-    */
+     * Get the value of Started / started.
+     *
+     * @return \DateTime
+     */
     public function getStarted()
     {
         $rtn    = $this->data['started'];
@@ -287,10 +287,10 @@ 
     }
 
     /**
-    * Get the value of Finished / finished.
-    *
-    * @return \DateTime
-    */
+     * Get the value of Finished / finished.
+     *
+     * @return \DateTime
+     */
     public function getFinished()
     {
         $rtn    = $this->data['finished'];
@@ -303,10 +303,10 @@ 
     }
 
     /**
-    * Get the value of CommitterEmail / committer_email.
-    *
-    * @return string
-    */
+     * Get the value of CommitterEmail / committer_email.
+     *
+     * @return string
+     */
     public function getCommitterEmail()
     {
         $rtn    = $this->data['committer_email'];
@@ -315,10 +315,10 @@ 
     }
 
     /**
-    * Get the value of CommitMessage / commit_message.
-    *
-    * @return string
-    */
+     * Get the value of CommitMessage / commit_message.
+     *
+     * @return string
+     */
     public function getCommitMessage()
     {
         $rtn    = $this->data['commit_message'];
@@ -327,10 +327,10 @@ 
     }
 
     /**
-    * Get the value of Extra / extra.
-    *
-    * @return string
-    */
+     * Get the value of Extra / extra.
+     *
+     * @return string
+     */
     public function getExtra()
     {
         $rtn    = $this->data['extra'];
@@ -339,11 +339,11 @@ 
     }
 
     /**
-    * Set the value of Id / id.
-    *
-    * Must not be null.
-    * @param $value int
-    */
+     * Set the value of Id / id.
+     *
+     * Must not be null.
+     * @param $value int
+     */
     public function setId($value)
     {
         $this->_validateNotNull('Id', $value);
@@ -359,11 +359,11 @@ 
     }
 
     /**
-    * Set the value of ProjectId / project_id.
-    *
-    * Must not be null.
-    * @param $value int
-    */
+     * Set the value of ProjectId / project_id.
+     *
+     * Must not be null.
+     * @param $value int
+     */
     public function setProjectId($value)
     {
         $this->_validateNotNull('ProjectId', $value);
@@ -379,11 +379,11 @@ 
     }
 
     /**
-    * Set the value of CommitId / commit_id.
-    *
-    * Must not be null.
-    * @param $value string
-    */
+     * Set the value of CommitId / commit_id.
+     *
+     * Must not be null.
+     * @param $value string
+     */
     public function setCommitId($value)
     {
         $this->_validateNotNull('CommitId', $value);
@@ -399,11 +399,11 @@ 
     }
 
     /**
-    * Set the value of Status / status.
-    *
-    * Must not be null.
-    * @param $value int
-    */
+     * Set the value of Status / status.
+     *
+     * Must not be null.
+     * @param $value int
+     */
     public function setStatus($value)
     {
         $this->_validateNotNull('Status', $value);
@@ -419,10 +419,10 @@ 
     }
 
     /**
-    * Set the value of Log / log.
-    *
-    * @param $value string
-    */
+     * Set the value of Log / log.
+     *
+     * @param $value string
+     */
     public function setLog($value)
     {
         $this->_validateString('Log', $value);
@@ -437,11 +437,11 @@ 
     }
 
     /**
-    * Set the value of Branch / branch.
-    *
-    * Must not be null.
-    * @param $value string
-    */
+     * Set the value of Branch / branch.
+     *
+     * Must not be null.
+     * @param $value string
+     */
     public function setBranch($value)
     {
         $this->_validateNotNull('Branch', $value);
@@ -457,10 +457,10 @@ 
     }
 
     /**
-    * Set the value of Created / created.
-    *
-    * @param $value \DateTime
-    */
+     * Set the value of Created / created.
+     *
+     * @param $value \DateTime
+     */
     public function setCreated($value)
     {
         $this->_validateDate('Created', $value);
@@ -475,10 +475,10 @@ 
     }
 
     /**
-    * Set the value of Started / started.
-    *
-    * @param $value \DateTime
-    */
+     * Set the value of Started / started.
+     *
+     * @param $value \DateTime
+     */
     public function setStarted($value)
     {
         $this->_validateDate('Started', $value);
@@ -493,10 +493,10 @@ 
     }
 
     /**
-    * Set the value of Finished / finished.
-    *
-    * @param $value \DateTime
-    */
+     * Set the value of Finished / finished.
+     *
+     * @param $value \DateTime
+     */
     public function setFinished($value)
     {
         $this->_validateDate('Finished', $value);
@@ -511,10 +511,10 @@ 
     }
 
     /**
-    * Set the value of CommitterEmail / committer_email.
-    *
-    * @param $value string
-    */
+     * Set the value of CommitterEmail / committer_email.
+     *
+     * @param $value string
+     */
     public function setCommitterEmail($value)
     {
         $this->_validateString('CommitterEmail', $value);
@@ -529,10 +529,10 @@ 
     }
 
     /**
-    * Set the value of CommitMessage / commit_message.
-    *
-    * @param $value string
-    */
+     * Set the value of CommitMessage / commit_message.
+     *
+     * @param $value string
+     */
     public function setCommitMessage($value)
     {
         $this->_validateString('CommitMessage', $value);
@@ -547,10 +547,10 @@ 
     }
 
     /**
-    * Set the value of Extra / extra.
-    *
-    * @param $value string
-    */
+     * Set the value of Extra / extra.
+     *
+     * @param $value string
+     */
     public function setExtra($value)
     {
         $this->_validateString('Extra', $value);
@@ -591,10 +591,10 @@ 
     }
 
     /**
-    * Set Project - Accepts an ID, an array representing a Project or a Project model.
-    *
-    * @param $value mixed
-    */
+     * Set Project - Accepts an ID, an array representing a Project or a Project model.
+     *
+     * @param $value mixed
+     */
     public function setProject($value)
     {
         // Is this an instance of Project?
@@ -612,10 +612,10 @@ 
     }
 
     /**
-    * Set Project - Accepts a Project model.
-    * 
-    * @param $value \PHPCI\Model\Project
-    */
+     * Set Project - Accepts a Project model.
+     * 
+     * @param $value \PHPCI\Model\Project
+     */
     public function setProjectObject(\PHPCI\Model\Project $value)
     {
         return $this->setProjectId($value->getId());

Spacing +16 added lines, -16 removed lines

@@ -189,7 +189,7 @@ 
     */
     public function getId()
     {
-        $rtn    = $this->data['id'];
+        $rtn = $this->data['id'];
 
         return $rtn;
     }
@@ -201,7 +201,7 @@ 
     */
     public function getProjectId()
     {
-        $rtn    = $this->data['project_id'];
+        $rtn = $this->data['project_id'];
 
         return $rtn;
     }
@@ -213,7 +213,7 @@ 
     */
     public function getCommitId()
     {
-        $rtn    = $this->data['commit_id'];
+        $rtn = $this->data['commit_id'];
 
         return $rtn;
     }
@@ -225,7 +225,7 @@ 
     */
     public function getStatus()
     {
-        $rtn    = $this->data['status'];
+        $rtn = $this->data['status'];
 
         return $rtn;
     }
@@ -237,7 +237,7 @@ 
     */
     public function getLog()
     {
-        $rtn    = $this->data['log'];
+        $rtn = $this->data['log'];
 
         return $rtn;
     }
@@ -249,7 +249,7 @@ 
     */
     public function getBranch()
     {
-        $rtn    = $this->data['branch'];
+        $rtn = $this->data['branch'];
 
         return $rtn;
     }
@@ -261,10 +261,10 @@ 
     */
     public function getCreated()
     {
-        $rtn    = $this->data['created'];
+        $rtn = $this->data['created'];
 
         if (!empty($rtn)) {
-            $rtn    = new \DateTime($rtn);
+            $rtn = new \DateTime($rtn);
         }
         
         return $rtn;
@@ -277,10 +277,10 @@ 
     */
     public function getStarted()
     {
-        $rtn    = $this->data['started'];
+        $rtn = $this->data['started'];
 
         if (!empty($rtn)) {
-            $rtn    = new \DateTime($rtn);
+            $rtn = new \DateTime($rtn);
         }
         
         return $rtn;
@@ -293,10 +293,10 @@ 
     */
     public function getFinished()
     {
-        $rtn    = $this->data['finished'];
+        $rtn = $this->data['finished'];
 
         if (!empty($rtn)) {
-            $rtn    = new \DateTime($rtn);
+            $rtn = new \DateTime($rtn);
         }
         
         return $rtn;
@@ -309,7 +309,7 @@ 
     */
     public function getCommitterEmail()
     {
-        $rtn    = $this->data['committer_email'];
+        $rtn = $this->data['committer_email'];
 
         return $rtn;
     }
@@ -321,7 +321,7 @@ 
     */
     public function getCommitMessage()
     {
-        $rtn    = $this->data['commit_message'];
+        $rtn = $this->data['commit_message'];
 
         return $rtn;
     }
@@ -333,7 +333,7 @@ 
     */
     public function getExtra()
     {
-        $rtn    = $this->data['extra'];
+        $rtn = $this->data['extra'];
 
         return $rtn;
     }
@@ -579,7 +579,7 @@ 
             return null;
         }
 
-        $cacheKey   = 'Cache.Project.' . $key;
+        $cacheKey   = 'Cache.Project.'.$key;
         $rtn        = $this->cache->get($cacheKey, null);
 
         if (empty($rtn)) {

PHPCI/Model/Base/BuildErrorBase.php

Doc Comments +2 added lines, -2 removed lines

@@ -428,7 +428,7 @@ 
     * Set the value of CreatedDate / created_date.
     *
     * Must not be null.
-    * @param $value \DateTime
+    * @param \DateTime $value \DateTime
     */
     public function setCreatedDate($value)
     {
@@ -473,7 +473,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)
     {

Indentation +104 added lines, -104 removed lines

@@ -15,23 +15,23 @@ 
 class BuildErrorBase extends Model
 {
     /**
-    * @var array
-    */
+     * @var array
+     */
     public static $sleepable = array();
 
     /**
-    * @var string
-    */
+     * @var string
+     */
     protected $tableName = 'build_error';
 
     /**
-    * @var string
-    */
+     * @var string
+     */
     protected $modelName = 'BuildError';
 
     /**
-    * @var array
-    */
+     * @var array
+     */
     protected $data = array(
         'id' => null,
         'build_id' => null,
@@ -45,8 +45,8 @@ 
     );
 
     /**
-    * @var array
-    */
+     * @var array
+     */
     protected $getters = array(
         // Direct property getters:
         'id' => 'getId',
@@ -64,8 +64,8 @@ 
     );
 
     /**
-    * @var array
-    */
+     * @var array
+     */
     protected $setters = array(
         // Direct property setters:
         'id' => 'setId',
@@ -83,8 +83,8 @@ 
     );
 
     /**
-    * @var array
-    */
+     * @var array
+     */
     public $columns = array(
         'id' => array(
             'type' => 'int',
@@ -138,16 +138,16 @@ 
     );
 
     /**
-    * @var array
-    */
+     * @var array
+     */
     public $indexes = array(
             'PRIMARY' => array('unique' => true, 'columns' => 'id'),
             'build_id' => array('columns' => 'build_id, created_date'),
     );
 
     /**
-    * @var array
-    */
+     * @var array
+     */
     public $foreignKeys = array(
             'build_error_ibfk_1' => array(
                 'local_col' => 'build_id',
@@ -159,10 +159,10 @@ 
     );
 
     /**
-    * Get the value of Id / id.
-    *
-    * @return int
-    */
+     * Get the value of Id / id.
+     *
+     * @return int
+     */
     public function getId()
     {
         $rtn    = $this->data['id'];
@@ -171,10 +171,10 @@ 
     }
 
     /**
-    * Get the value of BuildId / build_id.
-    *
-    * @return int
-    */
+     * Get the value of BuildId / build_id.
+     *
+     * @return int
+     */
     public function getBuildId()
     {
         $rtn    = $this->data['build_id'];
@@ -183,10 +183,10 @@ 
     }
 
     /**
-    * Get the value of Plugin / plugin.
-    *
-    * @return string
-    */
+     * Get the value of Plugin / plugin.
+     *
+     * @return string
+     */
     public function getPlugin()
     {
         $rtn    = $this->data['plugin'];
@@ -195,10 +195,10 @@ 
     }
 
     /**
-    * Get the value of File / file.
-    *
-    * @return string
-    */
+     * Get the value of File / file.
+     *
+     * @return string
+     */
     public function getFile()
     {
         $rtn    = $this->data['file'];
@@ -207,10 +207,10 @@ 
     }
 
     /**
-    * Get the value of LineStart / line_start.
-    *
-    * @return int
-    */
+     * Get the value of LineStart / line_start.
+     *
+     * @return int
+     */
     public function getLineStart()
     {
         $rtn    = $this->data['line_start'];
@@ -219,10 +219,10 @@ 
     }
 
     /**
-    * Get the value of LineEnd / line_end.
-    *
-    * @return int
-    */
+     * Get the value of LineEnd / line_end.
+     *
+     * @return int
+     */
     public function getLineEnd()
     {
         $rtn    = $this->data['line_end'];
@@ -231,10 +231,10 @@ 
     }
 
     /**
-    * Get the value of Severity / severity.
-    *
-    * @return int
-    */
+     * Get the value of Severity / severity.
+     *
+     * @return int
+     */
     public function getSeverity()
     {
         $rtn    = $this->data['severity'];
@@ -243,10 +243,10 @@ 
     }
 
     /**
-    * Get the value of Message / message.
-    *
-    * @return string
-    */
+     * Get the value of Message / message.
+     *
+     * @return string
+     */
     public function getMessage()
     {
         $rtn    = $this->data['message'];
@@ -255,10 +255,10 @@ 
     }
 
     /**
-    * Get the value of CreatedDate / created_date.
-    *
-    * @return \DateTime
-    */
+     * Get the value of CreatedDate / created_date.
+     *
+     * @return \DateTime
+     */
     public function getCreatedDate()
     {
         $rtn    = $this->data['created_date'];
@@ -271,11 +271,11 @@ 
     }
 
     /**
-    * Set the value of Id / id.
-    *
-    * Must not be null.
-    * @param $value int
-    */
+     * Set the value of Id / id.
+     *
+     * Must not be null.
+     * @param $value int
+     */
     public function setId($value)
     {
         $this->_validateNotNull('Id', $value);
@@ -291,11 +291,11 @@ 
     }
 
     /**
-    * Set the value of BuildId / build_id.
-    *
-    * Must not be null.
-    * @param $value int
-    */
+     * Set the value of BuildId / build_id.
+     *
+     * Must not be null.
+     * @param $value int
+     */
     public function setBuildId($value)
     {
         $this->_validateNotNull('BuildId', $value);
@@ -311,11 +311,11 @@ 
     }
 
     /**
-    * Set the value of Plugin / plugin.
-    *
-    * Must not be null.
-    * @param $value string
-    */
+     * Set the value of Plugin / plugin.
+     *
+     * Must not be null.
+     * @param $value string
+     */
     public function setPlugin($value)
     {
         $this->_validateNotNull('Plugin', $value);
@@ -331,10 +331,10 @@ 
     }
 
     /**
-    * Set the value of File / file.
-    *
-    * @param $value string
-    */
+     * Set the value of File / file.
+     *
+     * @param $value string
+     */
     public function setFile($value)
     {
         $this->_validateString('File', $value);
@@ -349,10 +349,10 @@ 
     }
 
     /**
-    * Set the value of LineStart / line_start.
-    *
-    * @param $value int
-    */
+     * Set the value of LineStart / line_start.
+     *
+     * @param $value int
+     */
     public function setLineStart($value)
     {
         $this->_validateInt('LineStart', $value);
@@ -367,10 +367,10 @@ 
     }
 
     /**
-    * Set the value of LineEnd / line_end.
-    *
-    * @param $value int
-    */
+     * Set the value of LineEnd / line_end.
+     *
+     * @param $value int
+     */
     public function setLineEnd($value)
     {
         $this->_validateInt('LineEnd', $value);
@@ -385,11 +385,11 @@ 
     }
 
     /**
-    * Set the value of Severity / severity.
-    *
-    * Must not be null.
-    * @param $value int
-    */
+     * Set the value of Severity / severity.
+     *
+     * Must not be null.
+     * @param $value int
+     */
     public function setSeverity($value)
     {
         $this->_validateNotNull('Severity', $value);
@@ -405,11 +405,11 @@ 
     }
 
     /**
-    * Set the value of Message / message.
-    *
-    * Must not be null.
-    * @param $value string
-    */
+     * Set the value of Message / message.
+     *
+     * Must not be null.
+     * @param $value string
+     */
     public function setMessage($value)
     {
         $this->_validateNotNull('Message', $value);
@@ -425,11 +425,11 @@ 
     }
 
     /**
-    * Set the value of CreatedDate / created_date.
-    *
-    * Must not be null.
-    * @param $value \DateTime
-    */
+     * Set the value of CreatedDate / created_date.
+     *
+     * Must not be null.
+     * @param $value \DateTime
+     */
     public function setCreatedDate($value)
     {
         $this->_validateNotNull('CreatedDate', $value);
@@ -471,10 +471,10 @@ 
     }
 
     /**
-    * Set Build - Accepts an ID, an array representing a Build or a Build model.
-    *
-    * @param $value mixed
-    */
+     * Set Build - Accepts an ID, an array representing a Build or a Build model.
+     *
+     * @param $value mixed
+     */
     public function setBuild($value)
     {
         // Is this an instance of Build?
@@ -492,10 +492,10 @@ 
     }
 
     /**
-    * Set Build - Accepts a Build model.
-    * 
-    * @param $value \PHPCI\Model\Build
-    */
+     * Set Build - Accepts a Build model.
+     * 
+     * @param $value \PHPCI\Model\Build
+     */
     public function setBuildObject(\PHPCI\Model\Build $value)
     {
         return $this->setBuildId($value->getId());

Spacing +11 added lines, -11 removed lines

@@ -165,7 +165,7 @@ 
     */
     public function getId()
     {
-        $rtn    = $this->data['id'];
+        $rtn = $this->data['id'];
 
         return $rtn;
     }
@@ -177,7 +177,7 @@ 
     */
     public function getBuildId()
     {
-        $rtn    = $this->data['build_id'];
+        $rtn = $this->data['build_id'];
 
         return $rtn;
     }
@@ -189,7 +189,7 @@ 
     */
     public function getPlugin()
     {
-        $rtn    = $this->data['plugin'];
+        $rtn = $this->data['plugin'];
 
         return $rtn;
     }
@@ -201,7 +201,7 @@ 
     */
     public function getFile()
     {
-        $rtn    = $this->data['file'];
+        $rtn = $this->data['file'];
 
         return $rtn;
     }
@@ -213,7 +213,7 @@ 
     */
     public function getLineStart()
     {
-        $rtn    = $this->data['line_start'];
+        $rtn = $this->data['line_start'];
 
         return $rtn;
     }
@@ -225,7 +225,7 @@ 
     */
     public function getLineEnd()
     {
-        $rtn    = $this->data['line_end'];
+        $rtn = $this->data['line_end'];
 
         return $rtn;
     }
@@ -237,7 +237,7 @@ 
     */
     public function getSeverity()
     {
-        $rtn    = $this->data['severity'];
+        $rtn = $this->data['severity'];
 
         return $rtn;
     }
@@ -249,7 +249,7 @@ 
     */
     public function getMessage()
     {
-        $rtn    = $this->data['message'];
+        $rtn = $this->data['message'];
 
         return $rtn;
     }
@@ -261,10 +261,10 @@ 
     */
     public function getCreatedDate()
     {
-        $rtn    = $this->data['created_date'];
+        $rtn = $this->data['created_date'];
 
         if (!empty($rtn)) {
-            $rtn    = new \DateTime($rtn);
+            $rtn = new \DateTime($rtn);
         }
         
         return $rtn;
@@ -459,7 +459,7 @@ 
             return null;
         }
 
-        $cacheKey   = 'Cache.Build.' . $key;
+        $cacheKey   = 'Cache.Build.'.$key;
         $rtn        = $this->cache->get($cacheKey, null);
 
         if (empty($rtn)) {

PHPCI/Model/Base/ProjectBase.php

Doc Comments +7 added lines, -7 removed lines

@@ -350,7 +350,7 @@ 
     * Set the value of Id / id.
     *
     * Must not be null.
-    * @param $value int
+    * @param integer $value int
     */
     public function setId($value)
     {
@@ -370,7 +370,7 @@ 
     * Set the value of Title / title.
     *
     * Must not be null.
-    * @param $value string
+    * @param string $value string
     */
     public function setTitle($value)
     {
@@ -390,7 +390,7 @@ 
     * Set the value of Reference / reference.
     *
     * Must not be null.
-    * @param $value string
+    * @param string $value string
     */
     public function setReference($value)
     {
@@ -448,7 +448,7 @@ 
     * Set the value of Type / type.
     *
     * Must not be null.
-    * @param $value string
+    * @param string $value string
     */
     public function setType($value)
     {
@@ -467,7 +467,7 @@ 
     /**
     * Set the value of AccessInformation / access_information.
     *
-    * @param $value string
+    * @param string $value string
     */
     public function setAccessInformation($value)
     {
@@ -540,7 +540,7 @@ 
     * Set the value of AllowPublicStatus / allow_public_status.
     *
     * Must not be null.
-    * @param $value int
+    * @param integer $value int
     */
     public function setAllowPublicStatus($value)
     {
@@ -560,7 +560,7 @@ 
     * Set the value of Archived / archived.
     *
     * Must not be null.
-    * @param $value int
+    * @param boolean $value int
     */
     public function setArchived($value)
     {

Indentation +138 added lines, -138 removed lines

@@ -15,23 +15,23 @@ 
 class ProjectBase extends Model
 {
     /**
-    * @var array
-    */
+     * @var array
+     */
     public static $sleepable = array();
 
     /**
-    * @var string
-    */
+     * @var string
+     */
     protected $tableName = 'project';
 
     /**
-    * @var string
-    */
+     * @var string
+     */
     protected $modelName = 'Project';
 
     /**
-    * @var array
-    */
+     * @var array
+     */
     protected $data = array(
         'id' => null,
         'title' => null,
@@ -49,8 +49,8 @@ 
     );
 
     /**
-    * @var array
-    */
+     * @var array
+     */
     protected $getters = array(
         // Direct property getters:
         'id' => 'getId',
@@ -72,8 +72,8 @@ 
     );
 
     /**
-    * @var array
-    */
+     * @var array
+     */
     protected $setters = array(
         // Direct property setters:
         'id' => 'setId',
@@ -95,8 +95,8 @@ 
     );
 
     /**
-    * @var array
-    */
+     * @var array
+     */
     public $columns = array(
         'id' => array(
             'type' => 'int',
@@ -169,8 +169,8 @@ 
     );
 
     /**
-    * @var array
-    */
+     * @var array
+     */
     public $indexes = array(
             'PRIMARY' => array('unique' => true, 'columns' => 'id'),
             'idx_project_title' => array('columns' => 'title'),
@@ -178,8 +178,8 @@ 
     );
 
     /**
-    * @var array
-    */
+     * @var array
+     */
     public $foreignKeys = array(
             'project_ibfk_1' => array(
                 'local_col' => 'group_id',
@@ -191,10 +191,10 @@ 
     );
 
     /**
-    * Get the value of Id / id.
-    *
-    * @return int
-    */
+     * Get the value of Id / id.
+     *
+     * @return int
+     */
     public function getId()
     {
         $rtn    = $this->data['id'];
@@ -203,10 +203,10 @@ 
     }
 
     /**
-    * Get the value of Title / title.
-    *
-    * @return string
-    */
+     * Get the value of Title / title.
+     *
+     * @return string
+     */
     public function getTitle()
     {
         $rtn    = $this->data['title'];
@@ -215,10 +215,10 @@ 
     }
 
     /**
-    * Get the value of Reference / reference.
-    *
-    * @return string
-    */
+     * Get the value of Reference / reference.
+     *
+     * @return string
+     */
     public function getReference()
     {
         $rtn    = $this->data['reference'];
@@ -227,10 +227,10 @@ 
     }
 
     /**
-    * Get the value of Branch / branch.
-    *
-    * @return string
-    */
+     * Get the value of Branch / branch.
+     *
+     * @return string
+     */
     public function getBranch()
     {
         $rtn    = $this->data['branch'];
@@ -239,10 +239,10 @@ 
     }
 
     /**
-    * Get the value of SshPrivateKey / ssh_private_key.
-    *
-    * @return string
-    */
+     * Get the value of SshPrivateKey / ssh_private_key.
+     *
+     * @return string
+     */
     public function getSshPrivateKey()
     {
         $rtn    = $this->data['ssh_private_key'];
@@ -251,10 +251,10 @@ 
     }
 
     /**
-    * Get the value of Type / type.
-    *
-    * @return string
-    */
+     * Get the value of Type / type.
+     *
+     * @return string
+     */
     public function getType()
     {
         $rtn    = $this->data['type'];
@@ -263,10 +263,10 @@ 
     }
 
     /**
-    * Get the value of AccessInformation / access_information.
-    *
-    * @return string
-    */
+     * Get the value of AccessInformation / access_information.
+     *
+     * @return string
+     */
     public function getAccessInformation()
     {
         $rtn    = $this->data['access_information'];
@@ -275,10 +275,10 @@ 
     }
 
     /**
-    * Get the value of LastCommit / last_commit.
-    *
-    * @return string
-    */
+     * Get the value of LastCommit / last_commit.
+     *
+     * @return string
+     */
     public function getLastCommit()
     {
         $rtn    = $this->data['last_commit'];
@@ -287,10 +287,10 @@ 
     }
 
     /**
-    * Get the value of BuildConfig / build_config.
-    *
-    * @return string
-    */
+     * Get the value of BuildConfig / build_config.
+     *
+     * @return string
+     */
     public function getBuildConfig()
     {
         $rtn    = $this->data['build_config'];
@@ -299,10 +299,10 @@ 
     }
 
     /**
-    * Get the value of SshPublicKey / ssh_public_key.
-    *
-    * @return string
-    */
+     * Get the value of SshPublicKey / ssh_public_key.
+     *
+     * @return string
+     */
     public function getSshPublicKey()
     {
         $rtn    = $this->data['ssh_public_key'];
@@ -311,10 +311,10 @@ 
     }
 
     /**
-    * Get the value of AllowPublicStatus / allow_public_status.
-    *
-    * @return int
-    */
+     * Get the value of AllowPublicStatus / allow_public_status.
+     *
+     * @return int
+     */
     public function getAllowPublicStatus()
     {
         $rtn    = $this->data['allow_public_status'];
@@ -323,10 +323,10 @@ 
     }
 
     /**
-    * Get the value of Archived / archived.
-    *
-    * @return int
-    */
+     * Get the value of Archived / archived.
+     *
+     * @return int
+     */
     public function getArchived()
     {
         $rtn    = $this->data['archived'];
@@ -335,10 +335,10 @@ 
     }
 
     /**
-    * Get the value of GroupId / group_id.
-    *
-    * @return int
-    */
+     * Get the value of GroupId / group_id.
+     *
+     * @return int
+     */
     public function getGroupId()
     {
         $rtn    = $this->data['group_id'];
@@ -347,11 +347,11 @@ 
     }
 
     /**
-    * Set the value of Id / id.
-    *
-    * Must not be null.
-    * @param $value int
-    */
+     * Set the value of Id / id.
+     *
+     * Must not be null.
+     * @param $value int
+     */
     public function setId($value)
     {
         $this->_validateNotNull('Id', $value);
@@ -367,11 +367,11 @@ 
     }
 
     /**
-    * Set the value of Title / title.
-    *
-    * Must not be null.
-    * @param $value string
-    */
+     * Set the value of Title / title.
+     *
+     * Must not be null.
+     * @param $value string
+     */
     public function setTitle($value)
     {
         $this->_validateNotNull('Title', $value);
@@ -387,11 +387,11 @@ 
     }
 
     /**
-    * Set the value of Reference / reference.
-    *
-    * Must not be null.
-    * @param $value string
-    */
+     * Set the value of Reference / reference.
+     *
+     * Must not be null.
+     * @param $value string
+     */
     public function setReference($value)
     {
         $this->_validateNotNull('Reference', $value);
@@ -407,11 +407,11 @@ 
     }
 
     /**
-    * Set the value of Branch / branch.
-    *
-    * Must not be null.
-    * @param $value string
-    */
+     * Set the value of Branch / branch.
+     *
+     * Must not be null.
+     * @param $value string
+     */
     public function setBranch($value)
     {
         $this->_validateNotNull('Branch', $value);
@@ -427,10 +427,10 @@ 
     }
 
     /**
-    * Set the value of SshPrivateKey / ssh_private_key.
-    *
-    * @param $value string
-    */
+     * Set the value of SshPrivateKey / ssh_private_key.
+     *
+     * @param $value string
+     */
     public function setSshPrivateKey($value)
     {
         $this->_validateString('SshPrivateKey', $value);
@@ -445,11 +445,11 @@ 
     }
 
     /**
-    * Set the value of Type / type.
-    *
-    * Must not be null.
-    * @param $value string
-    */
+     * Set the value of Type / type.
+     *
+     * Must not be null.
+     * @param $value string
+     */
     public function setType($value)
     {
         $this->_validateNotNull('Type', $value);
@@ -465,10 +465,10 @@ 
     }
 
     /**
-    * Set the value of AccessInformation / access_information.
-    *
-    * @param $value string
-    */
+     * Set the value of AccessInformation / access_information.
+     *
+     * @param $value string
+     */
     public function setAccessInformation($value)
     {
         $this->_validateString('AccessInformation', $value);
@@ -483,10 +483,10 @@ 
     }
 
     /**
-    * Set the value of LastCommit / last_commit.
-    *
-    * @param $value string
-    */
+     * Set the value of LastCommit / last_commit.
+     *
+     * @param $value string
+     */
     public function setLastCommit($value)
     {
         $this->_validateString('LastCommit', $value);
@@ -501,10 +501,10 @@ 
     }
 
     /**
-    * Set the value of BuildConfig / build_config.
-    *
-    * @param $value string
-    */
+     * Set the value of BuildConfig / build_config.
+     *
+     * @param $value string
+     */
     public function setBuildConfig($value)
     {
         $this->_validateString('BuildConfig', $value);
@@ -519,10 +519,10 @@ 
     }
 
     /**
-    * Set the value of SshPublicKey / ssh_public_key.
-    *
-    * @param $value string
-    */
+     * Set the value of SshPublicKey / ssh_public_key.
+     *
+     * @param $value string
+     */
     public function setSshPublicKey($value)
     {
         $this->_validateString('SshPublicKey', $value);
@@ -537,11 +537,11 @@ 
     }
 
     /**
-    * Set the value of AllowPublicStatus / allow_public_status.
-    *
-    * Must not be null.
-    * @param $value int
-    */
+     * Set the value of AllowPublicStatus / allow_public_status.
+     *
+     * Must not be null.
+     * @param $value int
+     */
     public function setAllowPublicStatus($value)
     {
         $this->_validateNotNull('AllowPublicStatus', $value);
@@ -557,11 +557,11 @@ 
     }
 
     /**
-    * Set the value of Archived / archived.
-    *
-    * Must not be null.
-    * @param $value int
-    */
+     * Set the value of Archived / archived.
+     *
+     * Must not be null.
+     * @param $value int
+     */
     public function setArchived($value)
     {
         $this->_validateNotNull('Archived', $value);
@@ -577,11 +577,11 @@ 
     }
 
     /**
-    * Set the value of GroupId / group_id.
-    *
-    * Must not be null.
-    * @param $value int
-    */
+     * Set the value of GroupId / group_id.
+     *
+     * Must not be null.
+     * @param $value int
+     */
     public function setGroupId($value)
     {
         $this->_validateNotNull('GroupId', $value);
@@ -623,10 +623,10 @@ 
     }
 
     /**
-    * Set Group - Accepts an ID, an array representing a ProjectGroup or a ProjectGroup model.
-    *
-    * @param $value mixed
-    */
+     * Set Group - Accepts an ID, an array representing a ProjectGroup or a ProjectGroup model.
+     *
+     * @param $value mixed
+     */
     public function setGroup($value)
     {
         // Is this an instance of ProjectGroup?
@@ -644,10 +644,10 @@ 
     }
 
     /**
-    * Set Group - Accepts a ProjectGroup model.
-    * 
-    * @param $value \PHPCI\Model\ProjectGroup
-    */
+     * Set Group - Accepts a ProjectGroup model.
+     * 
+     * @param $value \PHPCI\Model\ProjectGroup
+     */
     public function setGroupObject(\PHPCI\Model\ProjectGroup $value)
     {
         return $this->setGroupId($value->getId());

Spacing +14 added lines, -14 removed lines

@@ -197,7 +197,7 @@ 
     */
     public function getId()
     {
-        $rtn    = $this->data['id'];
+        $rtn = $this->data['id'];
 
         return $rtn;
     }
@@ -209,7 +209,7 @@ 
     */
     public function getTitle()
     {
-        $rtn    = $this->data['title'];
+        $rtn = $this->data['title'];
 
         return $rtn;
     }
@@ -221,7 +221,7 @@ 
     */
     public function getReference()
     {
-        $rtn    = $this->data['reference'];
+        $rtn = $this->data['reference'];
 
         return $rtn;
     }
@@ -233,7 +233,7 @@ 
     */
     public function getBranch()
     {
-        $rtn    = $this->data['branch'];
+        $rtn = $this->data['branch'];
 
         return $rtn;
     }
@@ -245,7 +245,7 @@ 
     */
     public function getSshPrivateKey()
     {
-        $rtn    = $this->data['ssh_private_key'];
+        $rtn = $this->data['ssh_private_key'];
 
         return $rtn;
     }
@@ -257,7 +257,7 @@ 
     */
     public function getType()
     {
-        $rtn    = $this->data['type'];
+        $rtn = $this->data['type'];
 
         return $rtn;
     }
@@ -269,7 +269,7 @@ 
     */
     public function getAccessInformation()
     {
-        $rtn    = $this->data['access_information'];
+        $rtn = $this->data['access_information'];
 
         return $rtn;
     }
@@ -281,7 +281,7 @@ 
     */
     public function getLastCommit()
     {
-        $rtn    = $this->data['last_commit'];
+        $rtn = $this->data['last_commit'];
 
         return $rtn;
     }
@@ -293,7 +293,7 @@ 
     */
     public function getBuildConfig()
     {
-        $rtn    = $this->data['build_config'];
+        $rtn = $this->data['build_config'];
 
         return $rtn;
     }
@@ -305,7 +305,7 @@ 
     */
     public function getSshPublicKey()
     {
-        $rtn    = $this->data['ssh_public_key'];
+        $rtn = $this->data['ssh_public_key'];
 
         return $rtn;
     }
@@ -317,7 +317,7 @@ 
     */
     public function getAllowPublicStatus()
     {
-        $rtn    = $this->data['allow_public_status'];
+        $rtn = $this->data['allow_public_status'];
 
         return $rtn;
     }
@@ -329,7 +329,7 @@ 
     */
     public function getArchived()
     {
-        $rtn    = $this->data['archived'];
+        $rtn = $this->data['archived'];
 
         return $rtn;
     }
@@ -341,7 +341,7 @@ 
     */
     public function getGroupId()
     {
-        $rtn    = $this->data['group_id'];
+        $rtn = $this->data['group_id'];
 
         return $rtn;
     }
@@ -611,7 +611,7 @@ 
             return null;
         }
 
-        $cacheKey   = 'Cache.ProjectGroup.' . $key;
+        $cacheKey   = 'Cache.ProjectGroup.'.$key;
         $rtn        = $this->cache->get($cacheKey, null);
 
         if (empty($rtn)) {

PHPCI/Model/Build.php

Doc Comments +2 added lines, -1 removed lines

@@ -74,6 +74,7 @@ 
 
     /**
      * Store build metadata
+     * @param string $key
      */
     public function storeMeta($key, $value)
     {
@@ -181,7 +182,7 @@ 
 
     /**
      * Return a value from the build's "extra" JSON array.
-     * @param null $key
+     * @param string $key
      * @return mixed|null|string
      */
     public function getExtra($key = null)

Indentation +12 added lines, -12 removed lines

@@ -1,11 +1,11 @@ 
 <?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\Model;
 
@@ -31,16 +31,16 @@ 
     public $currentBuildPath;
 
     /**
-    * Get link to commit from another source (i.e. Github)
-    */
+     * Get link to commit from another source (i.e. Github)
+     */
     public function getCommitLink()
     {
         return '#';
     }
 
     /**
-    * Get link to branch from another source (i.e. Github)
-    */
+     * Get link to branch from another source (i.e. Github)
+     */
     public function getBranchLink()
     {
         return '#';
@@ -56,8 +56,8 @@ 
     }
 
     /**
-    * Send status updates to any relevant third parties (i.e. Github)
-    */
+     * Send status updates to any relevant third parties (i.e. Github)
+     */
     public function sendStatusPostback()
     {
         return;

Spacing +7 added lines, -7 removed lines

@@ -105,13 +105,13 @@ 
         }
 
         // Try .phpci.yml
-        if (is_file($buildPath . '/.phpci.yml')) {
-            $build_config = file_get_contents($buildPath . '/.phpci.yml');
+        if (is_file($buildPath.'/.phpci.yml')) {
+            $build_config = file_get_contents($buildPath.'/.phpci.yml');
         }
 
         // Try phpci.yml first:
-        if (empty($build_config) && is_file($buildPath . '/phpci.yml')) {
-            $build_config = file_get_contents($buildPath . '/phpci.yml');
+        if (empty($build_config) && is_file($buildPath.'/phpci.yml')) {
+            $build_config = file_get_contents($buildPath.'/phpci.yml');
         }
 
         // Fall back to zero config plugins:
@@ -135,7 +135,7 @@ 
      */
     protected function getZeroConfigPlugins(Builder $builder)
     {
-        $pluginDir = PHPCI_DIR . 'PHPCI/Plugin/';
+        $pluginDir = PHPCI_DIR.'PHPCI/Plugin/';
         $dir = new \DirectoryIterator($pluginDir);
 
         $config = array(
@@ -257,8 +257,8 @@ 
         }
 
         if (empty($this->currentBuildPath)) {
-            $buildDirectory = $this->getId() . '_' . substr(md5(microtime(true)), 0, 5);
-            $this->currentBuildPath = PHPCI_BUILD_ROOT_DIR . $buildDirectory . DIRECTORY_SEPARATOR;
+            $buildDirectory = $this->getId().'_'.substr(md5(microtime(true)), 0, 5);
+            $this->currentBuildPath = PHPCI_BUILD_ROOT_DIR.$buildDirectory.DIRECTORY_SEPARATOR;
         }
 
         return $this->currentBuildPath;

PHPCI/Model/Build/RemoteGitBuild.php

Doc Comments +1 added lines, -1 removed lines

@@ -160,7 +160,7 @@
     /**
      * Create an SSH wrapper script for Git to use, to disable host key checking, etc.
      * @param $cloneTo
-     * @param $keyFile
+     * @param string $keyFile
      * @return string
      */
     protected function writeSshWrapper($cloneTo, $keyFile)

Indentation +14 added lines, -14 removed lines

@@ -1,11 +1,11 @@ 
 <?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\Model\Build;
 
@@ -21,16 +21,16 @@ 
 class RemoteGitBuild extends Build
 {
     /**
-    * Get the URL to be used to clone this remote repository.
-    */
+     * Get the URL to be used to clone this remote repository.
+     */
     protected function getCloneUrl()
     {
         return $this->getProject()->getReference();
     }
 
     /**
-    * Create a working copy by cloning, copying, or similar.
-    */
+     * Create a working copy by cloning, copying, or similar.
+     */
     public function createWorkingCopy(Builder $builder, $buildPath)
     {
         $key = trim($this->getProject()->getSshPrivateKey());
@@ -50,8 +50,8 @@ 
     }
 
     /**
-    * Use an HTTP-based git clone.
-    */
+     * Use an HTTP-based git clone.
+     */
     protected function cloneByHttp(Builder $builder, $cloneTo)
     {
         $cmd = 'git clone --recursive ';
@@ -73,8 +73,8 @@ 
     }
 
     /**
-    * Use an SSH-based git clone.
-    */
+     * Use an SSH-based git clone.
+     */
     protected function cloneBySsh(Builder $builder, $cloneTo)
     {
         $keyFile = $this->writeSshKey($cloneTo);

Spacing +9 added lines, -9 removed lines

@@ -59,7 +59,7 @@ 
         $depth = $builder->getConfig('clone_depth');
 
         if (!is_null($depth)) {
-            $cmd .= ' --depth ' . intval($depth) . ' ';
+            $cmd .= ' --depth '.intval($depth).' ';
         }
 
         $cmd .= ' -b %s %s "%s"';
@@ -89,13 +89,13 @@ 
         $depth = $builder->getConfig('clone_depth');
 
         if (!is_null($depth)) {
-            $cmd .= ' --depth ' . intval($depth) . ' ';
+            $cmd .= ' --depth '.intval($depth).' ';
         }
 
         $cmd .= ' -b %s %s "%s"';
 
         if (!IS_WIN) {
-            $cmd = 'export GIT_SSH="'.$gitSshWrapper.'" && ' . $cmd;
+            $cmd = 'export GIT_SSH="'.$gitSshWrapper.'" && '.$cmd;
         }
 
         $success = $builder->executeCommand($cmd, $this->getBranch(), $this->getCloneUrl(), $cloneTo);
@@ -127,12 +127,12 @@ 
         $chdir = IS_WIN ? 'cd /d "%s"' : 'cd "%s"';
 
         if (!empty($commit) && $commit != 'Manual') {
-            $cmd = $chdir . ' && git checkout %s --quiet';
+            $cmd = $chdir.' && git checkout %s --quiet';
             $success = $builder->executeCommand($cmd, $cloneTo, $commit);
         }
 
         // Always update the commit hash with the actual HEAD hash
-        if ($builder->executeCommand($chdir . ' && git rev-parse HEAD', $cloneTo)) {
+        if ($builder->executeCommand($chdir.' && git rev-parse HEAD', $cloneTo)) {
             $this->setCommitId(trim($builder->getLastOutput()));
         }
 
@@ -146,8 +146,8 @@ 
      */
     protected function writeSshKey($cloneTo)
     {
-        $keyPath = dirname($cloneTo . '/temp');
-        $keyFile = $keyPath . '.key';
+        $keyPath = dirname($cloneTo.'/temp');
+        $keyFile = $keyPath.'.key';
 
         // Write the contents of this project's git key to the file:
         file_put_contents($keyFile, $this->getProject()->getSshPrivateKey());
@@ -165,8 +165,8 @@ 
      */
     protected function writeSshWrapper($cloneTo, $keyFile)
     {
-        $path = dirname($cloneTo . '/temp');
-        $wrapperFile = $path . '.sh';
+        $path = dirname($cloneTo.'/temp');
+        $wrapperFile = $path.'.sh';
 
         $sshFlags = '-o CheckHostIP=no -o IdentitiesOnly=yes -o StrictHostKeyChecking=no -o PasswordAuthentication=no';
 

PHPCI/Model/Build/SubversionBuild.php

Doc Comments +1 added lines, -1 removed lines

@@ -157,7 +157,7 @@
     /**
      * Create an SSH wrapper script for Svn to use, to disable host key checking, etc.
      * @param $cloneTo
-     * @param $keyFile
+     * @param string $keyFile
      * @return string
      */
     protected function writeSshWrapper($cloneTo, $keyFile)

Indentation +6 added lines, -6 removed lines

@@ -1,11 +1,11 @@
 <?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\Model\Build;
 

Spacing +9 added lines, -9 removed lines

@@ -38,7 +38,7 @@ 
         if (empty($branch) || $branch == 'trunk') {
             $url .= 'trunk';
         } else {
-            $url .= 'branches/' . $branch;
+            $url .= 'branches/'.$branch;
         }
 
         return $url;
@@ -63,7 +63,7 @@ 
         $depth = $builder->getConfig('clone_depth');
 
         if (!is_null($depth)) {
-            $cmd .= ' --depth ' . intval($depth) . ' ';
+            $cmd .= ' --depth '.intval($depth).' ';
         }
 
         $this->svnCommand = $cmd;
@@ -117,12 +117,12 @@ 
      */
     protected function cloneBySsh(Builder $builder, $cloneTo)
     {
-        $cmd = $this->svnCommand . ' %s "%s"';
+        $cmd = $this->svnCommand.' %s "%s"';
 
         if (!IS_WIN) {
             $keyFile    = $this->writeSshKey($cloneTo);
             $sshWrapper = $this->writeSshWrapper($cloneTo, $keyFile);
-            $cmd        = 'export SVN_SSH="' . $sshWrapper . '" && ' . $cmd;
+            $cmd        = 'export SVN_SSH="'.$sshWrapper.'" && '.$cmd;
         }
 
         $success = $builder->executeCommand($cmd, $this->getCloneUrl(), $cloneTo);
@@ -143,8 +143,8 @@ 
      */
     protected function writeSshKey($cloneTo)
     {
-        $keyPath = dirname($cloneTo . '/temp');
-        $keyFile = $keyPath . '.key';
+        $keyPath = dirname($cloneTo.'/temp');
+        $keyFile = $keyPath.'.key';
 
         // Write the contents of this project's svn key to the file:
         file_put_contents($keyFile, $this->getProject()->getSshPrivateKey());
@@ -162,8 +162,8 @@ 
      */
     protected function writeSshWrapper($cloneTo, $keyFile)
     {
-        $path        = dirname($cloneTo . '/temp');
-        $wrapperFile = $path . '.sh';
+        $path        = dirname($cloneTo.'/temp');
+        $wrapperFile = $path.'.sh';
 
         $sshFlags = '-o CheckHostIP=no -o IdentitiesOnly=yes -o StrictHostKeyChecking=no -o PasswordAuthentication=no';
 
@@ -175,7 +175,7 @@ 
 OUT;
 
         file_put_contents($wrapperFile, $script);
-        shell_exec('chmod +x "' . $wrapperFile . '"');
+        shell_exec('chmod +x "'.$wrapperFile.'"');
 
         return $wrapperFile;
     }

PHPCI/Plugin/Campfire.php

Doc Comments +1 added lines, -1 removed lines

@@ -114,7 +114,7 @@
 
     /**
      * Make a request to Campfire.
-     * @param $page
+     * @param string $page
      * @param null $data
      * @return bool|mixed
      */

Indentation +6 added lines, -6 removed lines

@@ -1,11 +1,11 @@
 <?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

@@ -63,7 +63,7 @@ 
      */
     public function execute()
     {
-        $url = PHPCI_URL . "build/view/" . $this->build->getId();
+        $url = PHPCI_URL."build/view/".$this->build->getId();
         $message = str_replace("%buildurl%", $url, $this->message);
         $this->joinRoom($this->roomId);
         $status = $this->speak($message, $this->roomId);
@@ -120,7 +120,7 @@ 
      */
     private function getPageByPost($page, $data = null)
     {
-        $url = $this->url . $page;
+        $url = $this->url.$page;
         // The new API allows JSON, so we can pass
         // PHP data structures instead of old school POST
         $json = json_encode($data);
@@ -133,7 +133,7 @@ 
         curl_setopt($handle, CURLOPT_USERAGENT, $this->userAgent);
         curl_setopt($handle, CURLOPT_VERBOSE, $this->verbose);
         curl_setopt($handle, CURLOPT_FOLLOWLOCATION, 1);
-        curl_setopt($handle, CURLOPT_USERPWD, $this->authToken . ':x');
+        curl_setopt($handle, CURLOPT_USERPWD, $this->authToken.':x');
         curl_setopt($handle, CURLOPT_HTTPHEADER, array("Content-type: application/json"));
         curl_setopt($handle, CURLOPT_COOKIEFILE, $this->cookie);
 

PHPCI/Plugin/Codeception.php

Doc Comments +3 added lines, -3 removed lines

@@ -57,7 +57,7 @@ 
 
     /**
      * Try and find the codeception YML config file.
-     * @param $buildPath
+     * @param string $buildPath
      * @return null|string
      */
     public static function findConfigFile($buildPath)
@@ -113,8 +113,8 @@ 
 
     /**
      * Run tests from a Codeception config file.
-     * @param $configPath
-     * @return bool|mixed
+     * @param string $configPath
+     * @return boolean
      * @throws \Exception
      */
     protected function runConfigFile($configPath)

Unused Use Statements -1 removed lines

@@ -13,7 +13,6 @@
 use PHPCI\Helper\Lang;
 use PHPCI\Model\Build;
 use PHPCI\Plugin\Util\TestResultParsers\Codeception as Parser;
-use Psr\Log\LogLevel;
 
 /**
  * Codeception Plugin - Enables full acceptance, unit, and functional testing.

Indentation +6 added lines, -6 removed lines

@@ -1,11 +1,11 @@
 <?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 +9 added lines, -9 removed lines

@@ -62,11 +62,11 @@ 
      */
     public static function findConfigFile($buildPath)
     {
-        if (file_exists($buildPath . 'codeception.yml')) {
+        if (file_exists($buildPath.'codeception.yml')) {
             return 'codeception.yml';
         }
 
-        if (file_exists($buildPath . 'codeception.dist.yml')) {
+        if (file_exists($buildPath.'codeception.dist.yml')) {
             return 'codeception.dist.yml';
         }
 
@@ -83,7 +83,7 @@ 
     {
         $this->phpci = $phpci;
         $this->build = $build;
-        $this->path = 'tests' . DIRECTORY_SEPARATOR . '_output' . DIRECTORY_SEPARATOR;
+        $this->path = 'tests'.DIRECTORY_SEPARATOR.'_output'.DIRECTORY_SEPARATOR;
 
         if (empty($options['config'])) {
             $this->ymlConfigFile = self::findConfigFile($this->phpci->buildPath);
@@ -91,7 +91,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'];
@@ -129,21 +129,21 @@ 
             return false;
         }
 
-        $cmd = 'cd "%s" && ' . $codecept . ' run -c "%s" --xml ' . $this->args;
+        $cmd = 'cd "%s" && '.$codecept.' run -c "%s" --xml '.$this->args;
 
         if (IS_WIN) {
-            $cmd = 'cd /d "%s" && ' . $codecept . ' run -c "%s" --xml ' . $this->args;
+            $cmd = 'cd /d "%s" && '.$codecept.' run -c "%s" --xml '.$this->args;
         }
 
-        $configPath = $this->phpci->buildPath . $configPath;
+        $configPath = $this->phpci->buildPath.$configPath;
         $success = $this->phpci->executeCommand($cmd, $this->phpci->buildPath, $configPath);
 
         $this->phpci->log(
-            'Codeception XML path: '. $this->phpci->buildPath . $this->path . 'report.xml',
+            'Codeception XML path: '.$this->phpci->buildPath.$this->path.'report.xml',
             Loglevel::DEBUG
         );
 
-        $xml = file_get_contents($this->phpci->buildPath . $this->path . 'report.xml', false);
+        $xml = file_get_contents($this->phpci->buildPath.$this->path.'report.xml', false);
         $parser = new Parser($this->phpci, $xml);
         $output = $parser->parse();
 

PHPCI/Plugin/Composer.php

Unused Use Statements -1 removed lines

@@ -12,7 +12,6 @@
 use PHPCI;
 use PHPCI\Builder;
 use PHPCI\Model\Build;
-use PHPCI\Helper\Lang;
 
 /**
 * Composer Plugin - Provides access to Composer functionality.

Indentation +8 added lines, -8 removed lines

@@ -1,11 +1,11 @@ 
 <?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;
 
@@ -81,8 +81,8 @@ 
     }
 
     /**
-    * Executes Composer and runs a specified command (e.g. install / update)
-    */
+     * Executes Composer and runs a specified command (e.g. install / update)
+     */
     public function execute()
     {
         $composerLocation = $this->phpci->findBinary(array('composer', 'composer.phar'));

Spacing +3 added lines, -3 removed lines

@@ -38,7 +38,7 @@ 
      */
     public static function canExecute($stage, Builder $builder, Build $build)
     {
-        $path = $builder->buildPath . DIRECTORY_SEPARATOR . 'composer.json';
+        $path = $builder->buildPath.DIRECTORY_SEPARATOR.'composer.json';
 
         if (file_exists($path) && $stage == 'setup') {
             return true;
@@ -64,7 +64,7 @@ 
         $this->nodev      = false;
 
         if (array_key_exists('directory', $options)) {
-            $this->directory = $path . DIRECTORY_SEPARATOR . $options['directory'];
+            $this->directory = $path.DIRECTORY_SEPARATOR.$options['directory'];
         }
 
         if (array_key_exists('action', $options)) {
@@ -93,7 +93,7 @@ 
             $cmd = 'php ';
         }
 
-        $cmd .= $composerLocation . ' --no-ansi --no-interaction ';
+        $cmd .= $composerLocation.' --no-ansi --no-interaction ';
 
         if ($this->preferDist) {
             $this->phpci->log('Using --prefer-dist flag');