ci-bonfire / Sprint

myth/_generators/Model/forge.php

Indentation +1 added lines, -1 removed lines

@@ -31,7 +31,7 @@
  */
 
 $descriptions = [
-    'model' => ['model <name>', 'Creates a new model file that extends from CIDbModel.']
+	'model' => ['model <name>', 'Creates a new model file that extends from CIDbModel.']
 ];
 
 $long_description = <<<EOT

myth/_generators/Model/model.tpl.php

Spacing +1 added lines, -1 removed lines

@@ -35,7 +35,7 @@
     protected \$deleted_by_field    = 'deleted_by';
 ";
 
-$protect = "'". implode("', '", $protected) . "'";
+$protect = "'".implode("', '", $protected)."'";
 
 //--------------------------------------------------------------------
 // Create the full model

myth/_generators/Scaffold/ScaffoldGenerator.php

Indentation +37 added lines, -37 removed lines

@@ -38,7 +38,7 @@ 
 
 	protected $model_name = '';
 
-    protected $table_exists = false;
+	protected $table_exists = false;
 
 	//--------------------------------------------------------------------
 
@@ -46,32 +46,32 @@ 
 	{
 		$name = array_shift( $segments );
 
-        // If a table already exists then we don't need a migration
-        $this->load->database();
-        if ($this->db->table_exists($name))
-        {
-            $this->table_exists = true;
-        }
+		// If a table already exists then we don't need a migration
+		$this->load->database();
+		if ($this->db->table_exists($name))
+		{
+			$this->table_exists = true;
+		}
 
 		// If we didn't attach any fields, then we
 		// need to generate the barebones here.
 		$this->fields = CLI::option('fields');
 
-        // If we don't have any fields, and no model
-        // exists, then we need to provide some default fields.
-        if (empty($this->fields))
-        {
-            if (! $this->table_exists)
-            {
-                $this->fields = "id:int id:id title:string created_on:datetime modified_on:datetime";
-            }
-        }
+		// If we don't have any fields, and no model
+		// exists, then we need to provide some default fields.
+		if (empty($this->fields))
+		{
+			if (! $this->table_exists)
+			{
+				$this->fields = "id:int id:id title:string created_on:datetime modified_on:datetime";
+			}
+		}
 
 		// Perform the steps.
-        if (! $this->table_exists)
-        {
-            $this->makeMigration( $name );
-        }
+		if (! $this->table_exists)
+		{
+			$this->makeMigration( $name );
+		}
 		$this->makeSeed($name);
 		$this->makeModel($name);
 		$this->makeController($name);
@@ -98,7 +98,7 @@ 
 
 	public function makeSeed($name)
 	{
-	    $this->generate("seed {$name}", null, true);
+		$this->generate("seed {$name}", null, true);
 	}
 
 	//--------------------------------------------------------------------
@@ -106,33 +106,33 @@ 
 
 	public function makeModel($name)
 	{
-	    $this->load->helper('inflector');
+		$this->load->helper('inflector');
 
 		$name = singular($name);
 		$this->model_name = $name;
 
-        // If a table exists, we can build it from that
-        if ($this->table_exists)
-        {
-            $this->generate( "model {$name}", "-table $name -primary_key id", TRUE );
-        }
-        // Otherwise, we need to provide the fields to make the model out of.
-        else
-        {
-            $this->generate( "model {$name}", "-fields '{$this->fields}'", TRUE );
-        }
+		// If a table exists, we can build it from that
+		if ($this->table_exists)
+		{
+			$this->generate( "model {$name}", "-table $name -primary_key id", TRUE );
+		}
+		// Otherwise, we need to provide the fields to make the model out of.
+		else
+		{
+			$this->generate( "model {$name}", "-fields '{$this->fields}'", TRUE );
+		}
 	}
 
 	//--------------------------------------------------------------------
 
 	public function makeController($name)
 	{
-        $options = "-model {$this->model_name} -create_views";
+		$options = "-model {$this->model_name} -create_views";
 
-        if (! empty($this->fields))
-        {
-            $options .= " -fields '{$this->fields}'";
-        }
+		if (! empty($this->fields))
+		{
+			$options .= " -fields '{$this->fields}'";
+		}
 
 		$this->generate("controller {$name}", $options, true);
 	}

Spacing +8 added lines, -8 removed lines

@@ -42,9 +42,9 @@ 
 
 	//--------------------------------------------------------------------
 
-	public function run($segments = [ ], $quiet = false)
+	public function run($segments = [], $quiet = false)
 	{
-		$name = array_shift( $segments );
+		$name = array_shift($segments);
 
         // If a table already exists then we don't need a migration
         $this->load->database();
@@ -61,16 +61,16 @@ 
         // exists, then we need to provide some default fields.
         if (empty($this->fields))
         {
-            if (! $this->table_exists)
+            if ( ! $this->table_exists)
             {
                 $this->fields = "id:int id:id title:string created_on:datetime modified_on:datetime";
             }
         }
 
 		// Perform the steps.
-        if (! $this->table_exists)
+        if ( ! $this->table_exists)
         {
-            $this->makeMigration( $name );
+            $this->makeMigration($name);
         }
 		$this->makeSeed($name);
 		$this->makeModel($name);
@@ -114,12 +114,12 @@ 
         // If a table exists, we can build it from that
         if ($this->table_exists)
         {
-            $this->generate( "model {$name}", "-table $name -primary_key id", TRUE );
+            $this->generate("model {$name}", "-table $name -primary_key id", TRUE);
         }
         // Otherwise, we need to provide the fields to make the model out of.
         else
         {
-            $this->generate( "model {$name}", "-fields '{$this->fields}'", TRUE );
+            $this->generate("model {$name}", "-fields '{$this->fields}'", TRUE);
         }
 	}
 
@@ -129,7 +129,7 @@ 
 	{
         $options = "-model {$this->model_name} -create_views";
 
-        if (! empty($this->fields))
+        if ( ! empty($this->fields))
         {
             $options .= " -fields '{$this->fields}'";
         }

myth/_generators/Seed/SeedGenerator.php

Spacing +10 added lines, -10 removed lines

@@ -34,36 +34,36 @@
 
 class SeedGenerator extends \Myth\Forge\BaseGenerator {
 
-	public function run($segments=[], $quiet=false)
+	public function run($segments = [], $quiet = false)
 	{
-		$name = array_shift( $segments );
+		$name = array_shift($segments);
 
-		if ( empty( $name ) )
+		if (empty($name))
 		{
-			$name = CLI::prompt( 'Seed name' );
+			$name = CLI::prompt('Seed name');
 		}
 
 		// Format to CI Standards
-		$name = str_replace('.php', '', strtolower( $name ) );
-		if (substr( $name, -4) == 'seed')
+		$name = str_replace('.php', '', strtolower($name));
+		if (substr($name, -4) == 'seed')
 		{
 			$name = substr($name, 0, strlen($name) - 4);
 		}
-		$name = ucfirst($name) .'Seeder';
+		$name = ucfirst($name).'Seeder';
 
 		$data = [
 			'seed_name' => $name,
-			'today'     => date( 'Y-m-d H:ia' )
+			'today'     => date('Y-m-d H:ia')
 		];
 
-		$destination = $this->determineOutputPath( 'database/seeds' ) . $name . '.php';
+		$destination = $this->determineOutputPath('database/seeds').$name.'.php';
 
 		if (strpos($destination, 'modules') !== false)
 		{
 			$destination = str_replace('database/', '', $destination);
 		}
 
-		if (! $this->copyTemplate( 'seed', $destination, $data, $this->overwrite) )
+		if ( ! $this->copyTemplate('seed', $destination, $data, $this->overwrite))
 		{
 			CLI::error('Error creating seed file.');
 		}

myth/_generators/View/ViewGenerator.php

Spacing +7 added lines, -7 removed lines

@@ -34,21 +34,21 @@
 
 class ViewGenerator extends \Myth\Forge\BaseGenerator {
 
-	public function run($segments=[], $quiet=false)
+	public function run($segments = [], $quiet = false)
 	{
-		$name = array_shift( $segments );
+		$name = array_shift($segments);
 
-		if ( empty( $name ) )
+		if (empty($name))
 		{
-			$name = CLI::prompt( 'View name' );
+			$name = CLI::prompt('View name');
 		}
 
 		// Format to CI Standards
-		$name = str_replace('.php', '', strtolower( $name ) );
+		$name = str_replace('.php', '', strtolower($name));
 
-		$destination = $this->determineOutputPath( 'views' ) . $name . '.php';
+		$destination = $this->determineOutputPath('views').$name.'.php';
 
-		if (! $this->createFile($destination, "The {$name}.php view file.", $this->overwrite) )
+		if ( ! $this->createFile($destination, "The {$name}.php view file.", $this->overwrite))
 		{
 			CLI::error('Error creating view file.');
 		}

system/core/CodeIgniter.php

Spacing +10 added lines, -10 removed lines

@@ -185,7 +185,7 @@ 
  *  Start the timer... tick tock tick tock...
  * ------------------------------------------------------
  */
-	$BM =& load_class('Benchmark', 'core');
+	$BM = & load_class('Benchmark', 'core');
 	$BM->mark('total_execution_time_start');
 	$BM->mark('loading_time:_base_classes_start');
 
@@ -194,7 +194,7 @@ 
  *  Instantiate the hooks class
  * ------------------------------------------------------
  */
-	$EXT =& load_class('Hooks', 'core');
+	$EXT = & load_class('Hooks', 'core');
 
 /*
  * ------------------------------------------------------
@@ -213,7 +213,7 @@ 
  * depending on another class that uses it.
  *
  */
-	$CFG =& load_class('Config', 'core');
+	$CFG = & load_class('Config', 'core');
 
 	// Do we have any manually set config items in the index.php file?
 	if (isset($assign_to_config) && is_array($assign_to_config))
@@ -291,28 +291,28 @@ 
  *  Instantiate the UTF-8 class
  * ------------------------------------------------------
  */
-	$UNI =& load_class('Utf8', 'core');
+	$UNI = & load_class('Utf8', 'core');
 
 /*
  * ------------------------------------------------------
  *  Instantiate the URI class
  * ------------------------------------------------------
  */
-	$URI =& load_class('URI', 'core');
+	$URI = & load_class('URI', 'core');
 
 /*
  * ------------------------------------------------------
  *  Instantiate the routing class and set the routing
  * ------------------------------------------------------
  */
-	$RTR =& load_class('Router', 'core', isset($routing) ? $routing : NULL);
+	$RTR = & load_class('Router', 'core', isset($routing) ? $routing : NULL);
 
 /*
  * ------------------------------------------------------
  *  Instantiate the output class
  * ------------------------------------------------------
  */
-	$OUT =& load_class('Output', 'core');
+	$OUT = & load_class('Output', 'core');
 
 /*
  * ------------------------------------------------------
@@ -329,21 +329,21 @@ 
  * Load the security class for xss and csrf support
  * -----------------------------------------------------
  */
-	$SEC =& load_class('Security', 'core');
+	$SEC = & load_class('Security', 'core');
 
 /*
  * ------------------------------------------------------
  *  Load the Input class and sanitize globals
  * ------------------------------------------------------
  */
-	$IN	=& load_class('Input', 'core');
+	$IN = & load_class('Input', 'core');
 
 /*
  * ------------------------------------------------------
  *  Load the Language class
  * ------------------------------------------------------
  */
-	$LANG =& load_class('Lang', 'core');
+	$LANG = & load_class('Lang', 'core');
 
 /*
  * ------------------------------------------------------

Braces +8 added lines, -16 removed lines

@@ -169,12 +169,10 @@ 
 			file_exists(APPPATH.'vendor/autoload.php')
 				? require_once(APPPATH.'vendor/autoload.php')
 				: log_message('error', '$config[\'composer_autoload\'] is set to TRUE but '.APPPATH.'vendor/autoload.php was not found.');
-		}
-		elseif (file_exists($composer_autoload))
+		} elseif (file_exists($composer_autoload))
 		{
 			require_once($composer_autoload);
-		}
-		else
+		} else
 		{
 			log_message('error', 'Could not find the specified $config[\'composer_autoload\'] path: '.$composer_autoload);
 		}
@@ -250,8 +248,7 @@ 
 		// This is required for mb_convert_encoding() to strip invalid characters.
 		// That's utilized by CI_Utf8, but it's also done for consistency with iconv.
 		mb_substitute_character('none');
-	}
-	else
+	} else
 	{
 		define('MB_ENABLED', FALSE);
 	}
@@ -264,8 +261,7 @@ 
 		// iconv.internal_encoding is deprecated starting with PHP 5.6
 		// and it's usage triggers E_DEPRECATED messages.
 		@ini_set('iconv.internal_encoding', $charset);
-	}
-	else
+	} else
 	{
 		define('ICONV_ENABLED', FALSE);
 	}
@@ -402,16 +398,14 @@ 
 	if (empty($class) OR ! file_exists(APPPATH.'controllers/'.$RTR->directory.$class.'.php'))
 	{
 		$e404 = TRUE;
-	}
-	else
+	} else
 	{
 		require_once(APPPATH.'controllers/'.$RTR->directory.$class.'.php');
 
 		if ( ! class_exists($class, FALSE) OR $method[0] === '_' OR method_exists('CI_Controller', $method))
 		{
 			$e404 = TRUE;
-		}
-		elseif (method_exists($class, '_remap'))
+		} elseif (method_exists($class, '_remap'))
 		{
 			$params = array($method, array_slice($URI->rsegments, 2));
 			$method = '_remap';
@@ -453,8 +447,7 @@ 
 						$RTR->directory = '';
 					}
 				}
-			}
-			else
+			} else
 			{
 				$e404 = FALSE;
 			}
@@ -470,8 +463,7 @@ 
 				1 => $class,
 				2 => $method
 			);
-		}
-		else
+		} else
 		{
 			show_404($RTR->directory.$class.'/'.$method);
 		}

system/core/Controller.php

Spacing +3 added lines, -3 removed lines

@@ -65,17 +65,17 @@
 	 */
 	public function __construct()
 	{
-		self::$instance =& $this;
+		self::$instance = & $this;
 
 		// Assign all the class objects that were instantiated by the
 		// bootstrap file (CodeIgniter.php) to local class variables
 		// so that CI can run as one big super object.
 		foreach (is_loaded() as $var => $class)
 		{
-			$this->$var =& load_class($class);
+			$this->$var = & load_class($class);
 		}
 
-		$this->load =& load_class('Loader', 'core');
+		$this->load = & load_class('Loader', 'core');
 		$this->load->initialize();
 		log_message('info', 'Controller Class Initialized');
 	}

system/core/Lang.php

Spacing +3 added lines, -3 removed lines

@@ -53,14 +53,14 @@ 
 	 *
 	 * @var	array
 	 */
-	public $language =	array();
+	public $language = array();
 
 	/**
 	 * List of loaded language files
 	 *
 	 * @var	array
 	 */
-	public $is_loaded =	array();
+	public $is_loaded = array();
 
 	/**
 	 * Class constructor
@@ -108,7 +108,7 @@ 
 
 		if (empty($idiom) OR ! preg_match('/^[a-z_-]+$/i', $idiom))
 		{
-			$config =& get_config();
+			$config = & get_config();
 			$idiom = empty($config['language']) ? 'english' : $config['language'];
 		}
 

Braces +1 added lines, -2 removed lines

@@ -266,8 +266,7 @@
 		if ($reduce_linebreaks === TRUE)
 		{
 			$table['#<p>\n*</p>#'] = '';
-		}
-		else
+		} else
 		{
 			// If we have empty paragraph tags we add a non-breaking space
 			// otherwise most browsers won't treat them as true paragraphs

system/core/Log.php

Spacing +1 added lines, -1 removed lines

@@ -113,7 +113,7 @@
 	 */
 	public function __construct()
 	{
-		$config =& get_config();
+		$config = & get_config();
 
 		$this->_log_path = ($config['log_path'] !== '') ? $config['log_path'] : APPPATH.'logs/';
 		$this->_file_ext = (isset($config['log_file_extension']) && $config['log_file_extension'] !== '')

Braces +2 added lines, -4 removed lines

@@ -129,8 +129,7 @@ 
 		if (is_numeric($config['log_threshold']))
 		{
 			$this->_threshold = (int) $config['log_threshold'];
-		}
-		elseif (is_array($config['log_threshold']))
+		} elseif (is_array($config['log_threshold']))
 		{
 			$this->_threshold = 0;
 			$this->_threshold_array = array_flip($config['log_threshold']);
@@ -198,8 +197,7 @@ 
 			$microtime_short = sprintf("%06d", ($microtime_full - floor($microtime_full)) * 1000000);
 			$date = new DateTime(date('Y-m-d H:i:s.'.$microtime_short, $microtime_full));
 			$date = $date->format($this->_date_fmt);
-		}
-		else
+		} else
 		{
 			$date = date($this->_date_fmt);
 		}