[Bridge] [Doctrine] fixed coding conventions.

Hugo Hamon · Hugo Hamon · commit 240796e9f3ce · 2011-12-02T23:11:12.000+01:00
diff --git a/src/Symfony/Bridge/Doctrine/Form/ChoiceList/EntityChoiceList.php b/src/Symfony/Bridge/Doctrine/Form/ChoiceList/EntityChoiceList.php
@@ -173,7 +173,7 @@ private function loadEntities($entities, $group = null)
                     throw new FormException('Entities passed to the choice field must have a "__toString()" method defined (or you can also override the "property" option).');
                 }
 
-                $value = (string)$entity;
+                $value = (string) $entity;
             }
 
             if (count($this->identifier) > 1) {
diff --git a/src/Symfony/Bridge/Doctrine/Validator/Constraints/UniqueEntityValidator.php b/src/Symfony/Bridge/Doctrine/Validator/Constraints/UniqueEntityValidator.php
@@ -48,10 +48,10 @@ public function isValid($entity, Constraint $constraint)
             throw new UnexpectedTypeException($constraint->fields, 'array');
         }
 
-        $fields = (array)$constraint->fields;
+        $fields = (array) $constraint->fields;
 
-        if (count($fields) == 0) {
-            throw new ConstraintDefinitionException("At least one field has to be specified.");
+        if (0 === count($fields)) {
+            throw new ConstraintDefinitionException('At least one field has to be specified.');
         }
 
         $em = $this->registry->getEntityManager($constraint->em);
@@ -62,21 +62,21 @@ public function isValid($entity, Constraint $constraint)
         $criteria = array();
         foreach ($fields as $fieldName) {
             if (!isset($class->reflFields[$fieldName])) {
-                throw new ConstraintDefinitionException("Only field names mapped by Doctrine can be validated for uniqueness.");
+                throw new ConstraintDefinitionException('Only field names mapped by Doctrine can be validated for uniqueness.');
             }
 
             $criteria[$fieldName] = $class->reflFields[$fieldName]->getValue($entity);
 
-            if ($criteria[$fieldName] === null) {
+            if (null === $criteria[$fieldName]) {
                 return true;
-            } else if (isset($class->associationMappings[$fieldName])) {
+            }
+
+            if (isset($class->associationMappings[$fieldName])) {
                 $relatedClass = $em->getClassMetadata($class->associationMappings[$fieldName]['targetEntity']);
                 $relatedId = $relatedClass->getIdentifierValues($criteria[$fieldName]);
 
                 if (count($relatedId) > 1) {
-                    throw new ConstraintDefinitionException(
-                        "Associated entities are not allowed to have more than one identifier field to be " .
-                        "part of a unique constraint in: " . $class->name . "#" . $fieldName
+                    throw new ConstraintDefinitionException(sprintf('Associated entities are not allowed to have more than one identifier field to be part of a unique constraint in %s#%s.', $class->name, $fieldName));
                     );
                 }
                 $criteria[$fieldName] = array_pop($relatedId);
@@ -90,12 +90,12 @@ public function isValid($entity, Constraint $constraint)
          * which is the same as the entity being validated, the criteria is
          * unique.
          */
-        if (0 == count($result) || (1 == count($result) && $entity === $result[0])) {
+        if (0 === count($result) || (1 === count($result) && $entity === $result[0])) {
             return true;
         }
 
         $oldPath = $this->context->getPropertyPath();
-        $this->context->setPropertyPath( empty($oldPath) ? $fields[0] : $oldPath.".".$fields[0]);
+        $this->context->setPropertyPath( empty($oldPath) ? $fields[0] : $oldPath.'.'.$fields[0]);
         $this->context->addViolation($constraint->message, array(), $criteria[$fields[0]]);
         $this->context->setPropertyPath($oldPath);
 

Original file line number	Diff line number	Diff line change
`@@ -173,7 +173,7 @@ private function loadEntities($entities, $group = null)`
`173`	`173`	`throw new FormException('Entities passed to the choice field must have a "__toString()" method defined (or you can also override the "property" option).');`
`174`	`174`	`}`
`175`	`175`
`176`		`- $value = (string)$entity;`
	`176`	`+ $value = (string) $entity;`
`177`	`177`	`}`
`178`	`178`
`179`	`179`	`if (count($this->identifier) > 1) {`