diogo/symfony
Archived
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

[Form] Camelize 'add' and 'remove' methods in the PropertyPath

Browse Source 
This issue camelizes the 'add' and 'remove' methods,
as it is already done with the 'set' method.
This fixes a problem with properties like 'custom_messages',
where the 'add' and 'remove' methods are 'addCustom_message'
and 'removeCustom_message' instead of 'addCustomMessage'
and 'removeCustomMessage'.
This commit is contained in:
Albert Casademont 2012-06-27 19:26:17 +02:00
parent d0e154725c
commit 9fabb3dc2f
1 changed files with 3 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
src/Symfony/Component/Form/Util/PropertyPath.php
View File

@ -535,8 +535,8 @@ class PropertyPath implements \IteratorAggregate, PropertyPathInterface
private function findAdderAndRemover(\ReflectionClass $reflClass, $singular)
{
if (null !== $singular) {
$addMethod = 'add' . ucfirst($singular);
$removeMethod = 'remove' . ucfirst($singular);
$addMethod = 'add' . $this->camelize($singular);
$removeMethod = 'remove' . $this->camelize($singular);
if (!$this->isAccessible($reflClass, $addMethod, 1)) {
throw new InvalidPropertyException(sprintf(
@ -558,7 +558,7 @@ class PropertyPath implements \IteratorAggregate, PropertyPathInterface
}
// The plural form is the last element of the property path
$plural = ucfirst($this->elements[$this->length - 1]);
$plural = $this->camelize($this->elements[$this->length - 1]);
// Any of the two methods is required, but not yet known
$singulars = (array) FormUtil::singularify($plural);