ci-bonfire / Sprint

myth/CIModules/auth/views/login.php

Spacing +1 added lines, -1 removed lines

@@ -1,4 +1,4 @@
-<?= form_open( current_url(), ['id' => 'login_form']); ?>
+<?= form_open(current_url(), ['id' => 'login_form']); ?>
 
     <h2 class="form-signin-heading"><?= lang('auth.signin') ?></h2>
 

myth/CIModules/auth/views/register.php

Spacing +1 added lines, -1 removed lines

@@ -1,4 +1,4 @@
-<?= form_open( current_url(), ['id' => 'join_form'] ) ?>
+<?= form_open(current_url(), ['id' => 'join_form']) ?>
 
     <h2 class="form-signin-heading"><?= lang('auth.register') ?></h2>
 

myth/CIModules/cron/controllers/Cron.php

Spacing +21 added lines, -21 removed lines

@@ -62,7 +62,7 @@ 
         parent::__construct();
 
         // Load our tasks into the sytem.
-        require APPPATH .'config/cron.php';
+        require APPPATH.'config/cron.php';
     }
 
     //--------------------------------------------------------------------
@@ -73,7 +73,7 @@ 
      * @param string $alias
      * @return mixed
      */
-    public function run($alias=null)
+    public function run($alias = null)
     {
         // Has the system been disabled?
         if (Settings::get('is_disabled', 'cron') == 'y')
@@ -84,7 +84,7 @@ 
         $force_run = false;
 
         // Run one task or all?
-        if (! empty($alias))
+        if ( ! empty($alias))
         {
             $tasks = \Myth\Cron\CronManager::task($alias);
 
@@ -93,7 +93,7 @@ 
                 return CLI::error("Unable to find the task: '{$alias}'.");
             }
 
-            $tasks = [ $alias => $tasks];
+            $tasks = [$alias => $tasks];
             $force_run = true;
         }
         else
@@ -109,7 +109,7 @@ 
         // We need to be able to check against suspended tasks.
         $suspended = Settings::get('suspended_tasks', 'cron');
 
-        if (! is_array($suspended))
+        if ( ! is_array($suspended))
         {
             $suspended = array($suspended);
         }
@@ -154,7 +154,7 @@ 
      * @param null $task
      * @return mixed
      */
-    public function show($task=null)
+    public function show($task = null)
     {
         if (empty($task))
         {
@@ -170,15 +170,15 @@ 
             $tasks = \Myth\Cron\CronManager::task($task);
         }
 
-        if (! is_array($tasks))
+        if ( ! is_array($tasks))
         {
-            $tasks = [ $task => [
+            $tasks = [$task => [
                 'next_run'  => $tasks->nextRunDate(),
                 'prev_run'  => $tasks->previousRunDate()
             ]];
         }
 
-        if (! count($tasks))
+        if ( ! count($tasks))
         {
             return CLI::found('No tasks found.', 'red');
         }
@@ -193,7 +193,7 @@ 
          * Headers
          */
         echo CLI::write("Task\t\t\t\tNext Run\t\tPrevious Run");
-        echo CLI::write( str_repeat('-', 80) );
+        echo CLI::write(str_repeat('-', 80));
 
         foreach ($tasks as $alias => $task)
         {
@@ -201,14 +201,14 @@ 
             $color = 'yellow';
             $extra = '';
 
-            if (in_array($alias, $suspended) )
+            if (in_array($alias, $suspended))
             {
                 $color = 'blue';
                 $extra = "\t[Suspended]";
             }
 
             // Alias can only be 24 chars long.
-            $alias = strlen($alias) >= 32 ? substr($alias, 0, 28) .'... ' : $alias . str_repeat(" ", 32 - strlen($alias));
+            $alias = strlen($alias) >= 32 ? substr($alias, 0, 28).'... ' : $alias.str_repeat(" ", 32 - strlen($alias));
 
             $next = date('D Y-m-d H:i', $task['next_run']);
             $prev = date('D Y-m-d H:i', $task['prev_run']);
@@ -244,7 +244,7 @@ 
 
         $suspended[] = $alias;
 
-        if (Settings::save('suspended_tasks', $suspended, 'cron') )
+        if (Settings::save('suspended_tasks', $suspended, 'cron'))
         {
             return CLI::write('Done');
         }
@@ -273,11 +273,11 @@ 
         // Update the existing setting.
         $suspended = Settings::get('suspended_tasks', 'cron');
 
-        if (! empty($suspended))
+        if ( ! empty($suspended))
         {
-            unset($suspended[ array_search($alias, $suspended) ]);
+            unset($suspended[array_search($alias, $suspended)]);
 
-            if (! Settings::save('suspended_tasks', $suspended, 'cron') )
+            if ( ! Settings::save('suspended_tasks', $suspended, 'cron'))
             {
                 return CLI::error('Unkown problem saving the settings.');
             }
@@ -294,9 +294,9 @@ 
      */
     public function disable()
     {
-        if (! Settings::save('is_disabled', 'y', 'cron'))
+        if ( ! Settings::save('is_disabled', 'y', 'cron'))
         {
-            return CLI::error('Unknown problem saving the setting. '. CLI::color('Cron jobs will still run!', 'yellow'));
+            return CLI::error('Unknown problem saving the setting. '.CLI::color('Cron jobs will still run!', 'yellow'));
         }
 
         CLI::write('Done');
@@ -310,9 +310,9 @@ 
      */
     public function enable()
     {
-        if (! Settings::save('is_disabled', 'n', 'cron'))
+        if ( ! Settings::save('is_disabled', 'n', 'cron'))
         {
-            return CLI::error('Unknown problem saving the setting. '. CLI::color('Cron jobs will NOT run!', 'yellow'));
+            return CLI::error('Unknown problem saving the setting. '.CLI::color('Cron jobs will NOT run!', 'yellow'));
         }
 
         CLI::write('Done');
@@ -346,7 +346,7 @@ 
             $color = 'yellow';
             $extra = '';
 
-            if (in_array($alias, $suspended) )
+            if (in_array($alias, $suspended))
             {
                 $color = 'blue';
                 $extra = "[Suspended]";

myth/CIModules/database/controllers/Database.php

Spacing +22 added lines, -23 removed lines

@@ -36,11 +36,11 @@ 
 {
 
     protected $descriptions = [
-        'migrate'       => ['migrate [$to]',        'Runs the migrations up or down until schema at version \$to'],
-        'quietMigrate'  => ['quiteMigrate [$to]',   'Same as migrate but without any feedback.'],
-        'refresh'       => ['refresh',              'Runs migrations back to version 0 (uninstall) and then back to the most recent migration.'],
+        'migrate'       => ['migrate [$to]', 'Runs the migrations up or down until schema at version \$to'],
+        'quietMigrate'  => ['quiteMigrate [$to]', 'Same as migrate but without any feedback.'],
+        'refresh'       => ['refresh', 'Runs migrations back to version 0 (uninstall) and then back to the most recent migration.'],
         'newMigration'  => ['newMigration [$name]', 'Creates a new migration file.'],
-        'seed'          => ['seed [$name]',         'Runs the named database seeder.']
+        'seed'          => ['seed [$name]', 'Runs the named database seeder.']
     ];
 
     protected $long_descriptions = [
@@ -69,7 +69,7 @@ 
      * @param bool $silent If TRUE, will NOT display any prompts for verification.
      * @return bool|void
      */
-    public function migrate($type=null, $to = null, $silent = false)
+    public function migrate($type = null, $to = null, $silent = false)
     {
         $this->load->library('migration');
 
@@ -90,11 +90,11 @@ 
         $groups = array_keys($groups);
 
         // If it's not in the groups list, then assume it's a module.
-        if (! in_array($type, $groups))
+        if ( ! in_array($type, $groups))
         {
             if (strpos($type, 'mod:') !== 0)
             {
-                $type = 'mod:'. $type;
+                $type = 'mod:'.$type;
             }
         }
 
@@ -111,7 +111,7 @@ 
         $current = $this->migration->get_version($type);
 
         // Already at the desired version?
-        if ((! is_null($to) && $current == $to) OR (is_null($to) && $current == $latest))
+        if (( ! is_null($to) && $current == $to) OR (is_null($to) && $current == $latest))
         {
             return $silent ? true : CLI::write("\tDatabase is already at the desired version ({$current})", 'yellow');
         }
@@ -122,7 +122,7 @@ 
         // to the latest version.
         if (is_null($to)) {
             // If we're in silent mode, don't prompt, just go to the latest...
-            if (! $silent) {
+            if ( ! $silent) {
                 $go_ahead = CLI::prompt('Migrate to the latest available version?', array('y', 'n'));
 
                 if ($go_ahead == 'n') {
@@ -130,17 +130,16 @@ 
                 }
             }
 
-            if (! $this->migration->latest($type)) {
-                return CLI::error("\n\tERROR: " . $this->migration->error_string() . "\n");
+            if ( ! $this->migration->latest($type)) {
+                return CLI::error("\n\tERROR: ".$this->migration->error_string()."\n");
             }
         } else {
             if ($this->migration->version($type, $to) === false) {
-                return CLI::error("\n\tERROR: " . $this->migration->error_string() . "\n");
+                return CLI::error("\n\tERROR: ".$this->migration->error_string()."\n");
             }
         }
 
-        return $silent ? true :
-            CLI::write("\n\tSuccessfully migrated database from version {$current} to {$target}.\n", 'green');
+        return $silent ? true : CLI::write("\n\tSuccessfully migrated database from version {$current} to {$target}.\n", 'green');
     }
 
     //--------------------------------------------------------------------
@@ -154,7 +153,7 @@ 
      * @param null $to
      * @return bool|void
      */
-    public function quietMigrate($type='app', $to = null)
+    public function quietMigrate($type = 'app', $to = null)
     {
         return $this->migrate($type, $to, true);
     }
@@ -168,7 +167,7 @@ 
      * @param string $type  The group or module to refresh.
      * @return mixed
      */
-    public function refresh($type=null)
+    public function refresh($type = null)
     {
         $this->load->library('migration');
 
@@ -183,13 +182,13 @@ 
         }
 
         if ($this->migration->version($type, 0) === false) {
-            return CLI::error("\tERROR: " . $this->migration->error_string());
+            return CLI::error("\tERROR: ".$this->migration->error_string());
         }
 
         CLI::write(CLI::color("\tCleared the database.", 'green'));
 
         if ($this->migration->latest($type) === false) {
-            return CLI::error("\tERROR: " . $this->migration->error_string());
+            return CLI::error("\tERROR: ".$this->migration->error_string());
         }
 
         CLI::write("\tRe-installed the database to the latest migration.", 'green');
@@ -217,23 +216,23 @@ 
         $path = $this->migration->determine_migration_path($type);
 
         // Does the alias path exist in our config?
-        if (! $path) {
+        if ( ! $path) {
             return CLI::error("\tThe migration path for '{$type}' does not exist.'");
         }
 
         // Does the path really exist?
-        if (! is_dir($path)) {
+        if ( ! is_dir($path)) {
             return CLI::error("\tThe path for '{$type}' is not a directory.");
         }
 
         // Is the folder writeable?
-        if (! is_writeable($path)) {
+        if ( ! is_writeable($path)) {
             return CLI::error("\tThe folder for '{$type}' migrations is not writeable.");
         }
 
         $file = $this->migration->make_name($name);
 
-        $path = rtrim($path, '/') .'/'. $file;
+        $path = rtrim($path, '/').'/'.$file;
 
         $contents = <<<EOT
 <?php
@@ -269,7 +268,7 @@ 
         $this->load->helper('file');
 
         if (write_file($path, $contents)) {
-            return CLI::write("\tNew migration created: " . CLI::color($file, 'yellow'), 'green');
+            return CLI::write("\tNew migration created: ".CLI::color($file, 'yellow'), 'green');
         }
 
         return CLI::error("\tUnkown error trying to create migration: {$file}", 'red');

myth/CIModules/database/libraries/Seeder.php

Spacing +10 added lines, -10 removed lines

@@ -39,7 +39,7 @@ 
  */
 class Seeder {
 
-    public $error_string    = '';
+    public $error_string = '';
 
     protected $ci;
 
@@ -50,9 +50,9 @@ 
 
     //--------------------------------------------------------------------
 
-    public function __construct ()
+    public function __construct()
     {
-        $this->ci =& get_instance();
+        $this->ci = & get_instance();
 
         $this->is_cli = $this->ci->input->is_cli_request();
 
@@ -65,8 +65,8 @@ 
         $this->ci->load->dbforge();
 
         // Setup some convenience vars.
-        $this->db       =& $this->ci->db;
-        $this->dbforge  =& $this->ci->dbforge;
+        $this->db       = & $this->ci->db;
+        $this->dbforge  = & $this->ci->dbforge;
     }
 
     //--------------------------------------------------------------------
@@ -76,7 +76,7 @@ 
      * Run the database seeds. It's where the magic happens.
      * This method MUST be overridden by child classes.
      */
-    public function run ()
+    public function run()
     {
 
     }
@@ -89,23 +89,23 @@ 
      *
      * @param $class
      */
-    public function call ($class)
+    public function call($class)
     {
         if (empty($class))
         {
             // Ask the user...
-            $class = trim( CLI::prompt("Seeder name") );
+            $class = trim(CLI::prompt("Seeder name"));
 
             if (empty($class)) {
                 return CLI::error("\tNo Seeder was specified.");
             }
         }
 
-        $path = APPPATH .'database/seeds/'. str_replace('.php', '', $class) .'.php';
+        $path = APPPATH.'database/seeds/'.str_replace('.php', '', $class).'.php';
 
         if ( ! is_file($path))
         {
-            return CLI::error("\tUnable to find seed class: ". $class);
+            return CLI::error("\tUnable to find seed class: ".$class);
         }
 
         try {

myth/CIModules/docs/config/docs.php

Spacing +3 added lines, -3 removed lines

@@ -57,8 +57,8 @@ 
 | if realpath cannot find/read the folder.
 */
 $config['docs.folders'] = [
-    'application'   => APPPATH .'docs',
-    'developer'     => APPPATH .'../myth/_docs_src'
+    'application'   => APPPATH.'docs',
+    'developer'     => APPPATH.'../myth/_docs_src'
 ];
 
 /*
@@ -89,7 +89,7 @@ 
 | If you change it, ensure that it includes the period (.).
 */
 
-$config['docs.extension']    = '.md';
+$config['docs.extension'] = '.md';
 
 /*
  * If true, the 'developer' docs will be displayed in environments other than

myth/CIModules/docs/controllers/Docs.php

Spacing +10 added lines, -10 removed lines

@@ -75,9 +75,9 @@ 
         $this->load->helper('form');
         $this->load->helper('language');
 
-        $this->docbuilder = new \Myth\Docs\Builder( array('apppath' => APPPATH) );
+        $this->docbuilder = new \Myth\Docs\Builder(array('apppath' => APPPATH));
 
-        $formatter = function ($str) {
+        $formatter = function($str) {
             $converter = new \League\CommonMark\CommonMarkConverter();
             return $converter->convertToHtml($str);
         };
@@ -162,7 +162,7 @@ 
     {
         // Is displaying docs permitted for this environment?
         if (config_item('docs.permitted_environments')
-            && !in_array(ENVIRONMENT, config_item('docs.permitted_environments'))
+            && ! in_array(ENVIRONMENT, config_item('docs.permitted_environments'))
         ) {
             $this->setMessage(lang('docs_env_disabled'), 'error');
             redirect();
@@ -177,7 +177,7 @@ 
             $this->current_group = false;
         } // Are we allowed to show developer docs in this environment?
         elseif ($current_group == 'developer'
-                && !$this->showDevDocs
+                && ! $this->showDevDocs
                 && ENVIRONMENT != 'development'
         ) {
             if ($this->showAppDocs) {
@@ -216,8 +216,8 @@ 
 
         // If nothing left, then assign the default group and redirect to
         // a page we can do something with...
-        if (!count($segments)) {
-            redirect('docs/' . config_item('docs.default_group'));
+        if ( ! count($segments)) {
+            redirect('docs/'.config_item('docs.default_group'));
         }
 
         // Do we have a group specified? Bonfire Docs requires that a group
@@ -248,7 +248,7 @@ 
         $data = [];
 
         // Set the remaining data for the view
-        $data['docsDir'] = 'docs/' . $this->current_group . '/';
+        $data['docsDir'] = 'docs/'.$this->current_group.'/';
         $data['docsExt'] = config_item('docs.extension');
 
         $data['docMap'] = $this->docbuilder->buildDocumentMap($content);
@@ -267,7 +267,7 @@ 
      */
     private function buildTOC()
     {
-        $folder = $this->doc_folders[$this->current_group] . '/';
+        $folder = $this->doc_folders[$this->current_group].'/';
 
         $map = $this->docbuilder->buildTOC($folder);
 
@@ -291,11 +291,11 @@ 
         $docs_modules = array();
         foreach (\Bonfire\Modules::list_modules() as $module) {
             $ignored_folders = array();
-            $path            = \Bonfire\Modules::path($module) . $this->docsDir;
+            $path            = \Bonfire\Modules::path($module).$this->docsDir;
 
             // If these are developer docs, add the folder to the path.
             if ($this->current_group == $this->docsTypeBf) {
-                $path .= '/' . $this->docsTypeBf;
+                $path .= '/'.$this->docsTypeBf;
             } // For Application docs, ignore the 'developers' folder.
             else {
                 $ignored_folders[] = $this->docsTypeBf;

myth/CIModules/docs/views/_sidebar.php

Spacing +4 added lines, -4 removed lines

@@ -7,12 +7,12 @@ 
                         <div class="nav-header"><?php echo $file; ?></div>
                         <ul class="nav">
                             <?php foreach ($name as $line => $namer) : ?>
-                            <li><?php echo anchor($docsDir . str_replace($docsExt, '', $line), $namer); ?></li>
+                            <li><?php echo anchor($docsDir.str_replace($docsExt, '', $line), $namer); ?></li>
                             <?php endforeach; ?>
                         </ul>
                     </li>
                 <?php else : ?>
-                    <li><?php echo anchor($docsDir . str_replace($docsExt, '', $file), $name); ?></li>
+                    <li><?php echo anchor($docsDir.str_replace($docsExt, '', $file), $name); ?></li>
                 <?php endif ?>
             <?php endforeach ?>
         <?php else : ?>
@@ -29,12 +29,12 @@ 
                         <div class='nav-header'><?php echo $module; ?></div>
                         <ul class='nav'>
                         <?php foreach ($mod_files as $fileName => $title) : ?>
-                            <li><?php echo anchor(site_url($docsDir . '/' . str_replace($docsExt, '', $fileName)), ucwords($title)); ?></li>
+                            <li><?php echo anchor(site_url($docsDir.'/'.str_replace($docsExt, '', $fileName)), ucwords($title)); ?></li>
                         <?php endforeach; ?>
                         </ul>
                     </li>
                 <?php else : ?>
-                    <li class='parent'><?php echo anchor(site_url($docsDir . '/' . str_replace($docsExt, '', $module)), ucwords(str_replace('_', ' ', $module))); ?></li>
+                    <li class='parent'><?php echo anchor(site_url($docsDir.'/'.str_replace($docsExt, '', $module)), ucwords(str_replace('_', ' ', $module))); ?></li>
                 <?php endif ?>
             <?php endforeach ?>
         <?php endif ?>

myth/CIModules/docs/views/search.php

Spacing +1 added lines, -1 removed lines

@@ -2,7 +2,7 @@
 <h1><?php echo lang('docs_search_results') ?></h1>
 
 <div class="well">
-    <?php echo form_open( current_url(), 'class="form-inline"'); ?>
+    <?php echo form_open(current_url(), 'class="form-inline"'); ?>
         <input type="text" name="search_terms" class="form-control" style="width: 85%" value="<?php echo set_value('search_terms', $search_terms) ?>" />
         <input type="submit" name="submit" class="btn btn-primary" value="<?php echo lang('docs_search'); ?>"/>
     <?php echo form_close(); ?>