@@ -69,7 +69,7 @@ discard block |
||
| 69 | 69 | { |
| 70 | 70 | if (in_array($class, $this->alias)) |
| 71 | 71 | { |
| 72 | - $dir = __DIR__ . '/alias'; |
|
| 72 | + $dir = __DIR__.'/alias'; |
|
| 73 | 73 | $this->loadClassFile($dir, $class); |
| 74 | 74 | } |
| 75 | 75 | } |
@@ -88,7 +88,7 @@ discard block |
||
| 88 | 88 | } |
| 89 | 89 | else |
| 90 | 90 | { |
| 91 | - $dir = __DIR__ . '/exceptions'; |
|
| 91 | + $dir = __DIR__.'/exceptions'; |
|
| 92 | 92 | $this->loadClassFile($dir, $class); |
| 93 | 93 | } |
| 94 | 94 | } |
@@ -129,7 +129,7 @@ discard block |
||
| 129 | 129 | |
| 130 | 130 | protected function loadClassFile($dir, $class) |
| 131 | 131 | { |
| 132 | - $class_file = $dir . '/' . $class . '.php'; |
|
| 132 | + $class_file = $dir.'/'.$class.'.php'; |
|
| 133 | 133 | if (file_exists($class_file)) |
| 134 | 134 | { |
| 135 | 135 | require $class_file; |
@@ -44,8 +44,7 @@ discard block |
||
| 44 | 44 | APPPATH.'modules', |
| 45 | 45 | APPPATH.'hooks', |
| 46 | 46 | ]; |
| 47 | - } |
|
| 48 | - else |
|
| 47 | + } else |
|
| 49 | 48 | { |
| 50 | 49 | $this->dirs = $dirs; |
| 51 | 50 | } |
@@ -86,8 +85,7 @@ discard block |
||
| 86 | 85 | { |
| 87 | 86 | $dir = __DIR__; |
| 88 | 87 | $this->loadClassFile($dir, $class); |
| 89 | - } |
|
| 90 | - else |
|
| 88 | + } else |
|
| 91 | 89 | { |
| 92 | 90 | $dir = __DIR__ . '/exceptions'; |
| 93 | 91 | $this->loadClassFile($dir, $class); |
@@ -152,8 +150,7 @@ discard block |
||
| 152 | 150 | { |
| 153 | 151 | require $filename; |
| 154 | 152 | return true; |
| 155 | - } |
|
| 156 | - else |
|
| 153 | + } else |
|
| 157 | 154 | { |
| 158 | 155 | unset($this->cache[$class]); |
| 159 | 156 | } |
@@ -24,18 +24,18 @@ |
||
| 24 | 24 | } |
| 25 | 25 | |
| 26 | 26 | $dir = dirname($this->file); |
| 27 | - if (! is_dir($dir)) |
|
| 27 | + if ( ! is_dir($dir)) |
|
| 28 | 28 | { |
| 29 | 29 | if (@mkdir($dir, 0777, true) === false) |
| 30 | 30 | { |
| 31 | - throw new RuntimeException('Failed to create folder: ' . $dir); |
|
| 31 | + throw new RuntimeException('Failed to create folder: '.$dir); |
|
| 32 | 32 | } |
| 33 | 33 | } |
| 34 | 34 | |
| 35 | 35 | if (file_put_contents($this->file, '') === false) |
| 36 | 36 | { |
| 37 | 37 | throw new RuntimeException( |
| 38 | - 'Failed to write to cache file: ' . $this->file |
|
| 38 | + 'Failed to write to cache file: '.$this->file |
|
| 39 | 39 | ); |
| 40 | 40 | } |
| 41 | 41 | } |
@@ -67,8 +67,7 @@ |
||
| 67 | 67 | if ($this->offsetExists($key)) |
| 68 | 68 | { |
| 69 | 69 | return $this->map[$key]; |
| 70 | - } |
|
| 71 | - else |
|
| 70 | + } else |
|
| 72 | 71 | { |
| 73 | 72 | return null; |
| 74 | 73 | } |
@@ -21,7 +21,7 @@ |
||
| 21 | 21 | $ref_method->setAccessible(true); |
| 22 | 22 | $obj = (gettype($obj) === 'object') ? $obj : null; |
| 23 | 23 | |
| 24 | - return function () use ($obj, $ref_method) { |
|
| 24 | + return function() use ($obj, $ref_method) { |
|
| 25 | 25 | $args = func_get_args(); |
| 26 | 26 | return $ref_method->invokeArgs($obj, $args); |
| 27 | 27 | }; |
@@ -129,8 +129,7 @@ discard block |
||
| 129 | 129 | return $this->callControllerMethod( |
| 130 | 130 | $http_method, $argv, $params |
| 131 | 131 | ); |
| 132 | - } |
|
| 133 | - else |
|
| 132 | + } else |
|
| 134 | 133 | { |
| 135 | 134 | return $this->requestUri($http_method, $argv, $params); |
| 136 | 135 | } |
@@ -141,8 +140,7 @@ discard block |
||
| 141 | 140 | if ($e->getCode() === 0) |
| 142 | 141 | { |
| 143 | 142 | set_status_header(200); |
| 144 | - } |
|
| 145 | - else |
|
| 143 | + } else |
|
| 146 | 144 | { |
| 147 | 145 | set_status_header($e->getCode()); |
| 148 | 146 | } |
@@ -116,7 +116,7 @@ discard block |
||
| 116 | 116 | public function enableHooks() |
| 117 | 117 | { |
| 118 | 118 | $this->enableHooks = true; |
| 119 | - $this->hooks =& load_class('Hooks', 'core'); |
|
| 119 | + $this->hooks = & load_class('Hooks', 'core'); |
|
| 120 | 120 | } |
| 121 | 121 | |
| 122 | 122 | /** |
@@ -162,7 +162,7 @@ discard block |
||
| 162 | 162 | { |
| 163 | 163 | set_status_header($e->getCode()); |
| 164 | 164 | } |
| 165 | - $CI =& get_instance(); |
|
| 165 | + $CI = & get_instance(); |
|
| 166 | 166 | $CI->output->_status['redirect'] = $e->getMessage(); |
| 167 | 167 | } |
| 168 | 168 | // show_404() |
@@ -224,17 +224,17 @@ discard block |
||
| 224 | 224 | $this->setRawInputStream($request_params); |
| 225 | 225 | |
| 226 | 226 | // 404 checking |
| 227 | - if (! class_exists($class) || ! method_exists($class, $method)) |
|
| 227 | + if ( ! class_exists($class) || ! method_exists($class, $method)) |
|
| 228 | 228 | { |
| 229 | 229 | // If 404, CodeIgniter instance is not created yet. So create it here |
| 230 | 230 | // Because we need CI->output->_status to store info |
| 231 | - $CI =& get_instance(); |
|
| 231 | + $CI = & get_instance(); |
|
| 232 | 232 | if ($CI instanceof CIPHPUnitTestNullCodeIgniter) |
| 233 | 233 | { |
| 234 | 234 | CIPHPUnitTest::createCodeIgniterInstance(); |
| 235 | 235 | } |
| 236 | 236 | |
| 237 | - show_404($class.'::'.$method . '() is not found'); |
|
| 237 | + show_404($class.'::'.$method.'() is not found'); |
|
| 238 | 238 | } |
| 239 | 239 | |
| 240 | 240 | $params = $argv; |
@@ -296,7 +296,7 @@ discard block |
||
| 296 | 296 | { |
| 297 | 297 | if (is_string($string)) |
| 298 | 298 | { |
| 299 | - $INPUT =& load_class('Input', 'core'); |
|
| 299 | + $INPUT = & load_class('Input', 'core'); |
|
| 300 | 300 | CIPHPUnitTestReflection::setPrivateProperty( |
| 301 | 301 | $INPUT, |
| 302 | 302 | '_raw_input_stream', |
@@ -326,12 +326,12 @@ discard block |
||
| 326 | 326 | new CI(); |
| 327 | 327 | } |
| 328 | 328 | $controller = new $class; |
| 329 | - $CI =& get_instance(); |
|
| 329 | + $CI = & get_instance(); |
|
| 330 | 330 | |
| 331 | 331 | // Set CodeIgniter instance to TestCase |
| 332 | 332 | $this->testCase->setCI($CI); |
| 333 | 333 | |
| 334 | - if (! isset($CI->output->_status)) |
|
| 334 | + if ( ! isset($CI->output->_status)) |
|
| 335 | 335 | { |
| 336 | 336 | // Prevent overwriting, if already set in the $class::__construct() |
| 337 | 337 | // Set default response code 200 |
@@ -377,8 +377,8 @@ discard block |
||
| 377 | 377 | */ |
| 378 | 378 | public function getStatus() |
| 379 | 379 | { |
| 380 | - $CI =& get_instance(); |
|
| 381 | - if (! isset($CI->output->_status)) |
|
| 380 | + $CI = & get_instance(); |
|
| 381 | + if ( ! isset($CI->output->_status)) |
|
| 382 | 382 | { |
| 383 | 383 | throw new LogicException('Status code is not set. You must call $this->request() first'); |
| 384 | 384 | } |
@@ -29,16 +29,14 @@ |
||
| 29 | 29 | if (empty($class) OR ! file_exists(APPPATH.'controllers/'.$RTR->directory.$class.'.php')) |
| 30 | 30 | { |
| 31 | 31 | $e404 = TRUE; |
| 32 | - } |
|
| 33 | - else |
|
| 32 | + } else |
|
| 34 | 33 | { |
| 35 | 34 | require_once(APPPATH.'controllers/'.$RTR->directory.$class.'.php'); |
| 36 | 35 | |
| 37 | 36 | if ( ! class_exists($class, FALSE) OR $method[0] === '_' OR method_exists('CI_Controller', $method)) |
| 38 | 37 | { |
| 39 | 38 | $e404 = TRUE; |
| 40 | - } |
|
| 41 | - elseif (method_exists($class, '_remap')) |
|
| 39 | + } elseif (method_exists($class, '_remap')) |
|
| 42 | 40 | { |
| 43 | 41 | $params = array($method, array_slice($URI->rsegments, 2)); |
| 44 | 42 | $method = '_remap'; |
@@ -19,8 +19,8 @@ discard block |
||
| 19 | 19 | */ |
| 20 | 20 | public function getRoute() |
| 21 | 21 | { |
| 22 | - $RTR =& load_class('Router', 'core'); |
|
| 23 | - $URI =& load_class('URI', 'core'); |
|
| 22 | + $RTR = & load_class('Router', 'core'); |
|
| 23 | + $URI = & load_class('URI', 'core'); |
|
| 24 | 24 | |
| 25 | 25 | $e404 = FALSE; |
| 26 | 26 | $class = ucfirst($RTR->class); |
@@ -57,7 +57,7 @@ discard block |
||
| 57 | 57 | { |
| 58 | 58 | // If 404, CodeIgniter instance is not created yet. So create it here. |
| 59 | 59 | // Because we need CI->output->_status |
| 60 | - $CI =& get_instance(); |
|
| 60 | + $CI = & get_instance(); |
|
| 61 | 61 | if ($CI instanceof CIPHPUnitTestNullCodeIgniter) |
| 62 | 62 | { |
| 63 | 63 | CIPHPUnitTest::createCodeIgniterInstance(); |
@@ -39,8 +39,7 @@ |
||
| 39 | 39 | { |
| 40 | 40 | $CI->db->close(); |
| 41 | 41 | $CI->db = null; |
| 42 | - } |
|
| 43 | - else |
|
| 42 | + } else |
|
| 44 | 43 | { |
| 45 | 44 | // Don't close if SQLite in-memory database |
| 46 | 45 | // If we close it, all tables and stored data will be gone |
@@ -32,7 +32,7 @@ discard block |
||
| 32 | 32 | reset_config(); |
| 33 | 33 | |
| 34 | 34 | // Close db connection |
| 35 | - $CI =& get_instance(); |
|
| 35 | + $CI = & get_instance(); |
|
| 36 | 36 | if (isset($CI->db)) |
| 37 | 37 | { |
| 38 | 38 | if ( |
@@ -52,25 +52,25 @@ discard block |
||
| 52 | 52 | } |
| 53 | 53 | |
| 54 | 54 | // Load core classes |
| 55 | - $BM =& load_class('Benchmark', 'core'); |
|
| 55 | + $BM = & load_class('Benchmark', 'core'); |
|
| 56 | 56 | CIPHPUnitTestSuperGlobal::set_Global('BM', $BM); |
| 57 | - $EXT =& load_class('Hooks', 'core'); |
|
| 57 | + $EXT = & load_class('Hooks', 'core'); |
|
| 58 | 58 | CIPHPUnitTestSuperGlobal::set_Global('EXT', $EXT); |
| 59 | - $CFG =& load_class('Config', 'core'); |
|
| 59 | + $CFG = & load_class('Config', 'core'); |
|
| 60 | 60 | CIPHPUnitTestSuperGlobal::set_Global('CFG', $CFG); |
| 61 | - $UNI =& load_class('URI', 'core'); |
|
| 61 | + $UNI = & load_class('URI', 'core'); |
|
| 62 | 62 | CIPHPUnitTestSuperGlobal::set_Global('UNI', $UNI); |
| 63 | 63 | // $URI =& load_class('Utf8', 'core'); |
| 64 | 64 | // CIPHPUnitTestSuperGlobal::set_Global('URI', $URI); |
| 65 | - $RTR =& load_class('Router', 'core'); |
|
| 65 | + $RTR = & load_class('Router', 'core'); |
|
| 66 | 66 | CIPHPUnitTestSuperGlobal::set_Global('RTR', $RTR); |
| 67 | - $OUT =& load_class('Output', 'core'); |
|
| 67 | + $OUT = & load_class('Output', 'core'); |
|
| 68 | 68 | CIPHPUnitTestSuperGlobal::set_Global('OUT', $OUT); |
| 69 | - $SEC =& load_class('Security', 'core'); |
|
| 69 | + $SEC = & load_class('Security', 'core'); |
|
| 70 | 70 | CIPHPUnitTestSuperGlobal::set_Global('SEC', $SEC); |
| 71 | - $IN =& load_class('Input', 'core'); |
|
| 71 | + $IN = & load_class('Input', 'core'); |
|
| 72 | 72 | CIPHPUnitTestSuperGlobal::set_Global('IN', $IN); |
| 73 | - $LANG =& load_class('Lang', 'core'); |
|
| 73 | + $LANG = & load_class('Lang', 'core'); |
|
| 74 | 74 | CIPHPUnitTestSuperGlobal::set_Global('LANG', $LANG); |
| 75 | 75 | |
| 76 | 76 | CIPHPUnitTest::loadLoader(); |
@@ -45,17 +45,17 @@ discard block |
||
| 45 | 45 | throw new LogicException("No such directory: $dir"); |
| 46 | 46 | } |
| 47 | 47 | |
| 48 | - self::$src_cache_dir = self::$cache_dir . '/src'; |
|
| 48 | + self::$src_cache_dir = self::$cache_dir.'/src'; |
|
| 49 | 49 | self::$tmp_function_whitelist_file = |
| 50 | - self::$cache_dir . '/conf/func_whiltelist.php'; |
|
| 50 | + self::$cache_dir.'/conf/func_whiltelist.php'; |
|
| 51 | 51 | self::$tmp_function_blacklist_file = |
| 52 | - self::$cache_dir . '/conf/func_blacklist.php'; |
|
| 52 | + self::$cache_dir.'/conf/func_blacklist.php'; |
|
| 53 | 53 | self::$tmp_patcher_list_file = |
| 54 | - self::$cache_dir . '/conf/patcher_list.php'; |
|
| 54 | + self::$cache_dir.'/conf/patcher_list.php'; |
|
| 55 | 55 | self::$tmp_include_paths_file = |
| 56 | - self::$cache_dir . '/conf/include_paths.php'; |
|
| 56 | + self::$cache_dir.'/conf/include_paths.php'; |
|
| 57 | 57 | self::$tmp_exclude_paths_file = |
| 58 | - self::$cache_dir . '/conf/exclude_paths.php'; |
|
| 58 | + self::$cache_dir.'/conf/exclude_paths.php'; |
|
| 59 | 59 | } |
| 60 | 60 | |
| 61 | 61 | public static function getCacheDir() |
@@ -73,16 +73,16 @@ discard block |
||
| 73 | 73 | return false; |
| 74 | 74 | } |
| 75 | 75 | |
| 76 | - return self::$src_cache_dir . '/' . $relative_path; |
|
| 76 | + return self::$src_cache_dir.'/'.$relative_path; |
|
| 77 | 77 | } |
| 78 | 78 | |
| 79 | 79 | protected static function createDir($dir) |
| 80 | 80 | { |
| 81 | - if (! is_dir($dir)) |
|
| 81 | + if ( ! is_dir($dir)) |
|
| 82 | 82 | { |
| 83 | - if (! @mkdir($dir, 0777, true)) |
|
| 83 | + if ( ! @mkdir($dir, 0777, true)) |
|
| 84 | 84 | { |
| 85 | - throw new RuntimeException('Failed to create folder: ' . $dir); |
|
| 85 | + throw new RuntimeException('Failed to create folder: '.$dir); |
|
| 86 | 86 | } |
| 87 | 87 | } |
| 88 | 88 | } |
@@ -154,7 +154,7 @@ discard block |
||
| 154 | 154 | public static function appendTmpFunctionBlacklist($function) |
| 155 | 155 | { |
| 156 | 156 | file_put_contents( |
| 157 | - self::getTmpFunctionBlacklistFile(), $function . "\n", FILE_APPEND |
|
| 157 | + self::getTmpFunctionBlacklistFile(), $function."\n", FILE_APPEND |
|
| 158 | 158 | ); |
| 159 | 159 | } |
| 160 | 160 | |
@@ -234,20 +234,20 @@ discard block |
||
| 234 | 234 | { |
| 235 | 235 | $cache = self::getSrcCacheFilePath($orig_file); |
| 236 | 236 | @unlink($cache); |
| 237 | - MonkeyPatchManager::log('remove_src_cache: ' . $cache); |
|
| 237 | + MonkeyPatchManager::log('remove_src_cache: '.$cache); |
|
| 238 | 238 | return $cache; |
| 239 | 239 | } |
| 240 | 240 | |
| 241 | 241 | public static function clearSrcCache() |
| 242 | 242 | { |
| 243 | 243 | self::recursiveUnlink(self::$src_cache_dir); |
| 244 | - MonkeyPatchManager::log('clear_src_cache: cleared ' . self::$src_cache_dir); |
|
| 244 | + MonkeyPatchManager::log('clear_src_cache: cleared '.self::$src_cache_dir); |
|
| 245 | 245 | } |
| 246 | 246 | |
| 247 | 247 | public static function clearCache() |
| 248 | 248 | { |
| 249 | 249 | self::recursiveUnlink(self::$cache_dir); |
| 250 | - MonkeyPatchManager::log('clear_cache: cleared ' . self::$cache_dir); |
|
| 250 | + MonkeyPatchManager::log('clear_cache: cleared '.self::$cache_dir); |
|
| 251 | 251 | } |
| 252 | 252 | |
| 253 | 253 | /** |
@@ -257,7 +257,7 @@ discard block |
||
| 257 | 257 | */ |
| 258 | 258 | protected static function recursiveUnlink($dir) |
| 259 | 259 | { |
| 260 | - if (! is_dir($dir)) |
|
| 260 | + if ( ! is_dir($dir)) |
|
| 261 | 261 | { |
| 262 | 262 | return; |
| 263 | 263 | } |
@@ -59,7 +59,7 @@ |
||
| 59 | 59 | { |
| 60 | 60 | $this->unwrap(); |
| 61 | 61 | |
| 62 | - MonkeyPatchManager::log('stream_open: ' . $path); |
|
| 62 | + MonkeyPatchManager::log('stream_open: '.$path); |
|
| 63 | 63 | |
| 64 | 64 | $including = (bool) ($options & self::STREAM_OPEN_FOR_INCLUDE); |
| 65 | 65 | if ($including && $this->shouldPreprocess($path)) { |
@@ -54,7 +54,7 @@ discard block |
||
| 54 | 54 | PHPUnit_Framework_TestCase::assertEquals( |
| 55 | 55 | $expected_times, |
| 56 | 56 | $actual_times, |
| 57 | - $class_method . '() expected to be not invoked, but invoked ' . $actual_times . ' times.' |
|
| 57 | + $class_method.'() expected to be not invoked, but invoked '.$actual_times.' times.' |
|
| 58 | 58 | ); |
| 59 | 59 | } |
| 60 | 60 | elseif ($expected_times === '+') |
@@ -62,7 +62,7 @@ discard block |
||
| 62 | 62 | PHPUnit_Framework_TestCase::assertGreaterThanOrEqual( |
| 63 | 63 | 1, |
| 64 | 64 | $actual_times, |
| 65 | - $class_method . '() expected to be invoked at least one time, but invoked ' . $actual_times . ' times.' |
|
| 65 | + $class_method.'() expected to be invoked at least one time, but invoked '.$actual_times.' times.' |
|
| 66 | 66 | ); |
| 67 | 67 | } |
| 68 | 68 | else |
@@ -70,7 +70,7 @@ discard block |
||
| 70 | 70 | PHPUnit_Framework_TestCase::assertEquals( |
| 71 | 71 | $expected_times, |
| 72 | 72 | $actual_times, |
| 73 | - $class_method . '() expected to be invoked ' . $expected_times . ' times, but invoked ' . $actual_times . ' times.' |
|
| 73 | + $class_method.'() expected to be invoked '.$expected_times.' times, but invoked '.$actual_times.' times.' |
|
| 74 | 74 | ); |
| 75 | 75 | } |
| 76 | 76 | } |
@@ -31,12 +31,10 @@ discard block |
||
| 31 | 31 | if ($invoked === false) |
| 32 | 32 | { |
| 33 | 33 | $actual_times = 0; |
| 34 | - } |
|
| 35 | - elseif ($expected_params === null) |
|
| 34 | + } elseif ($expected_params === null) |
|
| 36 | 35 | { |
| 37 | 36 | $actual_times = count($invocations[$class_method]); |
| 38 | - } |
|
| 39 | - else |
|
| 37 | + } else |
|
| 40 | 38 | { |
| 41 | 39 | $count = 0; |
| 42 | 40 | foreach ($invocations[$class_method] as $actual_params) |
@@ -56,16 +54,14 @@ discard block |
||
| 56 | 54 | $actual_times, |
| 57 | 55 | $class_method . '() expected to be not invoked, but invoked ' . $actual_times . ' times.' |
| 58 | 56 | ); |
| 59 | - } |
|
| 60 | - elseif ($expected_times === '+') |
|
| 57 | + } elseif ($expected_times === '+') |
|
| 61 | 58 | { |
| 62 | 59 | PHPUnit_Framework_TestCase::assertGreaterThanOrEqual( |
| 63 | 60 | 1, |
| 64 | 61 | $actual_times, |
| 65 | 62 | $class_method . '() expected to be invoked at least one time, but invoked ' . $actual_times . ' times.' |
| 66 | 63 | ); |
| 67 | - } |
|
| 68 | - else |
|
| 64 | + } else |
|
| 69 | 65 | { |
| 70 | 66 | PHPUnit_Framework_TestCase::assertEquals( |
| 71 | 67 | $expected_times, |