MY_Loader

Complexity

Total Complexity

3

Size/Duplication

Total Lines	29
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	1

Test Coverage

Coverage

85.71%

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	3	1
A	database()	0	11	2

<?php

class MY_Loader extends CI_Loader {

	public function __construct() {
		parent::__construct();
	}

	/**
	 * Database Loader extension, to load our version of the caching engine
	 *
	 * @access   public
	 *
	 * @param    string  the DB credentials
	 * @param    bool    whether to return the DB object
	 * @param    bool    whether to enable active record (this allows us to override the config setting)
	 *
	 * @return   object
	 */
	function database($params = '', $return = FALSE, $active_record = NULL) {

It is generally recommended to explicitly declare the visibility for methods.

		// load our version of the CI_DB_Cache class. The database library checks
		// if this class is already loaded before instantiating it. Loading it now
		// makes sure our version is used when a controller enables query caching
		if(!class_exists('CI_DB_Cache')) {
			@include(APPPATH . 'libraries/MY_DB_cache.php');

It seems like you do not handle an error condition here. This can introduce security issues, and is generally not recommended.

		}

		// call the parent method to retain the CI functionality
		return parent::database($params, $return, $active_record);
	}
}