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

Adding docs + exceptions for name translation.

Browse source 
Closes #2367
This commit is contained in:
mark_story 2011-12-30 20:39:04 -05:00
parent a4e3790196
commit b5f918765e
2 changed files with 21 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

10
lib/Cake/Model/Behavior/TranslateBehavior.php
View file

@ -368,7 +368,10 @@ class TranslateBehavior extends ModelBehavior {
/**
* Bind translation for fields, optionally with hasMany association for
* fake field
* fake field.
*
* *Note* You should avoid binding translations that overlap existing model properties.
* This can cause un-expected and un-desirable behavior.
*
* @param Model $model instance of model
* @param string|array $fields string with field or array(field1, field2=>AssocName, field3)
@ -391,6 +394,11 @@ class TranslateBehavior extends ModelBehavior {
$field = $key;
$association = $value;
}
if ($field === 'name') {
throw new CakeException(
__d('cake_dev', 'You cannot bind a translation named "name".')
);
}
if (array_key_exists($field, $this->settings[$model->alias])) {
unset($this->settings[$model->alias][$field]);

12
lib/Cake/Test/Case/Model/Behavior/TranslateBehaviorTest.php
View file

@ -868,4 +868,16 @@ class TranslateBehaviorTest extends CakeTestCase {
$this->assertFalse($result);
}
/**
* Test that an exception is raised when you try to over-write the name attribute.
*
* @expectedException CakeException
* @return void
*/
public function testExceptionOnNameTranslation() {
$this->loadFixtures('Translate', 'TranslatedItem');
$TestModel = new TranslatedItem();
$TestModel->bindTranslation(array('name' => 'name'));
}
}