corretgecom / PHPCI

PHPCI/Plugin/Util/FilesPluginInformation.php

Doc Comments +1 added lines, -1 removed lines

@@ -28,7 +28,7 @@
     protected $pluginInfo = null;
 
     /**
-     * @param $dirPath
+     * @param string $dirPath
      * @return FilesPluginInformation
      */
     public static function newFromDir($dirPath)

Spacing +2 added lines, -2 removed lines

@@ -69,7 +69,7 @@ 
     public function getPluginClasses()
     {
         return array_map(
-            function (\stdClass $plugin) {
+            function(\stdClass $plugin) {
                 return $plugin->class;
             },
             $this->getInstalledPlugins()
@@ -129,7 +129,7 @@ 
             preg_match('#namespace +([A-Za-z\\\\]+);#i', $contents, $matches);
             $namespace = $matches[1];
 
-            return $namespace . '\\' . $className;
+            return $namespace.'\\'.$className;
         } else {
             return null;
         }

PHPCI/Service/BuildStatusService.php

Doc Comments +1 added lines, -1 removed lines

@@ -63,7 +63,7 @@
     }
 
     /**
-     * @param $url
+     * @param string $url
      */
     public function setUrl($url)
     {

Indentation +7 added lines, -7 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\Service;
 
@@ -199,7 +199,7 @@ 
      */
     public function getBuildUrl()
     {
-         return $this->url . 'build/view/' . $this->build->getId();
+            return $this->url . 'build/view/' . $this->build->getId();
     }
 
     /**

Spacing +2 added lines, -2 removed lines

@@ -118,7 +118,7 @@ 
      */
     public function getName()
     {
-        return $this->project->getTitle() . ' / ' . $this->branch;
+        return $this->project->getTitle().' / '.$this->branch;
     }
 
     /**
@@ -199,7 +199,7 @@ 
      */
     public function getBuildUrl()
     {
-         return $this->url . 'build/view/' . $this->build->getId();
+         return $this->url.'build/view/'.$this->build->getId();
     }
 
     /**

Tests/PHPCI/Command/InstallCommandTest.php

Doc Comments +4 added lines

@@ -117,6 +117,10 @@
         return $config;
     }
 
+    /**
+     * @param string $param
+     * @param \PHPUnit_Framework_MockObject_MockBuilder $dialog
+     */
     protected function executeWithoutParam($param = null, $dialog)
     {
         // Clean result variables.

Indentation +9 added lines, -9 removed lines

@@ -35,14 +35,14 @@ 
     {
         // We check that there's no interaction with user.
         $dialog = $this->getMockBuilder('Symfony\\Component\\Console\\Helper\\DialogHelper')
-                       ->setMethods(array(
-                           'ask',
-                           'askConfirmation',
-                           'askAndValidate',
-                           'askHiddenResponse',
-                           'askHiddenResponseAndValidate',
-                       ))
-                       ->getMock();
+                        ->setMethods(array(
+                            'ask',
+                            'askConfirmation',
+                            'askAndValidate',
+                            'askHiddenResponse',
+                            'askHiddenResponseAndValidate',
+                        ))
+                        ->getMock();
 
         return $dialog;
     }
@@ -111,7 +111,7 @@ 
         );
 
         if (!is_null($exclude)) {
-          unset($config[$exclude]);
+            unset($config[$exclude]);
         }
 
         return $config;

Spacing +2 added lines, -2 removed lines

@@ -72,12 +72,12 @@
         $command->expects($this->once())->method('verifyDatabaseDetails')->willReturn(true);
         $command->expects($this->once())->method('setupDatabase')->willReturn(true);
         $command->expects($this->once())->method('createAdminUser')->will(
-            $this->returnCallback(function ($adm) use ($self) {
+            $this->returnCallback(function($adm) use ($self) {
                 $self->admin = $adm;
             })
         );
         $command->expects($this->once())->method('writeConfigFile')->will(
-            $this->returnCallback(function ($cfg) use ($self) {
+            $this->returnCallback(function($cfg) use ($self) {
                 $self->config = $cfg;
             })
         );

Tests/PHPCI/Plugin/Util/FactoryTest.php

Doc Comments +3 added lines

@@ -209,6 +209,9 @@
         );
     }
 
+    /**
+     * @param string $pluginName
+     */
     protected function getFakePluginClassName($pluginName)
     {
         $pluginNamespace = '\\Tests\\PHPCI\\Plugin\\Util\\Fake\\';

Spacing +5 added lines, -5 removed lines

@@ -150,7 +150,7 @@ 
     public function testAddConfigFromFile_ReturnsTrueForValidFile()
     {
         $result = $this->testedFactory->addConfigFromFile(
-            realpath(__DIR__ . "/ExamplePluginConfig.php")
+            realpath(__DIR__."/ExamplePluginConfig.php")
         );
 
         $this->assertTrue($result);
@@ -159,7 +159,7 @@ 
     public function testAddConfigFromFile_RegistersResources()
     {
         $this->testedFactory->addConfigFromFile(
-            realpath(__DIR__ . "/ExamplePluginConfig.php")
+            realpath(__DIR__."/ExamplePluginConfig.php")
         );
 
         $pluginClass = $this->getFakePluginClassName('ExamplePluginWithSingleRequiredArg');
@@ -181,7 +181,7 @@ 
         $self = $this;
 
         $this->testedFactory->registerResource(
-            function () use ($self) {
+            function() use ($self) {
                 return $self->getMock(
                     'PHPCI\Builder',
                     array(),
@@ -195,7 +195,7 @@ 
         );
 
         $this->testedFactory->registerResource(
-            function () use ($self) {
+            function() use ($self) {
                 return $self->getMock(
                     'PHPCI\Model\Build',
                     array(),
@@ -213,7 +213,7 @@ 
     {
         $pluginNamespace = '\\Tests\\PHPCI\\Plugin\\Util\\Fake\\';
 
-        return $pluginNamespace . $pluginName;
+        return $pluginNamespace.$pluginName;
     }
 }
 

Tests/PHPCI/ProcessControl/ProcessControlTest.php

Doc Comments +3 added lines

@@ -116,6 +116,9 @@
 
     abstract public function testIsAvailable();
 
+    /**
+     * @return string
+     */
     abstract public function getTestCommand();
 
     protected function tearDown()

bootstrap.php

Spacing +7 added lines, -7 removed lines

@@ -15,7 +15,7 @@ 
     date_default_timezone_set('UTC');
 }
 
-$configFile = dirname(__FILE__) . '/PHPCI/config.yml';
+$configFile = dirname(__FILE__).'/PHPCI/config.yml';
 $configEnv = getenv('phpci_config_file');
 $usingCustomConfigFile = false;
 
@@ -33,7 +33,7 @@ 
 }
 
 // If composer has not been run, fail at this point and tell the user to install:
-if (!file_exists(dirname(__FILE__) . '/vendor/autoload.php') && defined('PHPCI_IS_CONSOLE') && PHPCI_IS_CONSOLE) {
+if (!file_exists(dirname(__FILE__).'/vendor/autoload.php') && defined('PHPCI_IS_CONSOLE') && PHPCI_IS_CONSOLE) {
     $message = 'Please install PHPCI with "composer install" (or "php composer.phar install"';
     $message .= ' for Windows) before using console';
     
@@ -42,19 +42,19 @@ 
 }
 
 // Load Composer autoloader:
-require_once(dirname(__FILE__) . '/vendor/autoload.php');
+require_once(dirname(__FILE__).'/vendor/autoload.php');
 
 \PHPCI\ErrorHandler::register();
 
 if (defined('PHPCI_IS_CONSOLE') && PHPCI_IS_CONSOLE) {
-    $loggerConfig = LoggerConfig::newFromFile(__DIR__ . "/loggerconfig.php");
+    $loggerConfig = LoggerConfig::newFromFile(__DIR__."/loggerconfig.php");
 }
 
 // Load configuration if present:
 $conf = array();
 $conf['b8']['app']['namespace'] = 'PHPCI';
 $conf['b8']['app']['default_controller'] = 'Home';
-$conf['b8']['view']['path'] = dirname(__FILE__) . '/PHPCI/View/';
+$conf['b8']['view']['path'] = dirname(__FILE__).'/PHPCI/View/';
 $conf['using_custom_file'] = $usingCustomConfigFile;
 
 $config = new b8\Config($conf);
@@ -69,11 +69,11 @@ 
  *
  * @ticket 781
  */
-$localVarsFile = dirname(__FILE__) . '/local_vars.php';
+$localVarsFile = dirname(__FILE__).'/local_vars.php';
 if (is_readable($localVarsFile)) {
     require_once $localVarsFile;
 }
 
-require_once(dirname(__FILE__) . '/vars.php');
+require_once(dirname(__FILE__).'/vars.php');
 
 \PHPCI\Helper\Lang::init($config);

phinx.php

Spacing +1 added lines, -1 removed lines

@@ -8,7 +8,7 @@
  * @link         https://www.phptesting.org/
  */
 
-require_once(dirname(__FILE__) . '/bootstrap.php');
+require_once(dirname(__FILE__).'/bootstrap.php');
 
 $writeServers = $config->get('b8.database.servers.write');
 

public/assets/plugins/datatables/extensions/Scroller/examples/data/ssp.php

Indentation +10 added lines, -10 removed lines

@@ -29,19 +29,19 @@ 
 // parameter represents the DataTables column identifier. In this case simple
 // indexes
 $columns = array(
-	array( 'db' => 'id',         'dt' => 0 ),
-	array( 'db' => 'firstname',  'dt' => 1 ),
-	array( 'db' => 'surname',    'dt' => 2 ),
-	array( 'db' => 'zip',        'dt' => 3 ),
-	array( 'db' => 'country',    'dt' => 4 )
+    array( 'db' => 'id',         'dt' => 0 ),
+    array( 'db' => 'firstname',  'dt' => 1 ),
+    array( 'db' => 'surname',    'dt' => 2 ),
+    array( 'db' => 'zip',        'dt' => 3 ),
+    array( 'db' => 'country',    'dt' => 4 )
 );
 
 // SQL server connection information
 $sql_details = array(
-	'user' => '',
-	'pass' => '',
-	'db'   => '',
-	'host' => ''
+    'user' => '',
+    'pass' => '',
+    'db'   => '',
+    'host' => ''
 );
 
 
@@ -53,6 +53,6 @@ 
 require( '../../../../examples/server_side/scripts/ssp.class.php' );
 
 echo json_encode(
-	SSP::simple( $_GET, $sql_details, $table, $primaryKey, $columns )
+    SSP::simple( $_GET, $sql_details, $table, $primaryKey, $columns )
 );
 

Spacing +7 added lines, -7 removed lines

@@ -29,11 +29,11 @@ 
 // parameter represents the DataTables column identifier. In this case simple
 // indexes
 $columns = array(
-	array( 'db' => 'id',         'dt' => 0 ),
-	array( 'db' => 'firstname',  'dt' => 1 ),
-	array( 'db' => 'surname',    'dt' => 2 ),
-	array( 'db' => 'zip',        'dt' => 3 ),
-	array( 'db' => 'country',    'dt' => 4 )
+	array('db' => 'id', 'dt' => 0),
+	array('db' => 'firstname', 'dt' => 1),
+	array('db' => 'surname', 'dt' => 2),
+	array('db' => 'zip', 'dt' => 3),
+	array('db' => 'country', 'dt' => 4)
 );
 
 // SQL server connection information
@@ -50,9 +50,9 @@ 
  * server-side, there is no need to edit below this line.
  */
 
-require( '../../../../examples/server_side/scripts/ssp.class.php' );
+require('../../../../examples/server_side/scripts/ssp.class.php');
 
 echo json_encode(
-	SSP::simple( $_GET, $sql_details, $table, $primaryKey, $columns )
+	SSP::simple($_GET, $sql_details, $table, $primaryKey, $columns)
 );
 

PHPCI/Helper/UnixCommandExecutor.php

Spacing +1 added lines, -1 removed lines

@@ -22,6 +22,6 @@
      */
     protected function findGlobalBinary($binary)
     {
-        return trim(shell_exec('which ' . $binary));
+        return trim(shell_exec('which '.$binary));
     }
 }