Browse Source

Merge branch 'master' into 2.5

ADmad 12 years ago
parent
commit
c27aa99af9

+ 2 - 2
app/Config/bootstrap.php

@@ -98,12 +98,12 @@ Configure::write('Dispatcher.filters', array(
  */
 App::uses('CakeLog', 'Log');
 CakeLog::config('debug', array(
-	'engine' => 'FileLog',
+	'engine' => 'File',
 	'types' => array('notice', 'info', 'debug'),
 	'file' => 'debug',
 ));
 CakeLog::config('error', array(
-	'engine' => 'FileLog',
+	'engine' => 'File',
 	'types' => array('warning', 'error', 'critical', 'alert', 'emergency'),
 	'file' => 'error',
 ));

+ 14 - 4
lib/Cake/Cache/Engine/FileEngine.php

@@ -210,7 +210,10 @@ class FileEngine extends CacheEngine {
 		}
 		$path = $this->_File->getRealPath();
 		$this->_File = null;
-		return unlink($path);
+
+		//@codingStandardsIgnoreStart
+		return @unlink($path);
+		//@codingStandardsIgnoreEnd
 	}
 
 /**
@@ -289,9 +292,12 @@ class FileEngine extends CacheEngine {
 				}
 			}
 			if ($file->isFile()) {
-				$_path = $file->getRealPath();
+				$filePath = $file->getRealPath();
 				$file = null;
-				unlink($_path);
+
+				//@codingStandardsIgnoreStart
+				@unlink($filePath);
+				//@codingStandardsIgnoreEnd
 			}
 		}
 	}
@@ -411,7 +417,11 @@ class FileEngine extends CacheEngine {
 			$containsGroup = strpos($object->getPathName(), DS . $group . DS) !== false;
 			$hasPrefix = strpos($object->getBaseName(), $this->settings['prefix']) === 0;
 			if ($object->isFile() && $containsGroup && $hasPrefix) {
-				unlink($object->getPathName());
+				$path = $object->getPathName();
+				$object = null;
+				//@codingStandardsIgnoreStart
+				@unlink($path);
+				//@codingStandardsIgnoreEnd
 			}
 		}
 		return true;

+ 2 - 2
lib/Cake/Console/Templates/skel/Config/bootstrap.php

@@ -89,12 +89,12 @@ Configure::write('Dispatcher.filters', array(
  */
 App::uses('CakeLog', 'Log');
 CakeLog::config('debug', array(
-	'engine' => 'FileLog',
+	'engine' => 'File',
 	'types' => array('notice', 'info', 'debug'),
 	'file' => 'debug',
 ));
 CakeLog::config('error', array(
-	'engine' => 'FileLog',
+	'engine' => 'File',
 	'types' => array('warning', 'error', 'critical', 'alert', 'emergency'),
 	'file' => 'error',
 ));

+ 5 - 1
lib/Cake/Model/Behavior/TranslateBehavior.php

@@ -320,9 +320,11 @@ class TranslateBehavior extends ModelBehavior {
  * beforeValidate Callback
  *
  * @param Model $Model Model invalidFields was called on.
+ * @param array $options Options passed from Model::save().
  * @return boolean
+ * @see Model::save()
  */
-	public function beforeValidate(Model $Model) {
+	public function beforeValidate(Model $Model, $options = array()) {
 		unset($this->runtime[$Model->alias]['beforeSave']);
 		$this->_setRuntimeData($Model);
 		return true;
@@ -335,7 +337,9 @@ class TranslateBehavior extends ModelBehavior {
  * disabled. Or the runtime data hasn't been set yet.
  *
  * @param Model $Model Model save was called on.
+ * @param array $options Options passed from Model::save().
  * @return boolean true.
+ * @see Model::save()
  */
 	public function beforeSave(Model $Model, $options = array()) {
 		if (isset($options['validate']) && !$options['validate']) {

+ 3 - 1
lib/Cake/Model/Behavior/TreeBehavior.php

@@ -175,9 +175,11 @@ class TreeBehavior extends ModelBehavior {
  *
  * @since         1.2
  * @param Model $Model Model instance
+ * @param array $options Options passed from Model::save().
  * @return boolean true to continue, false to abort the save
+ * @see Model::save()
  */
-	public function beforeSave(Model $Model) {
+	public function beforeSave(Model $Model, $options = array()) {
 		extract($this->settings[$Model->alias]);
 
 		$this->_addToWhitelist($Model, array($left, $right));

+ 0 - 1
lib/Cake/Model/Datasource/Database/Postgres.php

@@ -346,7 +346,6 @@ class Postgres extends DboSource {
 			$this->cacheSources = $cache;
 		}
 		if ($this->execute('DELETE FROM ' . $this->fullTableName($table))) {
-			$schema = $this->config['schema'];
 			if (isset($this->_sequenceMap[$table]) && $reset != true) {
 				foreach ($this->_sequenceMap[$table] as $sequence) {
 					list($schema, $sequence) = explode('.', $sequence);

+ 7 - 5
lib/Cake/Model/Datasource/DboSource.php

@@ -952,14 +952,14 @@ class DboSource extends DataSource {
 
 		if ($quote) {
 			if ($schema && !empty($schemaName)) {
-				if (false == strstr($table, '.')) {
+				if (strstr($table, '.') === false) {
 					return $this->name($schemaName) . '.' . $this->name($table);
 				}
 			}
 			return $this->name($table);
 		}
 		if ($schema && !empty($schemaName)) {
-			if (false == strstr($table, '.')) {
+			if (strstr($table, '.') === false) {
 				return $schemaName . '.' . $table;
 			}
 		}
@@ -1065,7 +1065,7 @@ class DboSource extends DataSource {
 					if ($bypass) {
 						$assocData['fields'] = false;
 					}
-					if (true === $this->generateAssociationQuery($model, $linkModel, $type, $assoc, $assocData, $queryData, $external, $null)) {
+					if ($this->generateAssociationQuery($model, $linkModel, $type, $assoc, $assocData, $queryData, $external, $null) === true) {
 						$linkedModels[$type . '/' . $assoc] = true;
 					}
 				}
@@ -1730,8 +1730,10 @@ class DboSource extends DataSource {
  * @return string
  */
 	public function renderJoinStatement($data) {
-		extract($data);
-		return trim("{$type} JOIN {$table} {$alias} ON ({$conditions})");
+		if (strtoupper($data['type']) === 'CROSS') {
+			return "{$data['type']} JOIN {$data['table']} {$data['alias']}";
+		}
+		return trim("{$data['type']} JOIN {$data['table']} {$data['alias']} ON ({$data['conditions']})");
 	}
 
 /**

+ 2 - 1
lib/Cake/Model/Model.php

@@ -3460,9 +3460,10 @@ class Model extends Object implements CakeEventListener {
  * Called during validation operations, before validation. Please note that custom
  * validation rules can be defined in $validate.
  *
- * @param array $options Options passed from model::save(), see $options of model::save().
+ * @param array $options Options passed from Model::save().
  * @return boolean True if validate operation should continue, false to abort
  * @link http://book.cakephp.org/2.0/en/models/callback-methods.html#beforevalidate
+ * @see Model::save()
  */
 	public function beforeValidate($options = array()) {
 		return true;

+ 6 - 2
lib/Cake/Model/ModelBehavior.php

@@ -141,9 +141,11 @@ class ModelBehavior extends Object {
  * will allow you to make the validation fail.
  *
  * @param Model $model Model using this behavior
+ * @param array $options Options passed from Model::save().
  * @return mixed False or null will abort the operation. Any other result will continue.
+ * @see Model::save()
  */
-	public function beforeValidate(Model $model) {
+	public function beforeValidate(Model $model, $options = array()) {
 		return true;
 	}
 
@@ -163,9 +165,11 @@ class ModelBehavior extends Object {
  * will abort the save operation.
  *
  * @param Model $model Model using this behavior
+ * @param array $options Options passed from Model::save().
  * @return mixed False if the operation should abort. Any other result will continue.
+ * @see Model::save()
  */
-	public function beforeSave(Model $model) {
+	public function beforeSave(Model $model, $options = array()) {
 		return true;
 	}
 

+ 11 - 7
lib/Cake/Test/Case/Model/BehaviorCollectionTest.php

@@ -104,10 +104,12 @@ class TestBehavior extends ModelBehavior {
 /**
  * beforeSave method
  *
- * @param Model $model
- * @return void
+ * @param Model $model Model using this behavior
+ * @param array $options Options passed from Model::save().
+ * @return mixed False if the operation should abort. Any other result will continue.
+ * @see Model::save()
  */
-	public function beforeSave(Model $model) {
+	public function beforeSave(Model $model, $options = array()) {
 		$settings = $this->settings[$model->alias];
 		if (!isset($settings['beforeSave']) || $settings['beforeSave'] === 'off') {
 			return parent::beforeSave($model);
@@ -155,12 +157,14 @@ class TestBehavior extends ModelBehavior {
 	}
 
 /**
- * beforeValidate method
+ * beforeValidate Callback
  *
- * @param Model $model
- * @return void
+ * @param Model $Model Model invalidFields was called on.
+ * @param array $options Options passed from Model::save().
+ * @return boolean
+ * @see Model::save()
  */
-	public function beforeValidate(Model $model) {
+	public function beforeValidate(Model $model, $options = array()) {
 		$settings = $this->settings[$model->alias];
 		if (!isset($settings['validate']) || $settings['validate'] === 'off') {
 			return parent::beforeValidate($model);

+ 7 - 1
lib/Cake/Test/Case/Model/Datasource/DboSourceTest.php

@@ -1111,9 +1111,15 @@ class DboSourceTest extends CakeTestCase {
  *
  * @return array
  */
-	public static function joinStatements($schema) {
+	public static function joinStatements() {
 		return array(
 			array(array(
+				'type' => 'CROSS',
+				'alias' => 'PostsTag',
+				'table' => 'posts_tags',
+				'conditions' => array('1 = 1')
+			), 'CROSS JOIN cakephp.posts_tags AS PostsTag'),
+			array(array(
 				'type' => 'LEFT',
 				'alias' => 'PostsTag',
 				'table' => 'posts_tags',

+ 6 - 2
lib/Cake/Test/Case/Model/models.php

@@ -2039,7 +2039,9 @@ class CallbackPostTestModel extends CakeTestModel {
 /**
  * beforeValidate callback
  *
- * @return boolean
+ * @param array $options Options passed from Model::save().
+ * @return boolean True if validate operation should continue, false to abort
+ * @see Model::save()
  */
 	public function beforeValidate($options = array()) {
 		return $this->beforeValidateReturn;
@@ -4980,7 +4982,9 @@ class CustomArticle extends AppModel {
 /**
  * Alters title data
  *
- * @return void
+ * @param array $options Options passed from Model::save().
+ * @return boolean True if validate operation should continue, false to abort
+ * @see Model::save()
  */
 	public function beforeValidate($options = array()) {
 		$this->data[$this->alias]['title'] = 'foo';

+ 20 - 0
lib/Cake/Test/Case/View/Helper/FormHelperTest.php

@@ -4078,6 +4078,26 @@ class FormHelperTest extends CakeTestCase {
 	}
 
 /**
+ * Test that radio() accepts an array for label
+ *
+ * @return void
+ */
+	public function testRadioLabelArray() {
+		$result = $this->Form->input('Model.field', array(
+			'type' => 'radio',
+			'legend' => false,
+			'label' => array(
+				'class' => 'checkbox float-left',
+			),
+			'options' => array('1' => 'Option A', '2' => 'Option B.')
+		));
+		$this->assertTextContains(
+			'<label for="ModelField1" class="checkbox float-left">Option A</label>',
+			$result
+		);
+	}
+
+/**
  * testSelect method
  *
  * Test select element generation.

+ 1 - 1
lib/Cake/Test/Case/View/Helper/RssHelperTest.php

@@ -610,7 +610,7 @@ class RssHelperTest extends CakeTestCase {
 
 		$this->assertTrue($File->write('123'), 'Could not write to ' . $tmpFile);
 
-		if (50300 <= PHP_VERSION_ID) {
+		if (PHP_VERSION_ID >= 50300) {
 			clearstatcache(true, $tmpFile);
 		} else {
 			clearstatcache();

+ 170 - 46
lib/Cake/Test/Case/View/JsonViewTest.php

@@ -36,78 +36,166 @@ class JsonViewTest extends CakeTestCase {
 	}
 
 /**
- * testRenderWithoutView method
+ * Generates testRenderWithoutView data.
+ *
+ * Note: array($data, $serialize, expected)
  *
  * @return void
  */
-	public function testRenderWithoutView() {
-		$Request = new CakeRequest();
-		$Response = new CakeResponse();
-		$Controller = new Controller($Request, $Response);
-		$data = array('user' => 'fake', 'list' => array('item1', 'item2'));
-		$Controller->set(array('data' => $data, '_serialize' => 'data'));
-		$View = new JsonView($Controller);
-		$output = $View->render(false);
+	public static function renderWithoutViewProvider() {
+		return array(
+			// Test render with a valid string in _serialize.
+			array(
+				array('data' => array('user' => 'fake', 'list' => array('item1', 'item2'))),
+				'data',
+				json_encode(array('user' => 'fake', 'list' => array('item1', 'item2')))
+			),
 
-		$this->assertSame(json_encode($data), $output);
-		$this->assertSame('application/json', $Response->type());
-	}
+			// Test render with a string with an invalid key in _serialize.
+			array(
+				array('data' => array('user' => 'fake', 'list' => array('item1', 'item2'))),
+				'no_key',
+				json_encode(null)
+			),
 
-/**
- * Test that rendering with _serialize does not load helpers
- *
- * @return void
- */
-	public function testRenderSerializeNoHelpers() {
-		$Request = new CakeRequest();
-		$Response = new CakeResponse();
-		$Controller = new Controller($Request, $Response);
-		$Controller->helpers = array('Html');
-		$Controller->set(array(
-			'_serialize' => 'tags',
-			'tags' => array('cakephp', 'framework')
-		));
-		$View = new JsonView($Controller);
-		$View->render();
-		$this->assertFalse(isset($View->Html), 'No helper loaded.');
+			// Test render with a valid array in _serialize.
+			array(
+				array('no' => 'nope', 'user' => 'fake', 'list' => array('item1', 'item2')),
+				array('no', 'user'),
+				json_encode(array('no' => 'nope', 'user' => 'fake'))
+			),
+
+			// Test render with an empty array in _serialize.
+			array(
+				array('no' => 'nope', 'user' => 'fake', 'list' => array('item1', 'item2')),
+				array(),
+				json_encode(null)
+			),
+
+			// Test render with a valid array with an invalid key in _serialize.
+			array(
+				array('no' => 'nope', 'user' => 'fake', 'list' => array('item1', 'item2')),
+				array('no', 'user', 'no_key'),
+				json_encode(array('no' => 'nope', 'user' => 'fake'))
+			),
+
+			// Test render with a valid array with only an invalid key in _serialize.
+			array(
+				array('no' => 'nope', 'user' => 'fake', 'list' => array('item1', 'item2')),
+				array('no_key'),
+				json_encode(null)
+			),
+
+			// Test render with Null in _serialize (unset).
+			array(
+				array('no' => 'nope', 'user' => 'fake', 'list' => array('item1', 'item2')),
+				null,
+				null
+			),
+
+			// Test render with False in _serialize.
+			array(
+				array('no' => 'nope', 'user' => 'fake', 'list' => array('item1', 'item2')),
+				false,
+				json_encode(null)
+			),
+
+			// Test render with True in _serialize.
+			array(
+				array('no' => 'nope', 'user' => 'fake', 'list' => array('item1', 'item2')),
+				true,
+				json_encode(null)
+			),
+
+			// Test render with empty string in _serialize.
+			array(
+				array('no' => 'nope', 'user' => 'fake', 'list' => array('item1', 'item2')),
+				'',
+				json_encode(null)
+			),
+
+			// Test render with a valid array in _serialize and alias.
+			array(
+				array('original_name' => 'my epic name', 'user' => 'fake', 'list' => array('item1', 'item2')),
+				array('new_name' => 'original_name', 'user'),
+				json_encode(array('new_name' => 'my epic name', 'user' => 'fake'))
+			),
+
+			// Test render with an a valid array in _serialize and alias of a null value.
+			array(
+				array('null' => null),
+				array('null'),
+				json_encode(array('null' => null))
+			),
+
+			// Test render with a False value to be serialized.
+			array(
+				array('false' => false),
+				'false',
+				json_encode(false)
+			),
+
+			// Test render with a True value to be serialized.
+			array(
+				array('true' => true),
+				'true',
+				json_encode(true)
+			),
+
+			// Test render with an empty string value to be serialized.
+			array(
+				array('empty' => ''),
+				'empty',
+				json_encode('')
+			),
+
+			// Test render with a zero value to be serialized.
+			array(
+				array('zero' => 0),
+				'zero',
+				json_encode(0)
+			),
+		);
 	}
 
 /**
- * Test render with an array in _serialize
+ * Test render with a valid string in _serialize.
  *
+ * @dataProvider renderWithoutViewProvider
  * @return void
  */
-	public function testRenderWithoutViewMultiple() {
+	public function testRenderWithoutView($data, $serialize, $expected) {
 		$Request = new CakeRequest();
 		$Response = new CakeResponse();
 		$Controller = new Controller($Request, $Response);
-		$data = array('no' => 'nope', 'user' => 'fake', 'list' => array('item1', 'item2'));
+
 		$Controller->set($data);
-		$Controller->set('_serialize', array('no', 'user'));
+		$Controller->set('_serialize', $serialize);
 		$View = new JsonView($Controller);
 		$output = $View->render(false);
 
-		$this->assertSame(json_encode(array('no' => $data['no'], 'user' => $data['user'])), $output);
-		$this->assertSame('application/json', $Response->type());
+		$this->assertSame($expected, $output);
 	}
 
 /**
- * Test render with an array in _serialize and alias
+ * Test that rendering with _serialize does not load helpers.
  *
  * @return void
  */
-	public function testRenderWithoutViewMultipleAndAlias() {
+	public function testRenderSerializeNoHelpers() {
 		$Request = new CakeRequest();
 		$Response = new CakeResponse();
 		$Controller = new Controller($Request, $Response);
-		$data = array('original_name' => 'my epic name', 'user' => 'fake', 'list' => array('item1', 'item2'));
-		$Controller->set($data);
-		$Controller->set('_serialize', array('new_name' => 'original_name', 'user'));
+
+		$Controller->helpers = array('Html');
+		$Controller->set(array(
+			'tags' => array('cakephp', 'framework'),
+			'_serialize' => 'tags'
+		));
 		$View = new JsonView($Controller);
-		$output = $View->render(false);
+		$View->render();
 
-		$this->assertSame(json_encode(array('new_name' => $data['original_name'], 'user' => $data['user'])), $output);
-		$this->assertSame('application/json', $Response->type());
+		$this->assertFalse(isset($View->Html), 'No helper loaded.');
 	}
 
 /**
@@ -119,8 +207,13 @@ class JsonViewTest extends CakeTestCase {
 		$Request = new CakeRequest();
 		$Response = new CakeResponse();
 		$Controller = new Controller($Request, $Response);
+
 		$data = array('user' => 'fake', 'list' => array('item1', 'item2'));
-		$Controller->set(array('data' => $data, '_serialize' => 'data', '_jsonp' => true));
+		$Controller->set(array(
+			'data' => $data,
+			'_serialize' => 'data',
+			'_jsonp' => true
+		));
 		$View = new JsonView($Controller);
 		$output = $View->render(false);
 
@@ -141,7 +234,7 @@ class JsonViewTest extends CakeTestCase {
 	}
 
 /**
- * testRenderWithView method
+ * Test render with a View file specified.
  *
  * @return void
  */
@@ -149,6 +242,38 @@ class JsonViewTest extends CakeTestCase {
 		App::build(array(
 			'View' => array(CAKE . 'Test' . DS . 'test_app' . DS . 'View' . DS)
 		));
+		$Request = new CakeRequest();
+		$Response = new CakeResponse();
+		$Controller = new Controller($Request, $Response);
+		$Controller->name = $Controller->viewPath = 'Posts';
+
+		$data = array(
+			'User' => array(
+				'username' => 'fake'
+			),
+			'Item' => array(
+				array('name' => 'item1'),
+				array('name' => 'item2')
+			)
+		);
+		$Controller->set('user', $data);
+		$View = new JsonView($Controller);
+		$output = $View->render('index');
+
+		$expected = json_encode(array('user' => 'fake', 'list' => array('item1', 'item2'), 'paging' => null));
+		$this->assertSame($expected, $output);
+		$this->assertSame('application/json', $Response->type());
+	}
+
+/**
+ * Test render with a View file specified and named parameters.
+ *
+ * @return void
+ */
+	public function testRenderWithViewAndNamed() {
+		App::build(array(
+			'View' => array(CAKE . 'Test' . DS . 'test_app' . DS . 'View' . DS)
+		));
 		$Request = new CakeRequest(null, false);
 		$Request->params['named'] = array('page' => 2);
 		$Response = new CakeResponse();
@@ -183,5 +308,4 @@ class JsonViewTest extends CakeTestCase {
 		$this->assertSame($expected, $output);
 		$this->assertSame('application/javascript', $Response->type());
 	}
-
 }

+ 1 - 1
lib/Cake/TestSuite/Reporter/CakeHtmlReporter.php

@@ -97,7 +97,7 @@ class CakeHtmlReporter extends CakeBaseReporter {
 			$urlExtra = '&plugin=' . $plugin;
 		}
 
-		if (1 > count($testCases)) {
+		if (count($testCases) < 1) {
 			$buffer .= "<strong>EMPTY</strong>";
 		}
 

+ 1 - 1
lib/Cake/TestSuite/Reporter/CakeTextReporter.php

@@ -161,7 +161,7 @@ class CakeTextReporter extends CakeBaseReporter {
 			$urlExtra = '&plugin=' . $plugin;
 		}
 
-		if (1 > count($testCases)) {
+		if (count($testCases) < 1) {
 			$buffer .= 'EMPTY';
 			echo $buffer;
 		}

+ 1 - 1
lib/Cake/Utility/Set.php

@@ -591,7 +591,7 @@ class Set {
 					}
 				}
 				return $tmp;
-			} elseif (false !== strpos($key, '{') && false !== strpos($key, '}')) {
+			} elseif (strpos($key, '{') !== false && strpos($key, '}') !== false) {
 				$pattern = substr($key, 1, -1);
 
 				foreach ($data as $j => $val) {

+ 4 - 1
lib/Cake/View/JsonView.php

@@ -135,8 +135,11 @@ class JsonView extends View {
 				if (is_numeric($alias)) {
 					$alias = $key;
 				}
-				$data[$alias] = $this->viewVars[$key];
+				if (array_key_exists($key, $this->viewVars)) {
+					$data[$alias] = $this->viewVars[$key];
+				}
 			}
+			$data = !empty($data) ? $data : null;
 		} else {
 			$data = isset($this->viewVars[$serialize]) ? $this->viewVars[$serialize] : null;
 		}