@@ -38,12 +38,12 @@ |
||
| 38 | 38 | $real_path = realpath($path); |
| 39 | 39 | if ($real_path === FALSE) |
| 40 | 40 | { |
| 41 | - throw new RuntimeException($path . ' does not exist?'); |
|
| 41 | + throw new RuntimeException($path.' does not exist?'); |
|
| 42 | 42 | } |
| 43 | 43 | if (is_dir($real_path)) |
| 44 | 44 | { |
| 45 | 45 | // Must use DIRECTORY_SEPARATOR for Windows |
| 46 | - $real_path = $real_path . DIRECTORY_SEPARATOR; |
|
| 46 | + $real_path = $real_path.DIRECTORY_SEPARATOR; |
|
| 47 | 47 | } |
| 48 | 48 | $new_paths[] = $excluded ? '-'.$real_path : $real_path; |
| 49 | 49 | } |
@@ -35,7 +35,7 @@ discard block |
||
| 35 | 35 | { |
| 36 | 36 | if ($this->db === null) |
| 37 | 37 | { |
| 38 | - $CI =& get_instance(); |
|
| 38 | + $CI = & get_instance(); |
|
| 39 | 39 | $CI->load->database(); |
| 40 | 40 | $this->db = $CI->db; |
| 41 | 41 | } |
@@ -54,7 +54,7 @@ discard block |
||
| 54 | 54 | */ |
| 55 | 55 | protected function tearDown() |
| 56 | 56 | { |
| 57 | - if (! empty($this->insertCache)) |
|
| 57 | + if ( ! empty($this->insertCache)) |
|
| 58 | 58 | { |
| 59 | 59 | foreach ($this->insertCache as $row) |
| 60 | 60 | { |
@@ -17,7 +17,7 @@ discard block |
||
| 17 | 17 | |
| 18 | 18 | public function __construct() |
| 19 | 19 | { |
| 20 | - $this->CI =& get_instance(); |
|
| 20 | + $this->CI = & get_instance(); |
|
| 21 | 21 | $this->CI->load->database(); |
| 22 | 22 | $this->CI->load->dbforge(); |
| 23 | 23 | $this->db = $this->CI->db; |
@@ -33,10 +33,10 @@ discard block |
||
| 33 | 33 | { |
| 34 | 34 | if ($this->seedPath === null) |
| 35 | 35 | { |
| 36 | - $this->seedPath = APPPATH . 'database/seeds/'; |
|
| 36 | + $this->seedPath = APPPATH.'database/seeds/'; |
|
| 37 | 37 | } |
| 38 | 38 | |
| 39 | - $file = $this->seedPath . $seeder . '.php'; |
|
| 39 | + $file = $this->seedPath.$seeder.'.php'; |
|
| 40 | 40 | require_once $file; |
| 41 | 41 | |
| 42 | 42 | $obj = new $seeder; |
@@ -48,8 +48,7 @@ |
||
| 48 | 48 | if ($constructor_params === false) |
| 49 | 49 | { |
| 50 | 50 | $mock->disableOriginalConstructor(); |
| 51 | - } |
|
| 52 | - elseif (is_array($constructor_params)) |
|
| 51 | + } elseif (is_array($constructor_params)) |
|
| 53 | 52 | { |
| 54 | 53 | $mock->setConstructorArgs($constructor_params); |
| 55 | 54 | } |
@@ -32,7 +32,7 @@ discard block |
||
| 32 | 32 | * |
| 33 | 33 | * @param string $classname |
| 34 | 34 | * @param array $params [method_name => return_value] |
| 35 | - * @param mixed $constructor_params false: disable construntor, array: construntor params |
|
| 35 | + * @param boolean $constructor_params false: disable construntor, array: construntor params |
|
| 36 | 36 | * |
| 37 | 37 | * @return object PHPUnit mock object |
| 38 | 38 | */ |
@@ -72,6 +72,10 @@ discard block |
||
| 72 | 72 | return $mock; |
| 73 | 73 | } |
| 74 | 74 | |
| 75 | + /** |
|
| 76 | + * @param string $method |
|
| 77 | + * @param string $with |
|
| 78 | + */ |
|
| 75 | 79 | protected function _verify($mock, $method, $params = null, $expects, $with) |
| 76 | 80 | { |
| 77 | 81 | $invocation = $mock->expects($expects) |
@@ -67,8 +67,7 @@ discard block |
||
| 67 | 67 | if ($filename === '' OR $data === '') |
| 68 | 68 | { |
| 69 | 69 | return; |
| 70 | - } |
|
| 71 | - elseif ($data === NULL) |
|
| 70 | + } elseif ($data === NULL) |
|
| 72 | 71 | { |
| 73 | 72 | if ( ! @is_file($filename) OR ($filesize = @filesize($filename)) === FALSE) |
| 74 | 73 | { |
@@ -78,8 +77,7 @@ discard block |
||
| 78 | 77 | $filepath = $filename; |
| 79 | 78 | $filename = explode('/', str_replace(DIRECTORY_SEPARATOR, '/', $filename)); |
| 80 | 79 | $filename = end($filename); |
| 81 | - } |
|
| 82 | - else |
|
| 80 | + } else |
|
| 83 | 81 | { |
| 84 | 82 | $filesize = strlen($data); |
| 85 | 83 | } |
@@ -158,8 +156,7 @@ discard block |
||
| 158 | 156 | if (ENVIRONMENT !== 'testing') |
| 159 | 157 | { |
| 160 | 158 | exit($data); |
| 161 | - } |
|
| 162 | - else |
|
| 159 | + } else |
|
| 163 | 160 | { |
| 164 | 161 | echo($data); |
| 165 | 162 | throw new CIPHPUnitTestExitException('exit() from force_download()'); |
@@ -176,8 +173,7 @@ discard block |
||
| 176 | 173 | if (ENVIRONMENT !== 'testing') |
| 177 | 174 | { |
| 178 | 175 | exit; |
| 179 | - } |
|
| 180 | - else |
|
| 176 | + } else |
|
| 181 | 177 | { |
| 182 | 178 | throw new CIPHPUnitTestExitException('exit() from force_download()'); |
| 183 | 179 | } |
@@ -101,7 +101,7 @@ discard block |
||
| 101 | 101 | } |
| 102 | 102 | |
| 103 | 103 | // Load the mime types |
| 104 | - $mimes =& get_mimes(); |
|
| 104 | + $mimes = & get_mimes(); |
|
| 105 | 105 | |
| 106 | 106 | // Only change the default MIME if we can find one |
| 107 | 107 | if (isset($mimes[$extension])) |
@@ -146,7 +146,7 @@ discard block |
||
| 146 | 146 | |
| 147 | 147 | if (ENVIRONMENT === 'testing') |
| 148 | 148 | { |
| 149 | - $CI =& get_instance(); |
|
| 149 | + $CI = & get_instance(); |
|
| 150 | 150 | $CI->output->set_header('Content-Type: '.$mime); |
| 151 | 151 | $CI->output->set_header('Content-Disposition: attachment; filename="'.$filename.'"'); |
| 152 | 152 | $CI->output->set_header('Expires: 0'); |
@@ -42,9 +42,9 @@ discard block |
||
| 42 | 42 | } |
| 43 | 43 | |
| 44 | 44 | // Handle single file |
| 45 | - $file =& $files[$field]; |
|
| 45 | + $file = & $files[$field]; |
|
| 46 | 46 | |
| 47 | - if (! file_exists($file['tmp_name'])) |
|
| 47 | + if ( ! file_exists($file['tmp_name'])) |
|
| 48 | 48 | { |
| 49 | 49 | $file['error'] = UPLOAD_ERR_NO_FILE; |
| 50 | 50 | $file['type'] = ''; |
@@ -110,7 +110,7 @@ discard block |
||
| 110 | 110 | // Set $_GET if URI string has query string |
| 111 | 111 | parse_str($query_string, $_GET); |
| 112 | 112 | // Remove query string from URI string |
| 113 | - $argv = substr($argv, 0, -strlen($query_string)-1); |
|
| 113 | + $argv = substr($argv, 0, -strlen($query_string) - 1); |
|
| 114 | 114 | } |
| 115 | 115 | } |
| 116 | 116 | |
@@ -143,12 +143,12 @@ discard block |
||
| 143 | 143 | if ($_GET !== []) |
| 144 | 144 | { |
| 145 | 145 | $_SERVER['REQUEST_URI'] = |
| 146 | - '/' . $path . '?' |
|
| 146 | + '/'.$path.'?' |
|
| 147 | 147 | . http_build_query($_GET); |
| 148 | 148 | } |
| 149 | 149 | else |
| 150 | 150 | { |
| 151 | - $_SERVER['REQUEST_URI'] = '/' . $path; |
|
| 151 | + $_SERVER['REQUEST_URI'] = '/'.$path; |
|
| 152 | 152 | } |
| 153 | 153 | } |
| 154 | 154 | |
@@ -165,7 +165,7 @@ discard block |
||
| 165 | 165 | |
| 166 | 166 | if ($query_string === false) |
| 167 | 167 | { |
| 168 | - throw new LogicException('Bad URI string: ' . $uri); |
|
| 168 | + throw new LogicException('Bad URI string: '.$uri); |
|
| 169 | 169 | } |
| 170 | 170 | |
| 171 | 171 | return $query_string; |
@@ -190,7 +190,7 @@ discard block |
||
| 190 | 190 | } |
| 191 | 191 | else |
| 192 | 192 | { |
| 193 | - $key = 'HTTP_' . $normalized_name; |
|
| 193 | + $key = 'HTTP_'.$normalized_name; |
|
| 194 | 194 | } |
| 195 | 195 | |
| 196 | 196 | $_SERVER[$key] = $value; |
@@ -76,8 +76,7 @@ discard block |
||
| 76 | 76 | if (is_numeric($value)) |
| 77 | 77 | { |
| 78 | 78 | return $value; |
| 79 | - } |
|
| 80 | - else |
|
| 79 | + } else |
|
| 81 | 80 | { |
| 82 | 81 | $value_length = strlen($value); |
| 83 | 82 | $number = substr($value, 0, $value_length - 1); |
@@ -133,8 +132,7 @@ discard block |
||
| 133 | 132 | if (is_string($argv)) |
| 134 | 133 | { |
| 135 | 134 | $path = $argv; |
| 136 | - } |
|
| 137 | - elseif (is_array($argv)) |
|
| 135 | + } elseif (is_array($argv)) |
|
| 138 | 136 | { |
| 139 | 137 | // Generate URI path from array of controller, method, arg, ... |
| 140 | 138 | $path = implode('/', $argv); |
@@ -145,8 +143,7 @@ discard block |
||
| 145 | 143 | $_SERVER['REQUEST_URI'] = |
| 146 | 144 | '/' . $path . '?' |
| 147 | 145 | . http_build_query($_GET); |
| 148 | - } |
|
| 149 | - else |
|
| 146 | + } else |
|
| 150 | 147 | { |
| 151 | 148 | $_SERVER['REQUEST_URI'] = '/' . $path; |
| 152 | 149 | } |
@@ -187,8 +184,7 @@ discard block |
||
| 187 | 184 | ) |
| 188 | 185 | { |
| 189 | 186 | $key = $normalized_name; |
| 190 | - } |
|
| 191 | - else |
|
| 187 | + } else |
|
| 192 | 188 | { |
| 193 | 189 | $key = 'HTTP_' . $normalized_name; |
| 194 | 190 | } |
@@ -293,8 +293,8 @@ discard block |
||
| 293 | 293 | { |
| 294 | 294 | empty($config) OR $this->initialize($config, FALSE); |
| 295 | 295 | |
| 296 | - $this->_mimes =& get_mimes(); |
|
| 297 | - $this->_CI =& get_instance(); |
|
| 296 | + $this->_mimes = & get_mimes(); |
|
| 297 | + $this->_CI = & get_instance(); |
|
| 298 | 298 | |
| 299 | 299 | log_message('info', 'Upload Class Initialized'); |
| 300 | 300 | } |
@@ -480,7 +480,7 @@ discard block |
||
| 480 | 480 | else |
| 481 | 481 | { |
| 482 | 482 | // An extension was provided, let's have it! |
| 483 | - $this->file_ext = $this->get_extension($this->_file_name_override); |
|
| 483 | + $this->file_ext = $this->get_extension($this->_file_name_override); |
|
| 484 | 484 | } |
| 485 | 485 | |
| 486 | 486 | if ( ! $this->is_allowed_filetype(TRUE)) |
@@ -493,7 +493,7 @@ discard block |
||
| 493 | 493 | // Convert the file size to kilobytes |
| 494 | 494 | if ($this->file_size > 0) |
| 495 | 495 | { |
| 496 | - $this->file_size = round($this->file_size/1024, 2); |
|
| 496 | + $this->file_size = round($this->file_size / 1024, 2); |
|
| 497 | 497 | } |
| 498 | 498 | |
| 499 | 499 | // Is the file size within the allowed maximum? |
@@ -818,10 +818,10 @@ discard block |
||
| 818 | 818 | { |
| 819 | 819 | $types = array(1 => 'gif', 2 => 'jpeg', 3 => 'png'); |
| 820 | 820 | |
| 821 | - $this->image_width = $D[0]; |
|
| 822 | - $this->image_height = $D[1]; |
|
| 823 | - $this->image_type = isset($types[$D[2]]) ? $types[$D[2]] : 'unknown'; |
|
| 824 | - $this->image_size_str = $D[3]; // string containing height and width |
|
| 821 | + $this->image_width = $D[0]; |
|
| 822 | + $this->image_height = $D[1]; |
|
| 823 | + $this->image_type = isset($types[$D[2]]) ? $types[$D[2]] : 'unknown'; |
|
| 824 | + $this->image_size_str = $D[3]; // string containing height and width |
|
| 825 | 825 | } |
| 826 | 826 | } |
| 827 | 827 | |
@@ -869,7 +869,7 @@ discard block |
||
| 869 | 869 | $this->file_type = 'image/jpeg'; |
| 870 | 870 | } |
| 871 | 871 | |
| 872 | - $img_mimes = array('image/gif', 'image/jpeg', 'image/png'); |
|
| 872 | + $img_mimes = array('image/gif', 'image/jpeg', 'image/png'); |
|
| 873 | 873 | |
| 874 | 874 | return in_array($this->file_type, $img_mimes, TRUE); |
| 875 | 875 | } |
@@ -1011,7 +1011,7 @@ discard block |
||
| 1011 | 1011 | return FALSE; |
| 1012 | 1012 | } |
| 1013 | 1013 | |
| 1014 | - $this->upload_path = preg_replace('/(.+?)\/*$/', '\\1/', $this->upload_path); |
|
| 1014 | + $this->upload_path = preg_replace('/(.+?)\/*$/', '\\1/', $this->upload_path); |
|
| 1015 | 1015 | return TRUE; |
| 1016 | 1016 | } |
| 1017 | 1017 | |
@@ -1055,9 +1055,9 @@ discard block |
||
| 1055 | 1055 | $ext = ''; |
| 1056 | 1056 | if (strpos($filename, '.') !== FALSE) |
| 1057 | 1057 | { |
| 1058 | - $parts = explode('.', $filename); |
|
| 1059 | - $ext = '.'.array_pop($parts); |
|
| 1060 | - $filename = implode('.', $parts); |
|
| 1058 | + $parts = explode('.', $filename); |
|
| 1059 | + $ext = '.'.array_pop($parts); |
|
| 1060 | + $filename = implode('.', $parts); |
|
| 1061 | 1061 | } |
| 1062 | 1062 | |
| 1063 | 1063 | return substr($filename, 0, ($length - strlen($ext))).$ext; |
@@ -331,19 +331,16 @@ discard block |
||
| 331 | 331 | if ($reflection->hasMethod('set_'.$key)) |
| 332 | 332 | { |
| 333 | 333 | $this->{'set_'.$key}($config[$key]); |
| 334 | - } |
|
| 335 | - else |
|
| 334 | + } else |
|
| 336 | 335 | { |
| 337 | 336 | $this->$key = $config[$key]; |
| 338 | 337 | } |
| 339 | - } |
|
| 340 | - else |
|
| 338 | + } else |
|
| 341 | 339 | { |
| 342 | 340 | $this->$key = $defaults[$key]; |
| 343 | 341 | } |
| 344 | 342 | } |
| 345 | - } |
|
| 346 | - else |
|
| 343 | + } else |
|
| 347 | 344 | { |
| 348 | 345 | foreach ($config as $key => &$value) |
| 349 | 346 | { |
@@ -352,8 +349,7 @@ discard block |
||
| 352 | 349 | if ($reflection->hasMethod('set_'.$key)) |
| 353 | 350 | { |
| 354 | 351 | $this->{'set_'.$key}($value); |
| 355 | - } |
|
| 356 | - else |
|
| 352 | + } else |
|
| 357 | 353 | { |
| 358 | 354 | $this->$key = $value; |
| 359 | 355 | } |
@@ -415,9 +411,11 @@ discard block |
||
| 415 | 411 | } |
| 416 | 412 | |
| 417 | 413 | // Was the file able to be uploaded? If not, determine the reason why. |
| 418 | - if ( ! file_exists($_file['tmp_name'])) // modified by ci-phpunit-test |
|
| 414 | + if ( ! file_exists($_file['tmp_name'])) { |
|
| 415 | + // modified by ci-phpunit-test |
|
| 419 | 416 | { |
| 420 | 417 | $error = isset($_file['error']) ? $_file['error'] : 4; |
| 418 | + } |
|
| 421 | 419 | |
| 422 | 420 | switch ($error) |
| 423 | 421 | { |
@@ -482,8 +480,7 @@ discard block |
||
| 482 | 480 | if (strpos($this->_file_name_override, '.') === FALSE) |
| 483 | 481 | { |
| 484 | 482 | $this->file_name .= $this->file_ext; |
| 485 | - } |
|
| 486 | - else |
|
| 483 | + } else |
|
| 487 | 484 | { |
| 488 | 485 | // An extension was provided, let's have it! |
| 489 | 486 | $this->file_ext = $this->get_extension($this->_file_name_override); |
@@ -687,8 +684,7 @@ discard block |
||
| 687 | 684 | { |
| 688 | 685 | $this->set_error('upload_bad_filename', 'debug'); |
| 689 | 686 | return FALSE; |
| 690 | - } |
|
| 691 | - else |
|
| 687 | + } else |
|
| 692 | 688 | { |
| 693 | 689 | return $new_filename; |
| 694 | 690 | } |
@@ -883,8 +879,7 @@ discard block |
||
| 883 | 879 | if (in_array($this->file_type, $png_mimes)) |
| 884 | 880 | { |
| 885 | 881 | $this->file_type = 'image/png'; |
| 886 | - } |
|
| 887 | - elseif (in_array($this->file_type, $jpeg_mimes)) |
|
| 882 | + } elseif (in_array($this->file_type, $jpeg_mimes)) |
|
| 888 | 883 | { |
| 889 | 884 | $this->file_type = 'image/jpeg'; |
| 890 | 885 | } |
@@ -1150,9 +1145,12 @@ discard block |
||
| 1150 | 1145 | |
| 1151 | 1146 | if (function_exists('getimagesize') && @getimagesize($file) !== FALSE) |
| 1152 | 1147 | { |
| 1153 | - if (($file = @fopen($file, 'rb')) === FALSE) // "b" to force binary |
|
| 1148 | + if (($file = @fopen($file, 'rb')) === FALSE) { |
|
| 1149 | + // "b" to force binary |
|
| 1154 | 1150 | { |
| 1155 | - return FALSE; // Couldn't open the file, return FALSE |
|
| 1151 | + return FALSE; |
|
| 1152 | + } |
|
| 1153 | + // Couldn't open the file, return FALSE |
|
| 1156 | 1154 | } |
| 1157 | 1155 | |
| 1158 | 1156 | $opening_bytes = fread($file, 256); |
@@ -1271,9 +1269,11 @@ discard block |
||
| 1271 | 1269 | if (function_exists('finfo_file')) |
| 1272 | 1270 | { |
| 1273 | 1271 | $finfo = @finfo_open(FILEINFO_MIME); |
| 1274 | - if (is_resource($finfo)) // It is possible that a FALSE value is returned, if there is no magic MIME database file found on the system |
|
| 1272 | + if (is_resource($finfo)) { |
|
| 1273 | + // It is possible that a FALSE value is returned, if there is no magic MIME database file found on the system |
|
| 1275 | 1274 | { |
| 1276 | 1275 | $mime = @finfo_file($finfo, $file['tmp_name']); |
| 1276 | + } |
|
| 1277 | 1277 | finfo_close($finfo); |
| 1278 | 1278 | |
| 1279 | 1279 | /* According to the comments section of the PHP manual page, |
@@ -1358,10 +1358,12 @@ discard block |
||
| 1358 | 1358 | if (function_exists('mime_content_type')) |
| 1359 | 1359 | { |
| 1360 | 1360 | $this->file_type = @mime_content_type($file['tmp_name']); |
| 1361 | - if (strlen($this->file_type) > 0) // It's possible that mime_content_type() returns FALSE or an empty string |
|
| 1361 | + if (strlen($this->file_type) > 0) { |
|
| 1362 | + // It's possible that mime_content_type() returns FALSE or an empty string |
|
| 1362 | 1363 | { |
| 1363 | 1364 | return; |
| 1364 | 1365 | } |
| 1366 | + } |
|
| 1365 | 1367 | } |
| 1366 | 1368 | |
| 1367 | 1369 | $this->file_type = $file['type']; |
@@ -200,16 +200,14 @@ discard block |
||
| 200 | 200 | if (empty($library)) |
| 201 | 201 | { |
| 202 | 202 | return $this; |
| 203 | - } |
|
| 204 | - elseif (is_array($library)) |
|
| 203 | + } elseif (is_array($library)) |
|
| 205 | 204 | { |
| 206 | 205 | foreach ($library as $key => $value) |
| 207 | 206 | { |
| 208 | 207 | if (is_int($key)) |
| 209 | 208 | { |
| 210 | 209 | $this->library($value, $params); |
| 211 | - } |
|
| 212 | - else |
|
| 210 | + } else |
|
| 213 | 211 | { |
| 214 | 212 | $this->library($key, $params, $value); |
| 215 | 213 | } |
@@ -246,8 +244,7 @@ discard block |
||
| 246 | 244 | if (empty($model)) |
| 247 | 245 | { |
| 248 | 246 | return $this; |
| 249 | - } |
|
| 250 | - elseif (is_array($model)) |
|
| 247 | + } elseif (is_array($model)) |
|
| 251 | 248 | { |
| 252 | 249 | foreach ($model as $key => $value) |
| 253 | 250 | { |
@@ -313,8 +310,7 @@ discard block |
||
| 313 | 310 | { |
| 314 | 311 | throw new RuntimeException($app_path."Model.php exists, but doesn't declare class CI_Model"); |
| 315 | 312 | } |
| 316 | - } |
|
| 317 | - elseif ( ! class_exists('CI_Model', FALSE)) |
|
| 313 | + } elseif ( ! class_exists('CI_Model', FALSE)) |
|
| 318 | 314 | { |
| 319 | 315 | require_once(BASEPATH.'core'.DIRECTORY_SEPARATOR.'Model.php'); |
| 320 | 316 | } |
@@ -471,8 +467,7 @@ discard block |
||
| 471 | 467 | require_once($driver_path); |
| 472 | 468 | $class = 'CI_DB_'.$db->dbdriver.'_'.$db->subdriver.'_forge'; |
| 473 | 469 | } |
| 474 | - } |
|
| 475 | - else |
|
| 470 | + } else |
|
| 476 | 471 | { |
| 477 | 472 | $class = 'CI_DB_'.$db->dbdriver.'_forge'; |
| 478 | 473 | } |
@@ -761,16 +756,14 @@ discard block |
||
| 761 | 756 | if (is_int($key)) |
| 762 | 757 | { |
| 763 | 758 | $this->driver($value, $params); |
| 764 | - } |
|
| 765 | - else |
|
| 759 | + } else |
|
| 766 | 760 | { |
| 767 | 761 | $this->driver($key, $params, $value); |
| 768 | 762 | } |
| 769 | 763 | } |
| 770 | 764 | |
| 771 | 765 | return $this; |
| 772 | - } |
|
| 773 | - elseif (empty($library)) |
|
| 766 | + } elseif (empty($library)) |
|
| 774 | 767 | { |
| 775 | 768 | return FALSE; |
| 776 | 769 | } |
@@ -869,8 +862,7 @@ discard block |
||
| 869 | 862 | array_shift($this->_ci_helper_paths); |
| 870 | 863 | array_shift($this->_ci_view_paths); |
| 871 | 864 | array_pop($config->_config_paths); |
| 872 | - } |
|
| 873 | - else |
|
| 865 | + } else |
|
| 874 | 866 | { |
| 875 | 867 | $path = rtrim($path, '/').'/'; |
| 876 | 868 | foreach (array('_ci_library_paths', '_ci_model_paths', '_ci_helper_paths') as $var) |
@@ -934,8 +926,7 @@ discard block |
||
| 934 | 926 | { |
| 935 | 927 | $_ci_x = explode('/', $_ci_path); |
| 936 | 928 | $_ci_file = end($_ci_x); |
| 937 | - } |
|
| 938 | - else |
|
| 929 | + } else |
|
| 939 | 930 | { |
| 940 | 931 | $_ci_ext = pathinfo($_ci_view, PATHINFO_EXTENSION); |
| 941 | 932 | $_ci_file = ($_ci_ext === '') ? $_ci_view.'.php' : $_ci_view; |
@@ -1002,8 +993,7 @@ discard block |
||
| 1002 | 993 | if ( ! is_php('5.4') && ! ini_get('short_open_tag') && config_item('rewrite_short_tags') === TRUE) |
| 1003 | 994 | { |
| 1004 | 995 | echo eval('?>'.preg_replace('/;*\s*\?>/', '; ?>', str_replace('<?=', '<?php echo ', file_get_contents($_ci_path)))); |
| 1005 | - } |
|
| 1006 | - else |
|
| 996 | + } else |
|
| 1007 | 997 | { |
| 1008 | 998 | include($_ci_path); // include() vs include_once() allows for multiple views with the same name |
| 1009 | 999 | } |
@@ -1030,8 +1020,7 @@ discard block |
||
| 1030 | 1020 | if (ob_get_level() > $this->_ci_ob_level + 1) |
| 1031 | 1021 | { |
| 1032 | 1022 | ob_end_flush(); |
| 1033 | - } |
|
| 1034 | - else |
|
| 1023 | + } else |
|
| 1035 | 1024 | { |
| 1036 | 1025 | $_ci_CI->output->append_output(ob_get_contents()); |
| 1037 | 1026 | @ob_end_clean(); |
@@ -1071,8 +1060,7 @@ discard block |
||
| 1071 | 1060 | |
| 1072 | 1061 | // Get the filename from the path |
| 1073 | 1062 | $class = substr($class, $last_slash); |
| 1074 | - } |
|
| 1075 | - else |
|
| 1063 | + } else |
|
| 1076 | 1064 | { |
| 1077 | 1065 | $subdir = ''; |
| 1078 | 1066 | } |
@@ -1210,8 +1198,7 @@ discard block |
||
| 1210 | 1198 | if (file_exists(APPPATH.'tests/_ci_phpunit_test/replacing/libraries/'.$file_path.$library_name.'.php')) |
| 1211 | 1199 | { |
| 1212 | 1200 | include_once(APPPATH.'tests/_ci_phpunit_test/replacing/libraries/'.$file_path.$library_name.'.php'); |
| 1213 | - } |
|
| 1214 | - else |
|
| 1201 | + } else |
|
| 1215 | 1202 | { |
| 1216 | 1203 | include_once(BASEPATH.'libraries/'.$file_path.$library_name.'.php'); |
| 1217 | 1204 | } |
@@ -1227,8 +1214,7 @@ discard block |
||
| 1227 | 1214 | { |
| 1228 | 1215 | $prefix = config_item('subclass_prefix'); |
| 1229 | 1216 | break; |
| 1230 | - } |
|
| 1231 | - else |
|
| 1217 | + } else |
|
| 1232 | 1218 | { |
| 1233 | 1219 | log_message('debug', $path.' exists, but does not declare '.$subclass); |
| 1234 | 1220 | } |
@@ -1277,8 +1263,7 @@ discard block |
||
| 1277 | 1263 | { |
| 1278 | 1264 | include($path.'config/'.strtolower($class).'.php'); |
| 1279 | 1265 | $found = TRUE; |
| 1280 | - } |
|
| 1281 | - elseif (file_exists($path.'config/'.ucfirst(strtolower($class)).'.php')) |
|
| 1266 | + } elseif (file_exists($path.'config/'.ucfirst(strtolower($class)).'.php')) |
|
| 1282 | 1267 | { |
| 1283 | 1268 | include($path.'config/'.ucfirst(strtolower($class)).'.php'); |
| 1284 | 1269 | $found = TRUE; |
@@ -1288,8 +1273,7 @@ discard block |
||
| 1288 | 1273 | { |
| 1289 | 1274 | include($path.'config/'.ENVIRONMENT.'/'.strtolower($class).'.php'); |
| 1290 | 1275 | $found = TRUE; |
| 1291 | - } |
|
| 1292 | - elseif (file_exists($path.'config/'.ENVIRONMENT.'/'.ucfirst(strtolower($class)).'.php')) |
|
| 1276 | + } elseif (file_exists($path.'config/'.ENVIRONMENT.'/'.ucfirst(strtolower($class)).'.php')) |
|
| 1293 | 1277 | { |
| 1294 | 1278 | include($path.'config/'.ENVIRONMENT.'/'.ucfirst(strtolower($class)).'.php'); |
| 1295 | 1279 | $found = TRUE; |
@@ -191,7 +191,7 @@ discard block |
||
| 191 | 191 | * @param string $library Library name |
| 192 | 192 | * @param array $params Optional parameters to pass to the library class constructor |
| 193 | 193 | * @param string $object_name An optional object name to assign to |
| 194 | - * @return object |
|
| 194 | + * @return CI_Loader |
|
| 195 | 195 | * |
| 196 | 196 | * @codeCoverageIgnore |
| 197 | 197 | */ |
@@ -237,7 +237,7 @@ discard block |
||
| 237 | 237 | * @param string $model Model name |
| 238 | 238 | * @param string $name An optional object name to assign to |
| 239 | 239 | * @param bool $db_conn An optional database connection configuration to initialize |
| 240 | - * @return object |
|
| 240 | + * @return CI_Loader |
|
| 241 | 241 | * |
| 242 | 242 | * modified by ci-phpunit-test |
| 243 | 243 | */ |
@@ -369,7 +369,7 @@ discard block |
||
| 369 | 369 | /** |
| 370 | 370 | * Database Loader |
| 371 | 371 | * |
| 372 | - * @param mixed $params Database configuration options |
|
| 372 | + * @param string|boolean $params Database configuration options |
|
| 373 | 373 | * @param bool $return Whether to return the database object |
| 374 | 374 | * @param bool $query_builder Whether to enable Query Builder |
| 375 | 375 | * (overrides the configuration setting) |
@@ -535,7 +535,7 @@ discard block |
||
| 535 | 535 | * An associative array or object containing values |
| 536 | 536 | * to be set, or a value's name if string |
| 537 | 537 | * @param string $val Value to set, only used if $vars is a string |
| 538 | - * @return object |
|
| 538 | + * @return CI_Loader |
|
| 539 | 539 | * |
| 540 | 540 | * @codeCoverageIgnore |
| 541 | 541 | */ |
@@ -609,7 +609,7 @@ discard block |
||
| 609 | 609 | * Helper Loader |
| 610 | 610 | * |
| 611 | 611 | * @param string|string[] $helpers Helper name(s) |
| 612 | - * @return object |
|
| 612 | + * @return CI_Loader |
|
| 613 | 613 | * |
| 614 | 614 | * @codeCoverageIgnore |
| 615 | 615 | */ |
@@ -688,7 +688,7 @@ discard block |
||
| 688 | 688 | * |
| 689 | 689 | * @uses CI_Loader::helper() |
| 690 | 690 | * @param string|string[] $helpers Helper name(s) |
| 691 | - * @return object |
|
| 691 | + * @return CI_Loader |
|
| 692 | 692 | * |
| 693 | 693 | * @codeCoverageIgnore |
| 694 | 694 | */ |
@@ -706,7 +706,7 @@ discard block |
||
| 706 | 706 | * |
| 707 | 707 | * @param string|string[] $files List of language file names to load |
| 708 | 708 | * @param string Language name |
| 709 | - * @return object |
|
| 709 | + * @return CI_Loader |
|
| 710 | 710 | * |
| 711 | 711 | * @codeCoverageIgnore |
| 712 | 712 | */ |
@@ -806,7 +806,7 @@ discard block |
||
| 806 | 806 | * |
| 807 | 807 | * @param string $path Path to add |
| 808 | 808 | * @param bool $view_cascade (default: TRUE) |
| 809 | - * @return object |
|
| 809 | + * @return CI_Loader |
|
| 810 | 810 | * |
| 811 | 811 | * @codeCoverageIgnore |
| 812 | 812 | */ |
@@ -854,7 +854,7 @@ discard block |
||
| 854 | 854 | * added path will be removed removed. |
| 855 | 855 | * |
| 856 | 856 | * @param string $path Path to remove |
| 857 | - * @return object |
|
| 857 | + * @return CI_Loader |
|
| 858 | 858 | * |
| 859 | 859 | * @codeCoverageIgnore |
| 860 | 860 | */ |
@@ -63,63 +63,63 @@ discard block |
||
| 63 | 63 | * |
| 64 | 64 | * @var array |
| 65 | 65 | */ |
| 66 | - protected $_ci_view_paths = array(VIEWPATH => TRUE); |
|
| 66 | + protected $_ci_view_paths = array(VIEWPATH => TRUE); |
|
| 67 | 67 | |
| 68 | 68 | /** |
| 69 | 69 | * List of paths to load libraries from |
| 70 | 70 | * |
| 71 | 71 | * @var array |
| 72 | 72 | */ |
| 73 | - protected $_ci_library_paths = array(APPPATH, BASEPATH); |
|
| 73 | + protected $_ci_library_paths = array(APPPATH, BASEPATH); |
|
| 74 | 74 | |
| 75 | 75 | /** |
| 76 | 76 | * List of paths to load models from |
| 77 | 77 | * |
| 78 | 78 | * @var array |
| 79 | 79 | */ |
| 80 | - protected $_ci_model_paths = array(APPPATH); |
|
| 80 | + protected $_ci_model_paths = array(APPPATH); |
|
| 81 | 81 | |
| 82 | 82 | /** |
| 83 | 83 | * List of paths to load helpers from |
| 84 | 84 | * |
| 85 | 85 | * @var array |
| 86 | 86 | */ |
| 87 | - protected $_ci_helper_paths = array(APPPATH, BASEPATH); |
|
| 87 | + protected $_ci_helper_paths = array(APPPATH, BASEPATH); |
|
| 88 | 88 | |
| 89 | 89 | /** |
| 90 | 90 | * List of cached variables |
| 91 | 91 | * |
| 92 | 92 | * @var array |
| 93 | 93 | */ |
| 94 | - protected $_ci_cached_vars = array(); |
|
| 94 | + protected $_ci_cached_vars = array(); |
|
| 95 | 95 | |
| 96 | 96 | /** |
| 97 | 97 | * List of loaded classes |
| 98 | 98 | * |
| 99 | 99 | * @var array |
| 100 | 100 | */ |
| 101 | - protected $_ci_classes = array(); |
|
| 101 | + protected $_ci_classes = array(); |
|
| 102 | 102 | |
| 103 | 103 | /** |
| 104 | 104 | * List of loaded models |
| 105 | 105 | * |
| 106 | 106 | * @var array |
| 107 | 107 | */ |
| 108 | - protected $_ci_models = array(); |
|
| 108 | + protected $_ci_models = array(); |
|
| 109 | 109 | |
| 110 | 110 | /** |
| 111 | 111 | * List of loaded helpers |
| 112 | 112 | * |
| 113 | 113 | * @var array |
| 114 | 114 | */ |
| 115 | - protected $_ci_helpers = array(); |
|
| 115 | + protected $_ci_helpers = array(); |
|
| 116 | 116 | |
| 117 | 117 | /** |
| 118 | 118 | * List of class name mappings |
| 119 | 119 | * |
| 120 | 120 | * @var array |
| 121 | 121 | */ |
| 122 | - protected $_ci_varmap = array( |
|
| 122 | + protected $_ci_varmap = array( |
|
| 123 | 123 | 'unit_test' => 'unit', |
| 124 | 124 | 'user_agent' => 'agent' |
| 125 | 125 | ); |
@@ -138,7 +138,7 @@ discard block |
||
| 138 | 138 | public function __construct() |
| 139 | 139 | { |
| 140 | 140 | $this->_ci_ob_level = ob_get_level(); |
| 141 | - $this->_ci_classes =& is_loaded(); |
|
| 141 | + $this->_ci_classes = & is_loaded(); |
|
| 142 | 142 | |
| 143 | 143 | log_message('info', 'Loader Class Initialized'); |
| 144 | 144 | } |
@@ -279,7 +279,7 @@ discard block |
||
| 279 | 279 | return $this; |
| 280 | 280 | } |
| 281 | 281 | |
| 282 | - $CI =& get_instance(); |
|
| 282 | + $CI = & get_instance(); |
|
| 283 | 283 | if (isset($CI->$name)) |
| 284 | 284 | { |
| 285 | 285 | throw new RuntimeException('The model name you are loading is the name of a resource that is already being used: '.$name); |
@@ -382,7 +382,7 @@ discard block |
||
| 382 | 382 | public function database($params = '', $return = FALSE, $query_builder = NULL) |
| 383 | 383 | { |
| 384 | 384 | // Grab the super object |
| 385 | - $CI =& get_instance(); |
|
| 385 | + $CI = & get_instance(); |
|
| 386 | 386 | |
| 387 | 387 | // Do we even need to load the database class? |
| 388 | 388 | if ($return === FALSE && $query_builder === NULL && isset($CI->db) && is_object($CI->db) && ! empty($CI->db->conn_id)) |
@@ -402,7 +402,7 @@ discard block |
||
| 402 | 402 | $CI->db = ''; |
| 403 | 403 | |
| 404 | 404 | // Load the DB class |
| 405 | - $CI->db =& DB($params, $query_builder); |
|
| 405 | + $CI->db = & DB($params, $query_builder); |
|
| 406 | 406 | return $this; |
| 407 | 407 | } |
| 408 | 408 | |
@@ -419,12 +419,12 @@ discard block |
||
| 419 | 419 | */ |
| 420 | 420 | public function dbutil($db = NULL, $return = FALSE) |
| 421 | 421 | { |
| 422 | - $CI =& get_instance(); |
|
| 422 | + $CI = & get_instance(); |
|
| 423 | 423 | |
| 424 | 424 | if ( ! is_object($db) OR ! ($db instanceof CI_DB)) |
| 425 | 425 | { |
| 426 | 426 | class_exists('CI_DB', FALSE) OR $this->database(); |
| 427 | - $db =& $CI->db; |
|
| 427 | + $db = & $CI->db; |
|
| 428 | 428 | } |
| 429 | 429 | |
| 430 | 430 | require_once(BASEPATH.'database/DB_utility.php'); |
@@ -453,11 +453,11 @@ discard block |
||
| 453 | 453 | */ |
| 454 | 454 | public function dbforge($db = NULL, $return = FALSE) |
| 455 | 455 | { |
| 456 | - $CI =& get_instance(); |
|
| 456 | + $CI = & get_instance(); |
|
| 457 | 457 | if ( ! is_object($db) OR ! ($db instanceof CI_DB)) |
| 458 | 458 | { |
| 459 | 459 | class_exists('CI_DB', FALSE) OR $this->database(); |
| 460 | - $db =& $CI->db; |
|
| 460 | + $db = & $CI->db; |
|
| 461 | 461 | } |
| 462 | 462 | |
| 463 | 463 | require_once(BASEPATH.'database/DB_forge.php'); |
@@ -821,7 +821,7 @@ discard block |
||
| 821 | 821 | $this->_ci_view_paths = array($path.'views/' => $view_cascade) + $this->_ci_view_paths; |
| 822 | 822 | |
| 823 | 823 | // Add config file path |
| 824 | - $config =& $this->_ci_get_component('config'); |
|
| 824 | + $config = & $this->_ci_get_component('config'); |
|
| 825 | 825 | $config->_config_paths[] = $path; |
| 826 | 826 | |
| 827 | 827 | return $this; |
@@ -860,7 +860,7 @@ discard block |
||
| 860 | 860 | */ |
| 861 | 861 | public function remove_package_path($path = '') |
| 862 | 862 | { |
| 863 | - $config =& $this->_ci_get_component('config'); |
|
| 863 | + $config = & $this->_ci_get_component('config'); |
|
| 864 | 864 | |
| 865 | 865 | if ($path === '') |
| 866 | 866 | { |
@@ -963,12 +963,12 @@ discard block |
||
| 963 | 963 | |
| 964 | 964 | // This allows anything loaded using $this->load (views, files, etc.) |
| 965 | 965 | // to become accessible from within the Controller and Model functions. |
| 966 | - $_ci_CI =& get_instance(); |
|
| 966 | + $_ci_CI = & get_instance(); |
|
| 967 | 967 | foreach (get_object_vars($_ci_CI) as $_ci_key => $_ci_var) |
| 968 | 968 | { |
| 969 | 969 | if ( ! isset($this->$_ci_key)) |
| 970 | 970 | { |
| 971 | - $this->$_ci_key =& $_ci_CI->$_ci_key; |
|
| 971 | + $this->$_ci_key = & $_ci_CI->$_ci_key; |
|
| 972 | 972 | } |
| 973 | 973 | } |
| 974 | 974 | |
@@ -1110,7 +1110,7 @@ discard block |
||
| 1110 | 1110 | // return a new instance of the object |
| 1111 | 1111 | if ($object_name !== NULL) |
| 1112 | 1112 | { |
| 1113 | - $CI =& get_instance(); |
|
| 1113 | + $CI = & get_instance(); |
|
| 1114 | 1114 | if ( ! isset($CI->$object_name)) |
| 1115 | 1115 | { |
| 1116 | 1116 | return $this->_ci_init_library($class, '', $params, $object_name); |
@@ -1173,7 +1173,7 @@ discard block |
||
| 1173 | 1173 | // return a new instance of the object |
| 1174 | 1174 | if ($object_name !== NULL) |
| 1175 | 1175 | { |
| 1176 | - $CI =& get_instance(); |
|
| 1176 | + $CI = & get_instance(); |
|
| 1177 | 1177 | if ( ! isset($CI->$object_name)) |
| 1178 | 1178 | { |
| 1179 | 1179 | return $this->_ci_init_library($library_name, $prefix, $params, $object_name); |
@@ -1326,7 +1326,7 @@ discard block |
||
| 1326 | 1326 | } |
| 1327 | 1327 | |
| 1328 | 1328 | // Don't overwrite existing properties |
| 1329 | - $CI =& get_instance(); |
|
| 1329 | + $CI = & get_instance(); |
|
| 1330 | 1330 | if (isset($CI->$object_name)) |
| 1331 | 1331 | { |
| 1332 | 1332 | if ($CI->$object_name instanceof $class_name) |
@@ -1477,7 +1477,7 @@ discard block |
||
| 1477 | 1477 | */ |
| 1478 | 1478 | protected function &_ci_get_component($component) |
| 1479 | 1479 | { |
| 1480 | - $CI =& get_instance(); |
|
| 1480 | + $CI = & get_instance(); |
|
| 1481 | 1481 | return $CI->$component; |
| 1482 | 1482 | } |
| 1483 | 1483 | } |
@@ -15,17 +15,17 @@ |
||
| 15 | 15 | |
| 16 | 16 | // php install.php -s |
| 17 | 17 | if (isset($argv[1]) && $argv[1] === '-s') { |
| 18 | - $silent = true; |
|
| 18 | + $silent = true; |
|
| 19 | 19 | |
| 20 | - // php install.php -s app |
|
| 21 | - if (isset($argv[2]) && is_dir($argv[2])) { |
|
| 22 | - $app_dir = $argv[2]; |
|
| 23 | - } |
|
| 20 | + // php install.php -s app |
|
| 21 | + if (isset($argv[2]) && is_dir($argv[2])) { |
|
| 22 | + $app_dir = $argv[2]; |
|
| 23 | + } |
|
| 24 | 24 | } |
| 25 | 25 | |
| 26 | 26 | // php install.php app |
| 27 | 27 | if (isset($argv[1]) && is_dir($argv[1])) { |
| 28 | - $app_dir = $argv[1]; |
|
| 28 | + $app_dir = $argv[1]; |
|
| 29 | 29 | } |
| 30 | 30 | |
| 31 | 31 | $installer = new Installer($silent); |
@@ -8,7 +8,7 @@ |
||
| 8 | 8 | * @link https://github.com/kenjis/ci-phpunit-test |
| 9 | 9 | */ |
| 10 | 10 | |
| 11 | -require __DIR__ . '/Installer.php'; |
|
| 11 | +require __DIR__.'/Installer.php'; |
|
| 12 | 12 | |
| 13 | 13 | $silent = false; |
| 14 | 14 | $app_dir = 'application'; |