ci-bonfire / Sprint

myth/Events/Events.php

Indentation +169 added lines, -169 removed lines

@@ -36,176 +36,176 @@
 
 class Events {
 
-    /**
-     * The list of listeners.
-     *
-     * @var array
-     */
-    protected static $listeners = [];
-
-    /**
-     * Flag to let us know if we've read from the config file
-     * and have all of the defined events.
-     *
-     * @var bool
-     */
-    protected static $have_read_from_file = false;
-
-    //--------------------------------------------------------------------
-
-    /**
-     * Registers an action to happen on an event. The action can be any sort
-     * of callable:
-     *
-     *  Events::on('create', 'myFunction');               // procedural function
-     *  Events::on('create', ['myClass', 'myMethod']);    // Class::method
-     *  Events::on('create', [$myInstance, 'myMethod']);  // Method on an existing instance
-     *  Events::on('create', function() {});              // Closure
-     *
-     * @param $event_name
-     * @param callable $callback
-     * @param int $priority
-     */
-    public static function on($event_name, callable $callback, $priority=EVENTS_PRIORITY_NORMAL)
-    {
-        if (! isset(self::$listeners[$event_name]))
-        {
-            self::$listeners[$event_name] = [
-                true,   // If there's only 1 item, it's sorted.
-                [$priority],
-                [$callback]
-            ];
-        }
-        else
-        {
-            self::$listeners[$event_name][0] = false; // Not sorted
-            self::$listeners[$event_name][1][] = $priority;
-            self::$listeners[$event_name][2][] = $callback;
-        }
-    }
+	/**
+	 * The list of listeners.
+	 *
+	 * @var array
+	 */
+	protected static $listeners = [];
+
+	/**
+	 * Flag to let us know if we've read from the config file
+	 * and have all of the defined events.
+	 *
+	 * @var bool
+	 */
+	protected static $have_read_from_file = false;
+
+	//--------------------------------------------------------------------
+
+	/**
+	 * Registers an action to happen on an event. The action can be any sort
+	 * of callable:
+	 *
+	 *  Events::on('create', 'myFunction');               // procedural function
+	 *  Events::on('create', ['myClass', 'myMethod']);    // Class::method
+	 *  Events::on('create', [$myInstance, 'myMethod']);  // Method on an existing instance
+	 *  Events::on('create', function() {});              // Closure
+	 *
+	 * @param $event_name
+	 * @param callable $callback
+	 * @param int $priority
+	 */
+	public static function on($event_name, callable $callback, $priority=EVENTS_PRIORITY_NORMAL)
+	{
+		if (! isset(self::$listeners[$event_name]))
+		{
+			self::$listeners[$event_name] = [
+				true,   // If there's only 1 item, it's sorted.
+				[$priority],
+				[$callback]
+			];
+		}
+		else
+		{
+			self::$listeners[$event_name][0] = false; // Not sorted
+			self::$listeners[$event_name][1][] = $priority;
+			self::$listeners[$event_name][2][] = $callback;
+		}
+	}
     
-    //--------------------------------------------------------------------
-
-    /**
-     * Runs through all subscribed methods running them one at a time,
-     * until either:
-     *  a) All subscribers have finished or
-     *  b) a method returns false, at which point execution of subscribers stops.
-     *
-     * @param $event_name
-     * @return bool
-     */
-    public static function trigger($event_name, array $arguments = [])
-    {
-        // Read in our config/events file so that we have them all!
-        if (! self::$have_read_from_file)
-        {
-            if (is_file(APPPATH .'config/events.php'))
-            {
-                include APPPATH .'config/events.php';
-            }
-            self::$have_read_from_file = true;
-        }
-
-        foreach (self::listeners($event_name) as $listener)
-        {
-            $result = call_user_func_array($listener, $arguments);
-
-            if ($result === false)
-            {
-                return false;
-            }
-        }
-
-        return true;
-    }
+	//--------------------------------------------------------------------
+
+	/**
+	 * Runs through all subscribed methods running them one at a time,
+	 * until either:
+	 *  a) All subscribers have finished or
+	 *  b) a method returns false, at which point execution of subscribers stops.
+	 *
+	 * @param $event_name
+	 * @return bool
+	 */
+	public static function trigger($event_name, array $arguments = [])
+	{
+		// Read in our config/events file so that we have them all!
+		if (! self::$have_read_from_file)
+		{
+			if (is_file(APPPATH .'config/events.php'))
+			{
+				include APPPATH .'config/events.php';
+			}
+			self::$have_read_from_file = true;
+		}
+
+		foreach (self::listeners($event_name) as $listener)
+		{
+			$result = call_user_func_array($listener, $arguments);
+
+			if ($result === false)
+			{
+				return false;
+			}
+		}
+
+		return true;
+	}
     
-    //--------------------------------------------------------------------
-
-    /**
-     * Returns an array of listeners for a single event. They are
-     * sorted by priority.
-     *
-     * If the listener could not be found, returns FALSE, or TRUE if
-     * it was removed.
-     *
-     * @param $event_name
-     * @return array
-     */
-    public static function listeners($event_name)
-    {
-        if (! isset(self::$listeners[$event_name]))
-        {
-            return [];
-        }
-
-        // The list is not sorted
-        if (! self::$listeners[$event_name][0])
-        {
-            // Sort it!
-            array_multisort(self::$listeners[$event_name][1], SORT_NUMERIC, self::$listeners[$event_name][2]);
-
-            // Mark it as sorted already!
-            self::$listeners[$event_name][0] = true;
-        }
-
-        return self::$listeners[$event_name][2];
-    }
-
-    //--------------------------------------------------------------------
-
-    /**
-     * Removes a single listener from an event.
-     *
-     * If the listener couldn't be found, returns FALSE, else TRUE if
-     * it was removed.
-     *
-     * @param $event_name
-     * @param callable $listener
-     * @return bool
-     */
-    public static function removeListener($event_name, callable $listener)
-    {
-        if (! isset(self::$listeners[$event_name]))
-        {
-            return false;
-        }
-
-        foreach (self::$listeners[$event_name][2] as $index => $check)
-        {
-            if ($check === $listener)
-            {
-                unset(self::$listeners[$event_name][1][$index]);
-                unset(self::$listeners[$event_name][2][$index]);
-
-                return true;
-            }
-        }
-
-        return false;
-    }
-
-    //--------------------------------------------------------------------
-
-    /**
-     * Removes all listeners.
-     *
-     * If the event_name is specified, only listeners for that event will be
-     * removed, otherwise all listeners for all events are removed.
-     *
-     * @param null $event_name
-     */
-    public static function removeAllListeners($event_name=null)
-    {
-        if (! is_null($event_name))
-        {
-            unset(self::$listeners[$event_name]);
-        }
-        else {
-            self::$listeners = [];
-        }
-    }
-
-    //--------------------------------------------------------------------
+	//--------------------------------------------------------------------
+
+	/**
+	 * Returns an array of listeners for a single event. They are
+	 * sorted by priority.
+	 *
+	 * If the listener could not be found, returns FALSE, or TRUE if
+	 * it was removed.
+	 *
+	 * @param $event_name
+	 * @return array
+	 */
+	public static function listeners($event_name)
+	{
+		if (! isset(self::$listeners[$event_name]))
+		{
+			return [];
+		}
+
+		// The list is not sorted
+		if (! self::$listeners[$event_name][0])
+		{
+			// Sort it!
+			array_multisort(self::$listeners[$event_name][1], SORT_NUMERIC, self::$listeners[$event_name][2]);
+
+			// Mark it as sorted already!
+			self::$listeners[$event_name][0] = true;
+		}
+
+		return self::$listeners[$event_name][2];
+	}
+
+	//--------------------------------------------------------------------
+
+	/**
+	 * Removes a single listener from an event.
+	 *
+	 * If the listener couldn't be found, returns FALSE, else TRUE if
+	 * it was removed.
+	 *
+	 * @param $event_name
+	 * @param callable $listener
+	 * @return bool
+	 */
+	public static function removeListener($event_name, callable $listener)
+	{
+		if (! isset(self::$listeners[$event_name]))
+		{
+			return false;
+		}
+
+		foreach (self::$listeners[$event_name][2] as $index => $check)
+		{
+			if ($check === $listener)
+			{
+				unset(self::$listeners[$event_name][1][$index]);
+				unset(self::$listeners[$event_name][2][$index]);
+
+				return true;
+			}
+		}
+
+		return false;
+	}
+
+	//--------------------------------------------------------------------
+
+	/**
+	 * Removes all listeners.
+	 *
+	 * If the event_name is specified, only listeners for that event will be
+	 * removed, otherwise all listeners for all events are removed.
+	 *
+	 * @param null $event_name
+	 */
+	public static function removeAllListeners($event_name=null)
+	{
+		if (! is_null($event_name))
+		{
+			unset(self::$listeners[$event_name]);
+		}
+		else {
+			self::$listeners = [];
+		}
+	}
+
+	//--------------------------------------------------------------------
 
 }

Spacing +11 added lines, -11 removed lines

@@ -66,12 +66,12 @@ 
      * @param callable $callback
      * @param int $priority
      */
-    public static function on($event_name, callable $callback, $priority=EVENTS_PRIORITY_NORMAL)
+    public static function on($event_name, callable $callback, $priority = EVENTS_PRIORITY_NORMAL)
     {
-        if (! isset(self::$listeners[$event_name]))
+        if ( ! isset(self::$listeners[$event_name]))
         {
             self::$listeners[$event_name] = [
-                true,   // If there's only 1 item, it's sorted.
+                true, // If there's only 1 item, it's sorted.
                 [$priority],
                 [$callback]
             ];
@@ -98,11 +98,11 @@ 
     public static function trigger($event_name, array $arguments = [])
     {
         // Read in our config/events file so that we have them all!
-        if (! self::$have_read_from_file)
+        if ( ! self::$have_read_from_file)
         {
-            if (is_file(APPPATH .'config/events.php'))
+            if (is_file(APPPATH.'config/events.php'))
             {
-                include APPPATH .'config/events.php';
+                include APPPATH.'config/events.php';
             }
             self::$have_read_from_file = true;
         }
@@ -134,13 +134,13 @@ 
      */
     public static function listeners($event_name)
     {
-        if (! isset(self::$listeners[$event_name]))
+        if ( ! isset(self::$listeners[$event_name]))
         {
             return [];
         }
 
         // The list is not sorted
-        if (! self::$listeners[$event_name][0])
+        if ( ! self::$listeners[$event_name][0])
         {
             // Sort it!
             array_multisort(self::$listeners[$event_name][1], SORT_NUMERIC, self::$listeners[$event_name][2]);
@@ -166,7 +166,7 @@ 
      */
     public static function removeListener($event_name, callable $listener)
     {
-        if (! isset(self::$listeners[$event_name]))
+        if ( ! isset(self::$listeners[$event_name]))
         {
             return false;
         }
@@ -195,9 +195,9 @@ 
      *
      * @param null $event_name
      */
-    public static function removeAllListeners($event_name=null)
+    public static function removeAllListeners($event_name = null)
     {
-        if (! is_null($event_name))
+        if ( ! is_null($event_name))
         {
             unset(self::$listeners[$event_name]);
         }

Braces +2 added lines, -4 removed lines

@@ -75,8 +75,7 @@ 
                 [$priority],
                 [$callback]
             ];
-        }
-        else
+        } else
         {
             self::$listeners[$event_name][0] = false; // Not sorted
             self::$listeners[$event_name][1][] = $priority;
@@ -200,8 +199,7 @@ 
         if (! is_null($event_name))
         {
             unset(self::$listeners[$event_name]);
-        }
-        else {
+        } else {
             self::$listeners = [];
         }
     }

myth/Forensics/Console.php

Indentation +5 added lines, -5 removed lines

@@ -167,11 +167,11 @@
 
 	public function reset()
 	{
-	    self::$logs = array(
-		    'console'		=> array(),
-		    'log_count'		=> 0,
-		    'memory_count'	=> 0,
-	    );
+		self::$logs = array(
+			'console'		=> array(),
+			'log_count'		=> 0,
+			'memory_count'	=> 0,
+		);
 	}
 
 	//--------------------------------------------------------------------

Spacing +6 added lines, -6 removed lines

@@ -92,7 +92,7 @@ 
 	*/
 	public static function init() 
 	{
-		self::$ci =& get_instance();
+		self::$ci = & get_instance();
 	}
 	
 	//--------------------------------------------------------------------
@@ -105,7 +105,7 @@ 
 		Parameters:
 			$data	- The variable to log.
 	*/
-	public static function log($data=null) 
+	public static function log($data = null) 
 	{
 		if ($data !== 0 && empty($data)) 
 		{ 
@@ -131,7 +131,7 @@ 
 			$object	- The object to store the memory usage of.
 			$name	- The name to be displayed in the console.
 	*/
-	public static function logMemory($object=false, $name='PHP')
+	public static function logMemory($object = false, $name = 'PHP')
 	{
 		$memory = memory_get_usage();
 		
@@ -182,15 +182,15 @@ 
 	// !PRIVATE METHODS
 	//--------------------------------------------------------------------
 	
-	protected static function addToConsole($log=null, $item=null)
+	protected static function addToConsole($log = null, $item = null)
 	{
 		if (empty($log) || empty($item)) 
 		{ 
 			return;
 		}
 		
-		self::$logs['console'][]	= $item;
-		self::$logs[$log] 			+= 1;
+		self::$logs['console'][] = $item;
+		self::$logs[$log] += 1;
 	}
 	
 	//--------------------------------------------------------------------

myth/Forensics/views/profiler_template.php

Spacing +2 added lines, -2 removed lines

@@ -196,7 +196,7 @@ 
 				Load Time
 			</a>
 			<a href="#" id="ci-profiler-menu-memory" onclick="ci_profiler_bar.show('ci-profiler-memory', 'ci-profiler-menu-memory'); return false;">
-				<span><?php echo (! function_exists('memory_get_usage')) ? '0' : round(memory_get_usage()/1024/1024, 2).' MB' ?></span>
+				<span><?php echo ( ! function_exists('memory_get_usage')) ? '0' : round(memory_get_usage() / 1024 / 1024, 2).' MB' ?></span>
 				Memory Used
 			</a>
 		<?php endif; ?>
@@ -384,7 +384,7 @@ 
 
 					<?php $append = ($section == 'get' || $section == 'post') ? '_data' : '' ?>
 					<a href="#" onclick="ci_profiler_bar.toggle_data_table('<?php echo $section ?>'); return false;">
-						<h2><?php echo lang('profiler_' . $section . $append) ?></h2>
+						<h2><?php echo lang('profiler_'.$section.$append) ?></h2>
 					</a>
 
 

Braces +35 added lines, -14 removed lines

@@ -257,9 +257,12 @@ 
 				<?php endforeach; ?>
 				</table>
 
-			<?php else : ?>
+			<?php else {
+	: ?>
 
-				<?php echo $sections['console']; ?>
+				<?php echo $sections['console'];
+}
+?>
 
 			<?php endif; ?>
 		</div>
@@ -288,9 +291,12 @@ 
 				<?php endforeach; ?>
 				</table>
 
-			<?php else : ?>
+			<?php else {
+	: ?>
 
-				<?php echo $sections['console']; ?>
+				<?php echo $sections['console'];
+}
+?>
 
 			<?php endif; ?>
 		</div>
@@ -309,9 +315,12 @@ 
 				<?php endforeach; ?>
 				</table>
 
-			<?php else : ?>
+			<?php else {
+	: ?>
 
-				<?php echo $sections['benchmarks']; ?>
+				<?php echo $sections['benchmarks'];
+}
+?>
 
 			<?php endif; ?>
 		</div>
@@ -332,9 +341,12 @@ 
 				<?php endforeach; ?>
 				</table>
 
-			<?php else : ?>
+			<?php else {
+	: ?>
 
-				<?php echo $sections['queries']; ?>
+				<?php echo $sections['queries'];
+}
+?>
 
 			<?php endif; ?>
 		</div>
@@ -394,8 +406,11 @@ 
 						<?php foreach ($sections[$section] as $key => $val) : ?>
 							<tr><td class="hilight"><?php echo $key ?></td><td><?php echo htmlspecialchars($val) ?></td></tr>
 						<?php endforeach; ?>
-						<?php else : ?>
-							<tr><td><?php echo $sections[$section]; ?></td></tr>
+						<?php else {
+	: ?>
+							<tr><td><?php echo $sections[$section];
+}
+?></td></tr>
 						<?php endif; ?>
 						</table>
 				<?php endif; ?>
@@ -422,19 +437,25 @@ 
 				<?php endforeach; ?>
 				</table>
 
-			<?php else : ?>
+			<?php else {
+	: ?>
 
-				<?php echo $sections['files']; ?>
+				<?php echo $sections['files'];
+}
+?>
 
 			<?php endif; ?>
 		</div>
 	<?php endif; ?>
 
 
-<?php else: ?>
+<?php else {
+	: ?>
 
 	<p class="ci-profiler-box"><?php echo lang('profiler_no_profiles') ?></p>
 
-<?php endif; ?>
+<?php endif;
+}
+?>
 
 </div>	<!-- /codeigniter_profiler -->

myth/Forge/FileKit.php

Indentation +197 added lines, -197 removed lines

@@ -32,206 +32,206 @@
 
 class FileKit {
 
-    /**
-     * Appends data to the end of a file.
-     *
-     * @param $file
-     * @param $content
-     * @return bool|int
-     */
-    public function append($file, $content)
-    {
-        if (empty($content))
-        {
-            return true;
-        }
-
-        // Ensure that $content has a newline at the end
-        $content = rtrim($content) ."\n";
-
-        $fh = fopen($file, 'a');
-        $result = fwrite($fh, $content);
-        fclose($fh);
-
-        return $result;
-    }
-
-    //--------------------------------------------------------------------
-
-    /**
-     * Prepends string content to a file. For very large files
-     * this method could have memory issues, but the primary usage
-     * of source files shouldn't ever get large enough to cause issues.
-     *
-     * @param $file
-     * @param $content
-     * @return bool|int
-     */
-    public function prepend($file, $content)
-    {
-        if (empty($content))
-        {
-            return true;
-        }
-
-        // Ensure that $content has a newline at the end
-        $content = rtrim($content) ."\n";
-
-        $file_contents = file_get_contents($file);
-
-        if ($file_contents === false)
-        {
-            throw new \RuntimeException( sprintf(lang('errors.reading_file'), $file));
-        }
-
-        $result = file_put_contents($file, $content . $file_contents);
-
-        return (bool)$result;
-    }
-
-    //--------------------------------------------------------------------
-
-    /**
-     * Inserts $content before the line that matches $before. NOT case-
-     * sensitive.
-     *
-     * @param $file
-     * @param $before
-     * @param $content
-     * @return int
-     */
-    public function before($file, $before, $content)
-    {
-        if (empty($content))
-        {
-            return true;
-        }
-
-        // Ensure that $content has a newline at the end
-        $content = rtrim($content) ."\n";
-
-        $lines = file($file);
-
-        if ($lines === false)
-        {
-            throw new \RuntimeException( sprintf( lang('errors.file_not_found'), $file ));
-        }
-
-        // Where to insert the row.
-        $location = null;
-
-        foreach ($lines as $index => $line)
-        {
-            if (strtolower($line) == strtolower($before) )
-            {
-                $location = $index;
-                break;
-            }
-        }
-
-        array_splice($lines, $location, 0, $content);
-
-        $result = file_put_contents($file, $lines);
-
-        return (bool)$result;
-    }
-
-    //--------------------------------------------------------------------
-
-    public function after($file, $after, $content)
-    {
-        if (empty($content))
-        {
-            return true;
-        }
-
-        // Ensure that $content has a newline at the end
-        $content = rtrim($content) ."\n";
-
-        $lines = file($file);
-
-        if ($lines === false)
-        {
-            throw new \RuntimeException( sprintf( lang('errors.file_not_found'), $file ) );
-        }
-
-        // Where to insert the row.
-        $location = null;
-
-        foreach ($lines as $index => $line)
-        {
-            if (strtolower($line) == strtolower($after) )
-            {
-                $location = $index;
-                break;
-            }
-        }
-
-        array_splice($lines, $location +1, 0, $content);
-
-        $result = file_put_contents($file, $lines);
-
-        return (bool)$result;
-    }
-
-    //--------------------------------------------------------------------
-
-    /**
-     * Replaces all instances of $search in the file with $replace.
-     *
-     * @param $file
-     * @param $search
-     * @param $replace
-     * @return int
-     */
-    public function replaceIn($file, $search, $replace)
-    {
-        $file_contents = file_get_contents($file);
-
-        if ($file_contents === false)
-        {
-            throw new \RuntimeException( sprintf( lang('errors.reading_file'), $file ) );
-        }
-
-        $file_contents = str_replace($search, $replace, $file_contents);
-
-        $result = file_put_contents($file, $file_contents);
-
-        return (bool)$result;
-    }
-
-    //--------------------------------------------------------------------
-
-    /**
-     * Uses preg_replace to replace content within the file.
-     *
-     * @param $file
-     * @param $pattern
-     * @param $replace
-     * @return int
-     */
-    public function replaceWithRegex($file, $pattern, $replace)
-    {
-        $file_contents = file_get_contents($file);
-
-        if ($file_contents === false)
-        {
-            throw new \RuntimeException( sprintf( lang('errors.reading_file'), $file ) );
-        }
+	/**
+	 * Appends data to the end of a file.
+	 *
+	 * @param $file
+	 * @param $content
+	 * @return bool|int
+	 */
+	public function append($file, $content)
+	{
+		if (empty($content))
+		{
+			return true;
+		}
+
+		// Ensure that $content has a newline at the end
+		$content = rtrim($content) ."\n";
+
+		$fh = fopen($file, 'a');
+		$result = fwrite($fh, $content);
+		fclose($fh);
+
+		return $result;
+	}
+
+	//--------------------------------------------------------------------
+
+	/**
+	 * Prepends string content to a file. For very large files
+	 * this method could have memory issues, but the primary usage
+	 * of source files shouldn't ever get large enough to cause issues.
+	 *
+	 * @param $file
+	 * @param $content
+	 * @return bool|int
+	 */
+	public function prepend($file, $content)
+	{
+		if (empty($content))
+		{
+			return true;
+		}
+
+		// Ensure that $content has a newline at the end
+		$content = rtrim($content) ."\n";
+
+		$file_contents = file_get_contents($file);
+
+		if ($file_contents === false)
+		{
+			throw new \RuntimeException( sprintf(lang('errors.reading_file'), $file));
+		}
+
+		$result = file_put_contents($file, $content . $file_contents);
+
+		return (bool)$result;
+	}
+
+	//--------------------------------------------------------------------
+
+	/**
+	 * Inserts $content before the line that matches $before. NOT case-
+	 * sensitive.
+	 *
+	 * @param $file
+	 * @param $before
+	 * @param $content
+	 * @return int
+	 */
+	public function before($file, $before, $content)
+	{
+		if (empty($content))
+		{
+			return true;
+		}
+
+		// Ensure that $content has a newline at the end
+		$content = rtrim($content) ."\n";
+
+		$lines = file($file);
+
+		if ($lines === false)
+		{
+			throw new \RuntimeException( sprintf( lang('errors.file_not_found'), $file ));
+		}
+
+		// Where to insert the row.
+		$location = null;
+
+		foreach ($lines as $index => $line)
+		{
+			if (strtolower($line) == strtolower($before) )
+			{
+				$location = $index;
+				break;
+			}
+		}
+
+		array_splice($lines, $location, 0, $content);
+
+		$result = file_put_contents($file, $lines);
+
+		return (bool)$result;
+	}
+
+	//--------------------------------------------------------------------
+
+	public function after($file, $after, $content)
+	{
+		if (empty($content))
+		{
+			return true;
+		}
+
+		// Ensure that $content has a newline at the end
+		$content = rtrim($content) ."\n";
+
+		$lines = file($file);
+
+		if ($lines === false)
+		{
+			throw new \RuntimeException( sprintf( lang('errors.file_not_found'), $file ) );
+		}
+
+		// Where to insert the row.
+		$location = null;
+
+		foreach ($lines as $index => $line)
+		{
+			if (strtolower($line) == strtolower($after) )
+			{
+				$location = $index;
+				break;
+			}
+		}
+
+		array_splice($lines, $location +1, 0, $content);
+
+		$result = file_put_contents($file, $lines);
+
+		return (bool)$result;
+	}
+
+	//--------------------------------------------------------------------
+
+	/**
+	 * Replaces all instances of $search in the file with $replace.
+	 *
+	 * @param $file
+	 * @param $search
+	 * @param $replace
+	 * @return int
+	 */
+	public function replaceIn($file, $search, $replace)
+	{
+		$file_contents = file_get_contents($file);
+
+		if ($file_contents === false)
+		{
+			throw new \RuntimeException( sprintf( lang('errors.reading_file'), $file ) );
+		}
+
+		$file_contents = str_replace($search, $replace, $file_contents);
+
+		$result = file_put_contents($file, $file_contents);
+
+		return (bool)$result;
+	}
+
+	//--------------------------------------------------------------------
+
+	/**
+	 * Uses preg_replace to replace content within the file.
+	 *
+	 * @param $file
+	 * @param $pattern
+	 * @param $replace
+	 * @return int
+	 */
+	public function replaceWithRegex($file, $pattern, $replace)
+	{
+		$file_contents = file_get_contents($file);
+
+		if ($file_contents === false)
+		{
+			throw new \RuntimeException( sprintf( lang('errors.reading_file'), $file ) );
+		}
 
-        $file_contents = preg_replace($pattern, $replace, $file_contents);
-
-        $result = false;
+		$file_contents = preg_replace($pattern, $replace, $file_contents);
+
+		$result = false;
 
-        // Don't let us erase a file!
-        if (! empty($file_contents))
-        {
-            $result = file_put_contents( $file, $file_contents );
-        }
+		// Don't let us erase a file!
+		if (! empty($file_contents))
+		{
+			$result = file_put_contents( $file, $file_contents );
+		}
 
-        return (bool)$result;
-    }
+		return (bool)$result;
+	}
 
-    //--------------------------------------------------------------------
+	//--------------------------------------------------------------------
 
 }

Spacing +20 added lines, -20 removed lines

@@ -47,7 +47,7 @@ 
         }
 
         // Ensure that $content has a newline at the end
-        $content = rtrim($content) ."\n";
+        $content = rtrim($content)."\n";
 
         $fh = fopen($file, 'a');
         $result = fwrite($fh, $content);
@@ -75,18 +75,18 @@ 
         }
 
         // Ensure that $content has a newline at the end
-        $content = rtrim($content) ."\n";
+        $content = rtrim($content)."\n";
 
         $file_contents = file_get_contents($file);
 
         if ($file_contents === false)
         {
-            throw new \RuntimeException( sprintf(lang('errors.reading_file'), $file));
+            throw new \RuntimeException(sprintf(lang('errors.reading_file'), $file));
         }
 
-        $result = file_put_contents($file, $content . $file_contents);
+        $result = file_put_contents($file, $content.$file_contents);
 
-        return (bool)$result;
+        return (bool) $result;
     }
 
     //--------------------------------------------------------------------
@@ -108,13 +108,13 @@ 
         }
 
         // Ensure that $content has a newline at the end
-        $content = rtrim($content) ."\n";
+        $content = rtrim($content)."\n";
 
         $lines = file($file);
 
         if ($lines === false)
         {
-            throw new \RuntimeException( sprintf( lang('errors.file_not_found'), $file ));
+            throw new \RuntimeException(sprintf(lang('errors.file_not_found'), $file));
         }
 
         // Where to insert the row.
@@ -122,7 +122,7 @@ 
 
         foreach ($lines as $index => $line)
         {
-            if (strtolower($line) == strtolower($before) )
+            if (strtolower($line) == strtolower($before))
             {
                 $location = $index;
                 break;
@@ -133,7 +133,7 @@ 
 
         $result = file_put_contents($file, $lines);
 
-        return (bool)$result;
+        return (bool) $result;
     }
 
     //--------------------------------------------------------------------
@@ -146,13 +146,13 @@ 
         }
 
         // Ensure that $content has a newline at the end
-        $content = rtrim($content) ."\n";
+        $content = rtrim($content)."\n";
 
         $lines = file($file);
 
         if ($lines === false)
         {
-            throw new \RuntimeException( sprintf( lang('errors.file_not_found'), $file ) );
+            throw new \RuntimeException(sprintf(lang('errors.file_not_found'), $file));
         }
 
         // Where to insert the row.
@@ -160,18 +160,18 @@ 
 
         foreach ($lines as $index => $line)
         {
-            if (strtolower($line) == strtolower($after) )
+            if (strtolower($line) == strtolower($after))
             {
                 $location = $index;
                 break;
             }
         }
 
-        array_splice($lines, $location +1, 0, $content);
+        array_splice($lines, $location + 1, 0, $content);
 
         $result = file_put_contents($file, $lines);
 
-        return (bool)$result;
+        return (bool) $result;
     }
 
     //--------------------------------------------------------------------
@@ -190,14 +190,14 @@ 
 
         if ($file_contents === false)
         {
-            throw new \RuntimeException( sprintf( lang('errors.reading_file'), $file ) );
+            throw new \RuntimeException(sprintf(lang('errors.reading_file'), $file));
         }
 
         $file_contents = str_replace($search, $replace, $file_contents);
 
         $result = file_put_contents($file, $file_contents);
 
-        return (bool)$result;
+        return (bool) $result;
     }
 
     //--------------------------------------------------------------------
@@ -216,7 +216,7 @@ 
 
         if ($file_contents === false)
         {
-            throw new \RuntimeException( sprintf( lang('errors.reading_file'), $file ) );
+            throw new \RuntimeException(sprintf(lang('errors.reading_file'), $file));
         }
 
         $file_contents = preg_replace($pattern, $replace, $file_contents);
@@ -224,12 +224,12 @@ 
         $result = false;
 
         // Don't let us erase a file!
-        if (! empty($file_contents))
+        if ( ! empty($file_contents))
         {
-            $result = file_put_contents( $file, $file_contents );
+            $result = file_put_contents($file, $file_contents);
         }
 
-        return (bool)$result;
+        return (bool) $result;
     }
 
     //--------------------------------------------------------------------

myth/Mail/BaseMailer.php

Indentation +283 added lines, -283 removed lines

@@ -40,290 +40,290 @@
  */
 class BaseMailer {
 
-    /**
-     * How the email is delivered.
-     * Either 'send' or 'queue'.
-     * @var string
-     */
-    protected $action = 'send';
-
-    protected $from     = null;
-    protected $to       = null;
-    protected $reply_to = null;
-    protected $cc       = null;
-    protected $bcc      = null;
-
-    protected $message  = null;
-
-    protected $theme    = 'email';
-    protected $layout   = 'index';
-    protected $view     = null;
-
-    /**
-     * The MailService to use. If NULL
-     * will use the system default.
-     * @var null
-     */
-    protected $service_name  = null;
-
-    protected $service = null;
-
-    /**
-     * Used for theming the email messages.
-     * @var null
-     */
-    protected $themer = null;
-
-    //--------------------------------------------------------------------
-
-    /**
-     * Constructor
-     *
-     * Simply allows us to override the default settings for this mailer.
-     *
-     * @param null $options
-     */
-    public function __construct($options=null)
-    {
-        if (! empty($options))
-        {
-            $this->setOptions($options);
-        }
-    }
-
-    //--------------------------------------------------------------------
-
-    /**
-     * Sets the basic options available to the mailer, like 'from', 'to',
-     * 'cc', 'bcc', etc.
-     *
-     * @param $options
-     */
-    public function setOptions($options)
-    {
-        if (is_array($options))
-        {
-            foreach ($options as $key => $value)
-            {
-                if ($key == 'service')
-                {
-                    $this->service =& $value;
-                    continue;
-                }
-
-                if (property_exists($this, $key))
-                {
-                    $this->$key = $value;
-                }
-            }
-        }
-    }
-
-    //--------------------------------------------------------------------
-
-
-
-    /**
-     * Sends an email immediately using the system-defined MailService.
-     *
-     * @param string $to // Who the email is being sent to.
-     * @param string $subject // The subject line for the email
-     * @param strign $data // the key/value pairs to send to the views.
-     * @param string $view // You can override the view used for the email here.
-     *                          // You can change themes by prepending theme name
-     *                          // like: 'newtheme:newview'
-     *
-     * @return bool
-     */
-    public function send($to, $subject, $data=[], $view=null)
-    {
-        // Are we pretending to send?
-        if (config_item('mail.pretend') === true)
-        {
-            return true;
-        }
-
-        $this->startMailService();
-
-        $this->service->to($to);
-        $this->service->subject($subject);
-
-        if (is_array($this->from)) {
-            $this->service->from($this->from[0], $this->from[1]);
-        }
-        else
-        {
-            $this->service->from($this->from);
-        }
-
-        if (! empty($this->cc))         $this->service->cc($this->cc);
-        if (! empty($this->bcc))        $this->service->bcc($this->bcc);
-
-        if (is_array($this->reply_to)) {
-            $this->service->reply_to($this->reply_to[0], $this->reply_to[1]);
-        }
-        else
-        {
-            $this->service->reply_to($this->reply_to);
-        }
-
-
-        // Determine the view to use. We have to hack this a bit with
-        // the debug_backtrace, though, to make it all function in the background.
-        list(, $method) = debug_backtrace(false);
-
-        $view = 'emails/'. strtolower( (new \ReflectionClass($this))->getShortName() ) .'/'. $method['function'];
-
-        // Get our message's text and html versions based on which files exist...
-        $basepath = APPPATH .'views/'. $view;
-
-        // Is a text version available?
-        if (file_exists($basepath .'.text.php'))
-        {
-            $text = $this->load->view($view .'.text.php', $data, true);
-            $this->service->text_message($text);
-        }
-
-        // If an html version is around, we need to theme it out
-        if (file_exists($basepath .'.html.php'))
-        {
-            $this->startThemer();
-
-            $this->themer->setTheme($this->theme);
-
-            // Determine the correct layout to use
-            $layout = ! empty($this->layout) ? $this->layout : NULL;
-            $this->themer->setLayout($layout);
-
-            $this->themer->set($data);
-
-            // Render the view into a var we can pass to the layout.
-            $content = $this->themer->display($view .'.html.php');
-
-            $this->themer->set('content', $content);
-
-            $this->service->html_message( $this->themer->display($this->theme .':'. $layout) );
-        }
-
-        if (! $this->service->send() )
-        {
-            // todo do something here
-            return false;
-        }
-
-        return true;
-    }
-
-    //--------------------------------------------------------------------
-
-    /**
-     * Allows you to customize the headers sent with the email. You can
-     * do them one at a time by passing $field and $value, or pass an array
-     * of $field => $value pairs as the first parameter.
-     *
-     * @param string|array  $field
-     * @param string        $value
-     */
-    public function header($field, $value=null)
-    {
-        $this->startMailService();
-
-        $this->service->setHeader($field, $value);
-    }
-
-    //--------------------------------------------------------------------
-
-    /**
-     * Adds an attachment to the current email that is being built.
-     *
-     * @param string    $filename
-     * @param string    $disposition    like 'inline'. Default is 'attachment'
-     * @param string    $newname        If you'd like to rename the file for delivery
-     * @param string    $mime           Custom defined mime type.
-     */
-    public function attach($filename, $disposition=null, $newname=null, $mime=null)
-    {
-        $this->startMailService();
-
-        $this->service->attach($filename, $disposition, $newname, $mime);
-    }
-
-    //--------------------------------------------------------------------
-
-    //--------------------------------------------------------------------
-    // Private Methods
-    //--------------------------------------------------------------------
-
-    /**
-     * Starts up the service name specified in $service_name.
-     *
-     * @param $service_name
-     */
-    protected function startMailService()
-    {
-        // Only once!
-        if (! empty($this->service) && is_object($this->service))
-        {
-            return;
-        }
-
-        $service_name = ! empty($this->service_name) ? $this->service_name : config_item('mail.default_service');
-
-        if (! class_exists($service_name))
-        {
-            throw new \RuntimeException( sprintf( lang('mail.invalid_service'), $service_name) );
-        }
-
-        $this->service = new $service_name();
-    }
-
-    //--------------------------------------------------------------------
-
-    /**
-     * Fires up the default themer so we can use it to theme our HTML messages.
-     */
-    protected function startThemer()
-    {
-        /*
+	/**
+	 * How the email is delivered.
+	 * Either 'send' or 'queue'.
+	 * @var string
+	 */
+	protected $action = 'send';
+
+	protected $from     = null;
+	protected $to       = null;
+	protected $reply_to = null;
+	protected $cc       = null;
+	protected $bcc      = null;
+
+	protected $message  = null;
+
+	protected $theme    = 'email';
+	protected $layout   = 'index';
+	protected $view     = null;
+
+	/**
+	 * The MailService to use. If NULL
+	 * will use the system default.
+	 * @var null
+	 */
+	protected $service_name  = null;
+
+	protected $service = null;
+
+	/**
+	 * Used for theming the email messages.
+	 * @var null
+	 */
+	protected $themer = null;
+
+	//--------------------------------------------------------------------
+
+	/**
+	 * Constructor
+	 *
+	 * Simply allows us to override the default settings for this mailer.
+	 *
+	 * @param null $options
+	 */
+	public function __construct($options=null)
+	{
+		if (! empty($options))
+		{
+			$this->setOptions($options);
+		}
+	}
+
+	//--------------------------------------------------------------------
+
+	/**
+	 * Sets the basic options available to the mailer, like 'from', 'to',
+	 * 'cc', 'bcc', etc.
+	 *
+	 * @param $options
+	 */
+	public function setOptions($options)
+	{
+		if (is_array($options))
+		{
+			foreach ($options as $key => $value)
+			{
+				if ($key == 'service')
+				{
+					$this->service =& $value;
+					continue;
+				}
+
+				if (property_exists($this, $key))
+				{
+					$this->$key = $value;
+				}
+			}
+		}
+	}
+
+	//--------------------------------------------------------------------
+
+
+
+	/**
+	 * Sends an email immediately using the system-defined MailService.
+	 *
+	 * @param string $to // Who the email is being sent to.
+	 * @param string $subject // The subject line for the email
+	 * @param strign $data // the key/value pairs to send to the views.
+	 * @param string $view // You can override the view used for the email here.
+	 *                          // You can change themes by prepending theme name
+	 *                          // like: 'newtheme:newview'
+	 *
+	 * @return bool
+	 */
+	public function send($to, $subject, $data=[], $view=null)
+	{
+		// Are we pretending to send?
+		if (config_item('mail.pretend') === true)
+		{
+			return true;
+		}
+
+		$this->startMailService();
+
+		$this->service->to($to);
+		$this->service->subject($subject);
+
+		if (is_array($this->from)) {
+			$this->service->from($this->from[0], $this->from[1]);
+		}
+		else
+		{
+			$this->service->from($this->from);
+		}
+
+		if (! empty($this->cc))         $this->service->cc($this->cc);
+		if (! empty($this->bcc))        $this->service->bcc($this->bcc);
+
+		if (is_array($this->reply_to)) {
+			$this->service->reply_to($this->reply_to[0], $this->reply_to[1]);
+		}
+		else
+		{
+			$this->service->reply_to($this->reply_to);
+		}
+
+
+		// Determine the view to use. We have to hack this a bit with
+		// the debug_backtrace, though, to make it all function in the background.
+		list(, $method) = debug_backtrace(false);
+
+		$view = 'emails/'. strtolower( (new \ReflectionClass($this))->getShortName() ) .'/'. $method['function'];
+
+		// Get our message's text and html versions based on which files exist...
+		$basepath = APPPATH .'views/'. $view;
+
+		// Is a text version available?
+		if (file_exists($basepath .'.text.php'))
+		{
+			$text = $this->load->view($view .'.text.php', $data, true);
+			$this->service->text_message($text);
+		}
+
+		// If an html version is around, we need to theme it out
+		if (file_exists($basepath .'.html.php'))
+		{
+			$this->startThemer();
+
+			$this->themer->setTheme($this->theme);
+
+			// Determine the correct layout to use
+			$layout = ! empty($this->layout) ? $this->layout : NULL;
+			$this->themer->setLayout($layout);
+
+			$this->themer->set($data);
+
+			// Render the view into a var we can pass to the layout.
+			$content = $this->themer->display($view .'.html.php');
+
+			$this->themer->set('content', $content);
+
+			$this->service->html_message( $this->themer->display($this->theme .':'. $layout) );
+		}
+
+		if (! $this->service->send() )
+		{
+			// todo do something here
+			return false;
+		}
+
+		return true;
+	}
+
+	//--------------------------------------------------------------------
+
+	/**
+	 * Allows you to customize the headers sent with the email. You can
+	 * do them one at a time by passing $field and $value, or pass an array
+	 * of $field => $value pairs as the first parameter.
+	 *
+	 * @param string|array  $field
+	 * @param string        $value
+	 */
+	public function header($field, $value=null)
+	{
+		$this->startMailService();
+
+		$this->service->setHeader($field, $value);
+	}
+
+	//--------------------------------------------------------------------
+
+	/**
+	 * Adds an attachment to the current email that is being built.
+	 *
+	 * @param string    $filename
+	 * @param string    $disposition    like 'inline'. Default is 'attachment'
+	 * @param string    $newname        If you'd like to rename the file for delivery
+	 * @param string    $mime           Custom defined mime type.
+	 */
+	public function attach($filename, $disposition=null, $newname=null, $mime=null)
+	{
+		$this->startMailService();
+
+		$this->service->attach($filename, $disposition, $newname, $mime);
+	}
+
+	//--------------------------------------------------------------------
+
+	//--------------------------------------------------------------------
+	// Private Methods
+	//--------------------------------------------------------------------
+
+	/**
+	 * Starts up the service name specified in $service_name.
+	 *
+	 * @param $service_name
+	 */
+	protected function startMailService()
+	{
+		// Only once!
+		if (! empty($this->service) && is_object($this->service))
+		{
+			return;
+		}
+
+		$service_name = ! empty($this->service_name) ? $this->service_name : config_item('mail.default_service');
+
+		if (! class_exists($service_name))
+		{
+			throw new \RuntimeException( sprintf( lang('mail.invalid_service'), $service_name) );
+		}
+
+		$this->service = new $service_name();
+	}
+
+	//--------------------------------------------------------------------
+
+	/**
+	 * Fires up the default themer so we can use it to theme our HTML messages.
+	 */
+	protected function startThemer()
+	{
+		/*
          * Setup our Template Engine
          */
-        $themer = config_item('active_themer');
-
-        if (empty($themer)) {
-            throw new \RuntimeException( lang('no_themer') );
-        }
-
-        if (empty($this->themer))
-        {
-            $this->themer = new $themer( get_instance() );
-        }
-
-        // Register our paths with the themer
-        $paths = config_item('theme.paths');
-
-        foreach ($paths as $key => $path) {
-            $this->themer->addThemePath($key, $path);
-        }
-
-        // Set our default theme.
-        $this->themer->setDefaultTheme( 'email' );
-    }
-
-    //--------------------------------------------------------------------
-
-    /**
-     * __get magic
-     *
-     * Allows models to access CI's loaded classes using the same
-     * syntax as controllers.
-     *
-     * @param	string	$key
-     */
-    public function __get($key)
-    {
-        return get_instance()->$key;
-    }
-
-    //--------------------------------------------------------------------
+		$themer = config_item('active_themer');
+
+		if (empty($themer)) {
+			throw new \RuntimeException( lang('no_themer') );
+		}
+
+		if (empty($this->themer))
+		{
+			$this->themer = new $themer( get_instance() );
+		}
+
+		// Register our paths with the themer
+		$paths = config_item('theme.paths');
+
+		foreach ($paths as $key => $path) {
+			$this->themer->addThemePath($key, $path);
+		}
+
+		// Set our default theme.
+		$this->themer->setDefaultTheme( 'email' );
+	}
+
+	//--------------------------------------------------------------------
+
+	/**
+	 * __get magic
+	 *
+	 * Allows models to access CI's loaded classes using the same
+	 * syntax as controllers.
+	 *
+	 * @param	string	$key
+	 */
+	public function __get($key)
+	{
+		return get_instance()->$key;
+	}
+
+	//--------------------------------------------------------------------
 
 }

Spacing +23 added lines, -23 removed lines

@@ -64,7 +64,7 @@ 
      * will use the system default.
      * @var null
      */
-    protected $service_name  = null;
+    protected $service_name = null;
 
     protected $service = null;
 
@@ -83,9 +83,9 @@ 
      *
      * @param null $options
      */
-    public function __construct($options=null)
+    public function __construct($options = null)
     {
-        if (! empty($options))
+        if ( ! empty($options))
         {
             $this->setOptions($options);
         }
@@ -107,7 +107,7 @@ 
             {
                 if ($key == 'service')
                 {
-                    $this->service =& $value;
+                    $this->service = & $value;
                     continue;
                 }
 
@@ -135,7 +135,7 @@ 
      *
      * @return bool
      */
-    public function send($to, $subject, $data=[], $view=null)
+    public function send($to, $subject, $data = [], $view = null)
     {
         // Are we pretending to send?
         if (config_item('mail.pretend') === true)
@@ -156,8 +156,8 @@ 
             $this->service->from($this->from);
         }
 
-        if (! empty($this->cc))         $this->service->cc($this->cc);
-        if (! empty($this->bcc))        $this->service->bcc($this->bcc);
+        if ( ! empty($this->cc))         $this->service->cc($this->cc);
+        if ( ! empty($this->bcc))        $this->service->bcc($this->bcc);
 
         if (is_array($this->reply_to)) {
             $this->service->reply_to($this->reply_to[0], $this->reply_to[1]);
@@ -172,20 +172,20 @@ 
         // the debug_backtrace, though, to make it all function in the background.
         list(, $method) = debug_backtrace(false);
 
-        $view = 'emails/'. strtolower( (new \ReflectionClass($this))->getShortName() ) .'/'. $method['function'];
+        $view = 'emails/'.strtolower((new \ReflectionClass($this))->getShortName()).'/'.$method['function'];
 
         // Get our message's text and html versions based on which files exist...
-        $basepath = APPPATH .'views/'. $view;
+        $basepath = APPPATH.'views/'.$view;
 
         // Is a text version available?
-        if (file_exists($basepath .'.text.php'))
+        if (file_exists($basepath.'.text.php'))
         {
-            $text = $this->load->view($view .'.text.php', $data, true);
+            $text = $this->load->view($view.'.text.php', $data, true);
             $this->service->text_message($text);
         }
 
         // If an html version is around, we need to theme it out
-        if (file_exists($basepath .'.html.php'))
+        if (file_exists($basepath.'.html.php'))
         {
             $this->startThemer();
 
@@ -198,14 +198,14 @@ 
             $this->themer->set($data);
 
             // Render the view into a var we can pass to the layout.
-            $content = $this->themer->display($view .'.html.php');
+            $content = $this->themer->display($view.'.html.php');
 
             $this->themer->set('content', $content);
 
-            $this->service->html_message( $this->themer->display($this->theme .':'. $layout) );
+            $this->service->html_message($this->themer->display($this->theme.':'.$layout));
         }
 
-        if (! $this->service->send() )
+        if ( ! $this->service->send())
         {
             // todo do something here
             return false;
@@ -224,7 +224,7 @@ 
      * @param string|array  $field
      * @param string        $value
      */
-    public function header($field, $value=null)
+    public function header($field, $value = null)
     {
         $this->startMailService();
 
@@ -241,7 +241,7 @@ 
      * @param string    $newname        If you'd like to rename the file for delivery
      * @param string    $mime           Custom defined mime type.
      */
-    public function attach($filename, $disposition=null, $newname=null, $mime=null)
+    public function attach($filename, $disposition = null, $newname = null, $mime = null)
     {
         $this->startMailService();
 
@@ -262,16 +262,16 @@ 
     protected function startMailService()
     {
         // Only once!
-        if (! empty($this->service) && is_object($this->service))
+        if ( ! empty($this->service) && is_object($this->service))
         {
             return;
         }
 
         $service_name = ! empty($this->service_name) ? $this->service_name : config_item('mail.default_service');
 
-        if (! class_exists($service_name))
+        if ( ! class_exists($service_name))
         {
-            throw new \RuntimeException( sprintf( lang('mail.invalid_service'), $service_name) );
+            throw new \RuntimeException(sprintf(lang('mail.invalid_service'), $service_name));
         }
 
         $this->service = new $service_name();
@@ -290,12 +290,12 @@ 
         $themer = config_item('active_themer');
 
         if (empty($themer)) {
-            throw new \RuntimeException( lang('no_themer') );
+            throw new \RuntimeException(lang('no_themer'));
         }
 
         if (empty($this->themer))
         {
-            $this->themer = new $themer( get_instance() );
+            $this->themer = new $themer(get_instance());
         }
 
         // Register our paths with the themer
@@ -306,7 +306,7 @@ 
         }
 
         // Set our default theme.
-        $this->themer->setDefaultTheme( 'email' );
+        $this->themer->setDefaultTheme('email');
     }
 
     //--------------------------------------------------------------------

Braces +8 added lines, -6 removed lines

@@ -150,19 +150,21 @@
 
         if (is_array($this->from)) {
             $this->service->from($this->from[0], $this->from[1]);
-        }
-        else
+        } else
         {
             $this->service->from($this->from);
         }
 
-        if (! empty($this->cc))         $this->service->cc($this->cc);
-        if (! empty($this->bcc))        $this->service->bcc($this->bcc);
+        if (! empty($this->cc)) {
+        	$this->service->cc($this->cc);
+        }
+        if (! empty($this->bcc)) {
+        	$this->service->bcc($this->bcc);
+        }
 
         if (is_array($this->reply_to)) {
             $this->service->reply_to($this->reply_to[0], $this->reply_to[1]);
-        }
-        else
+        } else
         {
             $this->service->reply_to($this->reply_to);
         }

myth/Mail/CIMailService.php

Indentation +222 added lines, -222 removed lines

@@ -40,226 +40,226 @@
  */
 class CIMailService implements MailServiceInterface {
 
-    protected $ci = null;
-
-    protected $format = 'html';
-
-    //--------------------------------------------------------------------
-
-    public function __construct()
-    {
-        $this->ci =& get_instance();
-
-        $this->ci->load->library('email');
-    }
-
-    //--------------------------------------------------------------------
-
-
-    /**
-     * Does the actual delivery of a message.
-     *
-     * @param bool  $clear_after    If TRUE, will reset the class after sending.
-     *
-     * @return mixed
-     */
-    public function send($clear_after=true)
-    {
-        $this->ci->email->mailtype = $this->format;
-
-        $result = $this->ci->email->send(false);
-
-        return $result;
-    }
-
-    //--------------------------------------------------------------------
-
-    /**
-     * Adds an attachment to the current email that is being built.
-     *
-     * @param string    $filename
-     * @param string    $disposition    like 'inline'. Default is 'attachment'
-     * @param string    $newname        If you'd like to rename the file for delivery
-     * @param string    $mime           Custom defined mime type.
-     */
-    public function attach($filename, $disposition=null, $newname=null, $mime=null)
-    {
-        $this->ci->email->attach($filename, $disposition, $newname, $mime);
-    }
-
-    //--------------------------------------------------------------------
-
-    /**
-     * Sets a header value for the email. Not every service will provide this.
-     *
-     * @param $field
-     * @param $value
-     * @return mixed
-     */
-    public function setHeader($field, $value)
-    {
-        $this->ci->email->set_header($field, $value);
-
-        return $this;
-    }
-
-    //--------------------------------------------------------------------
-
-    //--------------------------------------------------------------------
-    // Options
-    //--------------------------------------------------------------------
-
-    /**
-     * Sets the email address to send the email to.
-     *
-     * @param $email
-     * @return mixed
-     */
-    public function to($email)
-    {
-        $this->ci->email->to($email);
-
-        return $this;
-    }
-
-    //--------------------------------------------------------------------
-
-    /**
-     * Sets who the email is coming from.
-     *
-     * @param $email
-     * @param null $name
-     * @return mixed
-     */
-    public function from($email, $name=null)
-    {
-        $this->ci->email->from($email, $name);
-
-        return $this;
-    }
-
-    //--------------------------------------------------------------------
-
-    /**
-     * Sets a single additional email address to 'cc'.
-     *
-     * @param $email
-     * @return mixed
-     */
-    public function cc($email)
-    {
-        $this->ci->email->cc($email);
-
-        return $this;
-    }
-
-    //--------------------------------------------------------------------
-
-    /**
-     * Sets a single email address to 'bcc' to.
-     *
-     * @param $email
-     * @return mixed
-     */
-    public function bcc($email)
-    {
-        $this->ci->email->bcc($email);
-
-        return $this;
-    }
-
-    //--------------------------------------------------------------------
-
-    /**
-     * Sets the reply to address.
-     *
-     * @param $email
-     * @return mixed
-     */
-    public function reply_to($email, $name=null)
-    {
-        $this->ci->email->reply_to($email, $name);
-    }
-
-    //--------------------------------------------------------------------
-
-    /**
-     * Sets the subject line of the email.
-     *
-     * @param $subject
-     * @return mixed
-     */
-    public function subject($subject)
-    {
-        $this->ci->email->subject($subject);
-
-        return $this;
-    }
-
-    //--------------------------------------------------------------------
-
-    /**
-     * Sets the HTML portion of the email address. Optional.
-     *
-     * @param $message
-     * @return mixed
-     */
-    public function html_message($message)
-    {
-        $this->ci->email->message($message);
-
-        $this->format = 'html';
-
-        return $this;
-    }
-
-    //--------------------------------------------------------------------
-
-    /**
-     * Sets the text portion of the email address. Optional.
-     *
-     * @param $message
-     * @return mixed
-     */
-    public function text_message($message)
-    {
-        $this->ci->email->set_alt_message($message);
-
-        $this->format = 'text';
-
-        return $this;
-    }
-
-    //--------------------------------------------------------------------
-
-    /**
-     * Sets the format to send the email in. Either 'html' or 'text'.
-     *
-     * @param $format
-     * @return mixed
-     */
-    public function format($format)
-    {
-        $this->format = $format;
-
-        return $this;
-    }
-
-    //--------------------------------------------------------------------
-    /**
-     * Resets the state to blank, ready for a new email. Useful when
-     * sending emails in a loop and you need to make sure that the
-     * email is reset.
-     *
-     * @param bool $clear_attachments
-     * @return mixed
-     */
-    public function reset($clear_attachments=true)
-    {
-        $this->ci->email->clear($clear_attachments);
-
-        return $this;
-    }
-
-    //--------------------------------------------------------------------
+	protected $ci = null;
+
+	protected $format = 'html';
+
+	//--------------------------------------------------------------------
+
+	public function __construct()
+	{
+		$this->ci =& get_instance();
+
+		$this->ci->load->library('email');
+	}
+
+	//--------------------------------------------------------------------
+
+
+	/**
+	 * Does the actual delivery of a message.
+	 *
+	 * @param bool  $clear_after    If TRUE, will reset the class after sending.
+	 *
+	 * @return mixed
+	 */
+	public function send($clear_after=true)
+	{
+		$this->ci->email->mailtype = $this->format;
+
+		$result = $this->ci->email->send(false);
+
+		return $result;
+	}
+
+	//--------------------------------------------------------------------
+
+	/**
+	 * Adds an attachment to the current email that is being built.
+	 *
+	 * @param string    $filename
+	 * @param string    $disposition    like 'inline'. Default is 'attachment'
+	 * @param string    $newname        If you'd like to rename the file for delivery
+	 * @param string    $mime           Custom defined mime type.
+	 */
+	public function attach($filename, $disposition=null, $newname=null, $mime=null)
+	{
+		$this->ci->email->attach($filename, $disposition, $newname, $mime);
+	}
+
+	//--------------------------------------------------------------------
+
+	/**
+	 * Sets a header value for the email. Not every service will provide this.
+	 *
+	 * @param $field
+	 * @param $value
+	 * @return mixed
+	 */
+	public function setHeader($field, $value)
+	{
+		$this->ci->email->set_header($field, $value);
+
+		return $this;
+	}
+
+	//--------------------------------------------------------------------
+
+	//--------------------------------------------------------------------
+	// Options
+	//--------------------------------------------------------------------
+
+	/**
+	 * Sets the email address to send the email to.
+	 *
+	 * @param $email
+	 * @return mixed
+	 */
+	public function to($email)
+	{
+		$this->ci->email->to($email);
+
+		return $this;
+	}
+
+	//--------------------------------------------------------------------
+
+	/**
+	 * Sets who the email is coming from.
+	 *
+	 * @param $email
+	 * @param null $name
+	 * @return mixed
+	 */
+	public function from($email, $name=null)
+	{
+		$this->ci->email->from($email, $name);
+
+		return $this;
+	}
+
+	//--------------------------------------------------------------------
+
+	/**
+	 * Sets a single additional email address to 'cc'.
+	 *
+	 * @param $email
+	 * @return mixed
+	 */
+	public function cc($email)
+	{
+		$this->ci->email->cc($email);
+
+		return $this;
+	}
+
+	//--------------------------------------------------------------------
+
+	/**
+	 * Sets a single email address to 'bcc' to.
+	 *
+	 * @param $email
+	 * @return mixed
+	 */
+	public function bcc($email)
+	{
+		$this->ci->email->bcc($email);
+
+		return $this;
+	}
+
+	//--------------------------------------------------------------------
+
+	/**
+	 * Sets the reply to address.
+	 *
+	 * @param $email
+	 * @return mixed
+	 */
+	public function reply_to($email, $name=null)
+	{
+		$this->ci->email->reply_to($email, $name);
+	}
+
+	//--------------------------------------------------------------------
+
+	/**
+	 * Sets the subject line of the email.
+	 *
+	 * @param $subject
+	 * @return mixed
+	 */
+	public function subject($subject)
+	{
+		$this->ci->email->subject($subject);
+
+		return $this;
+	}
+
+	//--------------------------------------------------------------------
+
+	/**
+	 * Sets the HTML portion of the email address. Optional.
+	 *
+	 * @param $message
+	 * @return mixed
+	 */
+	public function html_message($message)
+	{
+		$this->ci->email->message($message);
+
+		$this->format = 'html';
+
+		return $this;
+	}
+
+	//--------------------------------------------------------------------
+
+	/**
+	 * Sets the text portion of the email address. Optional.
+	 *
+	 * @param $message
+	 * @return mixed
+	 */
+	public function text_message($message)
+	{
+		$this->ci->email->set_alt_message($message);
+
+		$this->format = 'text';
+
+		return $this;
+	}
+
+	//--------------------------------------------------------------------
+
+	/**
+	 * Sets the format to send the email in. Either 'html' or 'text'.
+	 *
+	 * @param $format
+	 * @return mixed
+	 */
+	public function format($format)
+	{
+		$this->format = $format;
+
+		return $this;
+	}
+
+	//--------------------------------------------------------------------
+	/**
+	 * Resets the state to blank, ready for a new email. Useful when
+	 * sending emails in a loop and you need to make sure that the
+	 * email is reset.
+	 *
+	 * @param bool $clear_attachments
+	 * @return mixed
+	 */
+	public function reset($clear_attachments=true)
+	{
+		$this->ci->email->clear($clear_attachments);
+
+		return $this;
+	}
+
+	//--------------------------------------------------------------------
 }

Spacing +6 added lines, -6 removed lines

@@ -48,7 +48,7 @@ 
 
     public function __construct()
     {
-        $this->ci =& get_instance();
+        $this->ci = & get_instance();
 
         $this->ci->load->library('email');
     }
@@ -63,7 +63,7 @@ 
      *
      * @return mixed
      */
-    public function send($clear_after=true)
+    public function send($clear_after = true)
     {
         $this->ci->email->mailtype = $this->format;
 
@@ -82,7 +82,7 @@ 
      * @param string    $newname        If you'd like to rename the file for delivery
      * @param string    $mime           Custom defined mime type.
      */
-    public function attach($filename, $disposition=null, $newname=null, $mime=null)
+    public function attach($filename, $disposition = null, $newname = null, $mime = null)
     {
         $this->ci->email->attach($filename, $disposition, $newname, $mime);
     }
@@ -131,7 +131,7 @@ 
      * @param null $name
      * @return mixed
      */
-    public function from($email, $name=null)
+    public function from($email, $name = null)
     {
         $this->ci->email->from($email, $name);
 
@@ -176,7 +176,7 @@ 
      * @param $email
      * @return mixed
      */
-    public function reply_to($email, $name=null)
+    public function reply_to($email, $name = null)
     {
         $this->ci->email->reply_to($email, $name);
     }
@@ -254,7 +254,7 @@ 
      * @param bool $clear_attachments
      * @return mixed
      */
-    public function reset($clear_attachments=true)
+    public function reset($clear_attachments = true)
     {
         $this->ci->email->clear($clear_attachments);
 

myth/Mail/MailServiceInterface.php

Indentation +140 added lines, -140 removed lines

@@ -42,145 +42,145 @@
  */
 interface MailServiceInterface {
 
-    /**
-     * Does the actual delivery of a message.
-     *
-     * @param bool  $clear_after    If TRUE, will reset the class after sending.
-     *
-     * @return mixed
-     */
-    public function send($clear_after=true);
-
-    //--------------------------------------------------------------------
-
-    /**
-     * Adds an attachment to the current email that is being built.
-     *
-     * @param string    $filename
-     * @param string    $disposition    like 'inline'. Default is 'attachment'
-     * @param string    $newname        If you'd like to rename the file for delivery
-     * @param string    $mime           Custom defined mime type.
-     */
-    public function attach($filename, $disposition=null, $newname=null, $mime=null);
-
-    //--------------------------------------------------------------------
-
-    /**
-     * Sets a header value for the email. Not every service will provide this.
-     *
-     * @param $field
-     * @param $value
-     * @return mixed
-     */
-    public function setHeader($field, $value);
-
-    //--------------------------------------------------------------------
-
-    //--------------------------------------------------------------------
-    // Options
-    //--------------------------------------------------------------------
-
-    /**
-     * Sets the email address to send the email to.
-     *
-     * @param $email
-     * @return mixed
-     */
-    public function to($email);
+	/**
+	 * Does the actual delivery of a message.
+	 *
+	 * @param bool  $clear_after    If TRUE, will reset the class after sending.
+	 *
+	 * @return mixed
+	 */
+	public function send($clear_after=true);
+
+	//--------------------------------------------------------------------
+
+	/**
+	 * Adds an attachment to the current email that is being built.
+	 *
+	 * @param string    $filename
+	 * @param string    $disposition    like 'inline'. Default is 'attachment'
+	 * @param string    $newname        If you'd like to rename the file for delivery
+	 * @param string    $mime           Custom defined mime type.
+	 */
+	public function attach($filename, $disposition=null, $newname=null, $mime=null);
+
+	//--------------------------------------------------------------------
+
+	/**
+	 * Sets a header value for the email. Not every service will provide this.
+	 *
+	 * @param $field
+	 * @param $value
+	 * @return mixed
+	 */
+	public function setHeader($field, $value);
+
+	//--------------------------------------------------------------------
+
+	//--------------------------------------------------------------------
+	// Options
+	//--------------------------------------------------------------------
+
+	/**
+	 * Sets the email address to send the email to.
+	 *
+	 * @param $email
+	 * @return mixed
+	 */
+	public function to($email);
     
-    //--------------------------------------------------------------------
-
-    /**
-     * Sets who the email is coming from.
-     *
-     * @param $email
-     * @param null $name
-     * @return mixed
-     */
-    public function from($email, $name=null);
-
-    //--------------------------------------------------------------------
-
-    /**
-     * Sets a single additional email address to 'cc'.
-     *
-     * @param $email
-     * @return mixed
-     */
-    public function cc($email);
-
-    //--------------------------------------------------------------------
-
-    /**
-     * Sets a single email address to 'bcc' to.
-     *
-     * @param $email
-     * @return mixed
-     */
-    public function bcc($email);
-
-    //--------------------------------------------------------------------
-
-    /**
-     * Sets the reply to address.
-     *
-     * @param $email
-     * @param $name
-     * @return mixed
-     */
-    public function reply_to($email, $name=null);
-
-    //--------------------------------------------------------------------
-
-    /**
-     * Sets the subject line of the email.
-     *
-     * @param $subject
-     * @return mixed
-     */
-    public function subject($subject);
-
-    //--------------------------------------------------------------------
-
-    /**
-     * Sets the HTML portion of the email address. Optional.
-     *
-     * @param $message
-     * @return mixed
-     */
-    public function html_message($message);
-
-    //--------------------------------------------------------------------
-
-    /**
-     * Sets the text portion of the email address. Optional.
-     *
-     * @param $message
-     * @return mixed
-     */
-    public function text_message($message);
-
-    //--------------------------------------------------------------------
-
-    /**
-     * Sets the format to send the email in. Either 'html' or 'text'.
-     *
-     * @param $format
-     * @return mixed
-     */
-    public function format($format);
-
-    //--------------------------------------------------------------------
-
-    /**
-     * Resets the state to blank, ready for a new email. Useful when
-     * sending emails in a loop and you need to make sure that the
-     * email is reset.
-     *
-     * @param bool $clear_attachments
-     * @return mixed
-     */
-    public function reset($clear_attachments=true);
-
-    //--------------------------------------------------------------------
+	//--------------------------------------------------------------------
+
+	/**
+	 * Sets who the email is coming from.
+	 *
+	 * @param $email
+	 * @param null $name
+	 * @return mixed
+	 */
+	public function from($email, $name=null);
+
+	//--------------------------------------------------------------------
+
+	/**
+	 * Sets a single additional email address to 'cc'.
+	 *
+	 * @param $email
+	 * @return mixed
+	 */
+	public function cc($email);
+
+	//--------------------------------------------------------------------
+
+	/**
+	 * Sets a single email address to 'bcc' to.
+	 *
+	 * @param $email
+	 * @return mixed
+	 */
+	public function bcc($email);
+
+	//--------------------------------------------------------------------
+
+	/**
+	 * Sets the reply to address.
+	 *
+	 * @param $email
+	 * @param $name
+	 * @return mixed
+	 */
+	public function reply_to($email, $name=null);
+
+	//--------------------------------------------------------------------
+
+	/**
+	 * Sets the subject line of the email.
+	 *
+	 * @param $subject
+	 * @return mixed
+	 */
+	public function subject($subject);
+
+	//--------------------------------------------------------------------
+
+	/**
+	 * Sets the HTML portion of the email address. Optional.
+	 *
+	 * @param $message
+	 * @return mixed
+	 */
+	public function html_message($message);
+
+	//--------------------------------------------------------------------
+
+	/**
+	 * Sets the text portion of the email address. Optional.
+	 *
+	 * @param $message
+	 * @return mixed
+	 */
+	public function text_message($message);
+
+	//--------------------------------------------------------------------
+
+	/**
+	 * Sets the format to send the email in. Either 'html' or 'text'.
+	 *
+	 * @param $format
+	 * @return mixed
+	 */
+	public function format($format);
+
+	//--------------------------------------------------------------------
+
+	/**
+	 * Resets the state to blank, ready for a new email. Useful when
+	 * sending emails in a loop and you need to make sure that the
+	 * email is reset.
+	 *
+	 * @param bool $clear_attachments
+	 * @return mixed
+	 */
+	public function reset($clear_attachments=true);
+
+	//--------------------------------------------------------------------
 }

Spacing +5 added lines, -5 removed lines

@@ -49,7 +49,7 @@ 
      *
      * @return mixed
      */
-    public function send($clear_after=true);
+    public function send($clear_after = true);
 
     //--------------------------------------------------------------------
 
@@ -61,7 +61,7 @@ 
      * @param string    $newname        If you'd like to rename the file for delivery
      * @param string    $mime           Custom defined mime type.
      */
-    public function attach($filename, $disposition=null, $newname=null, $mime=null);
+    public function attach($filename, $disposition = null, $newname = null, $mime = null);
 
     //--------------------------------------------------------------------
 
@@ -97,7 +97,7 @@ 
      * @param null $name
      * @return mixed
      */
-    public function from($email, $name=null);
+    public function from($email, $name = null);
 
     //--------------------------------------------------------------------
 
@@ -128,7 +128,7 @@ 
      * @param $name
      * @return mixed
      */
-    public function reply_to($email, $name=null);
+    public function reply_to($email, $name = null);
 
     //--------------------------------------------------------------------
 
@@ -180,7 +180,7 @@ 
      * @param bool $clear_attachments
      * @return mixed
      */
-    public function reset($clear_attachments=true);
+    public function reset($clear_attachments = true);
 
     //--------------------------------------------------------------------
 }

myth/Mail/Queue.php

Indentation +32 added lines, -32 removed lines

@@ -1,41 +1,41 @@
 <?php namespace Myth\Mail;
 /**
- * Sprint
- *
- * A set of power tools to enhance the CodeIgniter framework and provide consistent workflow.
- *
- * Permission is hereby granted, free of charge, to any person obtaining a copy
- * of this software and associated documentation files (the "Software"), to deal
- * in the Software without restriction, including without limitation the rights
- * to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
- * copies of the Software, and to permit persons to whom the Software is
- * furnished to do so, subject to the following conditions:
- *
- * The above copyright notice and this permission notice shall be included in
- * all copies or substantial portions of the Software.
- *
- * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
- * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
- * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
- * AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
- * LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
- * OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN
- * THE SOFTWARE.
- *
- * @package     Sprint
- * @author      Lonnie Ezell
- * @copyright   Copyright 2014-2015, New Myth Media, LLC (http://newmythmedia.com)
- * @license     http://opensource.org/licenses/MIT  (MIT)
- * @link        http://sprintphp.com
- * @since       Version 1.0
- */
+	 * Sprint
+	 *
+	 * A set of power tools to enhance the CodeIgniter framework and provide consistent workflow.
+	 *
+	 * Permission is hereby granted, free of charge, to any person obtaining a copy
+	 * of this software and associated documentation files (the "Software"), to deal
+	 * in the Software without restriction, including without limitation the rights
+	 * to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
+	 * copies of the Software, and to permit persons to whom the Software is
+	 * furnished to do so, subject to the following conditions:
+	 *
+	 * The above copyright notice and this permission notice shall be included in
+	 * all copies or substantial portions of the Software.
+	 *
+	 * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
+	 * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
+	 * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
+	 * AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
+	 * LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
+	 * OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN
+	 * THE SOFTWARE.
+	 *
+	 * @package     Sprint
+	 * @author      Lonnie Ezell
+	 * @copyright   Copyright 2014-2015, New Myth Media, LLC (http://newmythmedia.com)
+	 * @license     http://opensource.org/licenses/MIT  (MIT)
+	 * @link        http://sprintphp.com
+	 * @since       Version 1.0
+	 */
 
 use Myth\Models\CIDbModel;
 
 class Queue extends CIDbModel {
 
-    protected $table_name = 'mail_queue';
+	protected $table_name = 'mail_queue';
 
-    protected $set_created = false;
-    protected $set_modified = false;
+	protected $set_created = false;
+	protected $set_modified = false;
 }

myth/Settings/ConfigStore.php

Indentation +138 added lines, -138 removed lines

@@ -49,143 +49,143 @@
  */
 class ConfigStore implements SettingsStoreInterface {
 
-    protected $ci;
-
-    //--------------------------------------------------------------------
-
-    public function __construct( $ci=null )
-    {
-        if (is_object($ci))
-        {
-            $this->ci =& $ci;
-        }
-        else {
-            $this->ci =& get_instance();
-        }
-    }
-
-    //--------------------------------------------------------------------
-
-    /**
-     * Inserts/Replaces a single setting item.
-     *
-     * @param $key
-     * @param null $value
-     * @param string $group
-     */
-    public function save($key, $value=null, $group='app')
-    {
-        // Treat the 'app' group as our primary, un-sectioned
-        // config files.
-        if ($group != 'app')
-        {
-            return $this->ci->config->set_item($key, $value);
-        }
-
-        // Otherwise, we'll need to ensure a section exists
-        if (! isset($this->ci->config->config[$group]))
-        {
-            $this->ci->config->config[$group] = [];
-        }
-
-        $this->ci->config->config[$group][$key] = $value;
-    }
-
-    //--------------------------------------------------------------------
-
-    /**
-     * Retrieves a single item. If the config system doesn't have the
-     * value loaded, yet, it will attempt to load a config file matching
-     * the 'group' name and grab the value from that.
-     *
-     * @param $key
-     * @param string $group
-     * @return mixed
-     */
-    public function get($key, $group='application')
-    {
-        // First, see if CI already has this key loaded.
-        $result = $this->ci->config->item($key);
-
-        // This bit will fail when the actual value is NULL
-        // but should result in false hits infrequently.
-        if ($result !== null)
-        {
-            return $result;
-        }
-
-        // Try to load the 'group' file, then try to load a
-        // config file that matches the group name
-        $this->ci->load->config($group, false, true);
+	protected $ci;
+
+	//--------------------------------------------------------------------
+
+	public function __construct( $ci=null )
+	{
+		if (is_object($ci))
+		{
+			$this->ci =& $ci;
+		}
+		else {
+			$this->ci =& get_instance();
+		}
+	}
+
+	//--------------------------------------------------------------------
+
+	/**
+	 * Inserts/Replaces a single setting item.
+	 *
+	 * @param $key
+	 * @param null $value
+	 * @param string $group
+	 */
+	public function save($key, $value=null, $group='app')
+	{
+		// Treat the 'app' group as our primary, un-sectioned
+		// config files.
+		if ($group != 'app')
+		{
+			return $this->ci->config->set_item($key, $value);
+		}
+
+		// Otherwise, we'll need to ensure a section exists
+		if (! isset($this->ci->config->config[$group]))
+		{
+			$this->ci->config->config[$group] = [];
+		}
+
+		$this->ci->config->config[$group][$key] = $value;
+	}
+
+	//--------------------------------------------------------------------
+
+	/**
+	 * Retrieves a single item. If the config system doesn't have the
+	 * value loaded, yet, it will attempt to load a config file matching
+	 * the 'group' name and grab the value from that.
+	 *
+	 * @param $key
+	 * @param string $group
+	 * @return mixed
+	 */
+	public function get($key, $group='application')
+	{
+		// First, see if CI already has this key loaded.
+		$result = $this->ci->config->item($key);
+
+		// This bit will fail when the actual value is NULL
+		// but should result in false hits infrequently.
+		if ($result !== null)
+		{
+			return $result;
+		}
+
+		// Try to load the 'group' file, then try to load a
+		// config file that matches the group name
+		$this->ci->load->config($group, false, true);
         
-        $result = $this->ci->config->item($key);
-
-        return $result;
-    }
-
-    //--------------------------------------------------------------------
-
-    /**
-     * Deletes a single item.
-     *
-     * NOT supported by this store.
-     *
-     * @param $key
-     * @param $group
-     * @return mixed
-     */
-    public function delete($key, $group='app')
-    {
-        return false;
-    }
-
-    //--------------------------------------------------------------------
-
-    /**
-     * Searches the store for any items with $field = $value.
-     *
-     * Does not work.
-     *
-     * @param $field
-     * @param $value
-     * @return mixed
-     */
-    public function findBy($field, $value)
-    {
-        return false;
-    }
-
-    //--------------------------------------------------------------------
-
-    /**
-     * Retrieves all items in the store either globally or for a single group.
-     *
-     * @param string $group
-     * @return mixed
-     */
-    public function all($group=null)
-    {
-        if (empty($group))
-        {
-            return $this->ci->config->config;
-        }
-
-        // If we're a group, does the group already exists
-        // as a 'section' in the config array?
-        if (isset($this->ci->config->config[$group]))
-        {
-            return $this->ci->config->config[$group];
-        }
-
-        // Still here? Attempt to load the file into a section
-        // and try one last time.
-        if ($this->ci->load->config($group))
-        {
-            return $this->ci->config->config($group);
-        }
-
-        return null;
-    }
-
-    //--------------------------------------------------------------------
+		$result = $this->ci->config->item($key);
+
+		return $result;
+	}
+
+	//--------------------------------------------------------------------
+
+	/**
+	 * Deletes a single item.
+	 *
+	 * NOT supported by this store.
+	 *
+	 * @param $key
+	 * @param $group
+	 * @return mixed
+	 */
+	public function delete($key, $group='app')
+	{
+		return false;
+	}
+
+	//--------------------------------------------------------------------
+
+	/**
+	 * Searches the store for any items with $field = $value.
+	 *
+	 * Does not work.
+	 *
+	 * @param $field
+	 * @param $value
+	 * @return mixed
+	 */
+	public function findBy($field, $value)
+	{
+		return false;
+	}
+
+	//--------------------------------------------------------------------
+
+	/**
+	 * Retrieves all items in the store either globally or for a single group.
+	 *
+	 * @param string $group
+	 * @return mixed
+	 */
+	public function all($group=null)
+	{
+		if (empty($group))
+		{
+			return $this->ci->config->config;
+		}
+
+		// If we're a group, does the group already exists
+		// as a 'section' in the config array?
+		if (isset($this->ci->config->config[$group]))
+		{
+			return $this->ci->config->config[$group];
+		}
+
+		// Still here? Attempt to load the file into a section
+		// and try one last time.
+		if ($this->ci->load->config($group))
+		{
+			return $this->ci->config->config($group);
+		}
+
+		return null;
+	}
+
+	//--------------------------------------------------------------------
 }

Spacing +8 added lines, -8 removed lines

@@ -53,14 +53,14 @@ 
 
     //--------------------------------------------------------------------
 
-    public function __construct( $ci=null )
+    public function __construct($ci = null)
     {
         if (is_object($ci))
         {
-            $this->ci =& $ci;
+            $this->ci = & $ci;
         }
         else {
-            $this->ci =& get_instance();
+            $this->ci = & get_instance();
         }
     }
 
@@ -73,7 +73,7 @@ 
      * @param null $value
      * @param string $group
      */
-    public function save($key, $value=null, $group='app')
+    public function save($key, $value = null, $group = 'app')
     {
         // Treat the 'app' group as our primary, un-sectioned
         // config files.
@@ -83,7 +83,7 @@ 
         }
 
         // Otherwise, we'll need to ensure a section exists
-        if (! isset($this->ci->config->config[$group]))
+        if ( ! isset($this->ci->config->config[$group]))
         {
             $this->ci->config->config[$group] = [];
         }
@@ -102,7 +102,7 @@ 
      * @param string $group
      * @return mixed
      */
-    public function get($key, $group='application')
+    public function get($key, $group = 'application')
     {
         // First, see if CI already has this key loaded.
         $result = $this->ci->config->item($key);
@@ -134,7 +134,7 @@ 
      * @param $group
      * @return mixed
      */
-    public function delete($key, $group='app')
+    public function delete($key, $group = 'app')
     {
         return false;
     }
@@ -163,7 +163,7 @@ 
      * @param string $group
      * @return mixed
      */
-    public function all($group=null)
+    public function all($group = null)
     {
         if (empty($group))
         {

Braces +1 added lines, -2 removed lines

@@ -43,8 +43,7 @@
         if (is_object($ci))
         {
             $this->ci =& $ci;
-        }
-        else {
+        } else {
             $this->ci =& get_instance();
         }