adulau/cakephp2-php8
1
0
Fork 0
mirror of https://github.com/kamilwylegala/cakephp2-php8.git synced 2024-11-15 03:18:26 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions

The actual config file must always have .php extension.

Browse source
This commit is contained in:
ber clausen 2012-08-10 10:46:18 -03:00 committed by Ber Clausen
parent 65e63c51c8
commit 0dec093f48
3 changed files with 31 additions and 12 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

9
lib/Cake/Configure/PhpReader.php
View file

@ -66,22 +66,21 @@ class PhpReader implements ConfigReaderInterface {
$key = substr($key, 0, -4);
}
list($plugin, $key) = pluginSplit($key);
$key .= '.php';
if ($plugin) {
$file = App::pluginPath($plugin) . 'Config' . DS . $key;
} else {
$file = $this->_path . $key;
}
$file .= '.php';
if (!is_file($file)) {
if (!is_file(substr($file, 0, -4))) {
throw new ConfigureException(__d('cake_dev', 'Could not load configuration files: %s or %s', $file, substr($file, 0, -4)));
}
throw new ConfigureException(__d('cake_dev', 'Could not load configuration file: %s', $file));
}
include $file;
if (!isset($config)) {
throw new ConfigureException(
sprintf(__d('cake_dev', 'No variable $config found in %s.php'), $file)
sprintf(__d('cake_dev', 'No variable $config found in %s'), $file)
);
}
return $config;

25
lib/Cake/Test/Case/Configure/PhpReaderTest.php
View file

@ -41,7 +41,7 @@ class PhpReaderTest extends CakeTestCase {
);
/**
* setup
* Setup.
*
* @return void
*/
@ -51,7 +51,7 @@ class PhpReaderTest extends CakeTestCase {
}
/**
* test reading files
* Test reading files.
*
* @return void
*/
@ -65,21 +65,32 @@ class PhpReaderTest extends CakeTestCase {
$this->assertEquals('value', $values['Read']);
}
/**
* Test an exception is thrown by reading files that exist without .php extension.
*
* @expectedException ConfigureException
* @return void
*/
public function testReadWithExistentFileWithoutExtension() {
$reader = new PhpReader($this->path);
$reader->read('no_php_extension');
}
/**
* Test an exception is thrown by reading files that don't exist.
*
* @expectedException ConfigureException
* @return void
*/
public function testReadWithNonExistantFile() {
public function testReadWithNonExistentFile() {
$reader = new PhpReader($this->path);
$reader->read('fake_values');
}
/**
* test reading an empty file.
* Test reading an empty file.
*
* @expectedException RuntimeException
* @expectedException ConfigureException
* @return void
*/
public function testReadEmptyFile() {
@ -88,7 +99,7 @@ class PhpReaderTest extends CakeTestCase {
}
/**
* test reading keys with ../ doesn't work
* Test reading keys with ../ doesn't work.
*
* @expectedException ConfigureException
* @return void
@ -99,7 +110,7 @@ class PhpReaderTest extends CakeTestCase {
}
/**
* test reading from plugins
* Test reading from plugins.
*
* @return void
*/

9
lib/Cake/Test/test_app/Config/no_php_extension Normal file
View file

@ -0,0 +1,9 @@
<?php
// Test file for testing config file without .php extension.
$config = array(
'Deep' => array(
'Third' => array(
'ThirdDeepest' => 'buried3'
)
)
);