AmsTaFFix / extjs-bundle

TpgExtjsBundle.php

Braces +12 added lines, -8 removed lines

@@ -17,18 +17,22 @@
 
         $builder->addCompilerPass(new LazyServiceMapPass('tpg_extjs.serialization_visitor', 'format',
             function(ContainerBuilder $container, Definition $def) {
-                if ($container->hasDefinition("tpg_extjs.orm_serializer"))
-                    $container->getDefinition('tpg_extjs.orm_serializer')->replaceArgument(3, $def);
-                if ($container->hasDefinition("tpg_extjs.odm_serializer"))
-                    $container->getDefinition('tpg_extjs.odm_serializer')->replaceArgument(3, $def);
+                if ($container->hasDefinition("tpg_extjs.orm_serializer")) {
+                                    $container->getDefinition('tpg_extjs.orm_serializer')->replaceArgument(3, $def);
+                }
+                if ($container->hasDefinition("tpg_extjs.odm_serializer")) {
+                                    $container->getDefinition('tpg_extjs.odm_serializer')->replaceArgument(3, $def);
+                }
             }
         ));
         $builder->addCompilerPass(new LazyServiceMapPass('tpg_extjs.deserialization_visitor', 'format',
             function(ContainerBuilder $container, Definition $def) {
-                if ($container->hasDefinition("tpg_extjs.orm_serializer"))
-                    $container->getDefinition('tpg_extjs.orm_serializer')->replaceArgument(4, $def);
-                if ($container->hasDefinition("tpg_extjs.odm_serializer"))
-                    $container->getDefinition('tpg_extjs.odm_serializer')->replaceArgument(4, $def);
+                if ($container->hasDefinition("tpg_extjs.orm_serializer")) {
+                                    $container->getDefinition('tpg_extjs.orm_serializer')->replaceArgument(4, $def);
+                }
+                if ($container->hasDefinition("tpg_extjs.odm_serializer")) {
+                                    $container->getDefinition('tpg_extjs.odm_serializer')->replaceArgument(4, $def);
+                }
             }
         ));
 

Service/GeneratorService.php

Braces +20 added lines, -8 removed lines

@@ -99,8 +99,12 @@ 
                 $structure['proxy'] = array(
                     'type'=>$classModelProxyAnnotation->name,
                 ) + $classModelProxyAnnotation->option;
-                if ($classModelProxyAnnotation->reader != array()) $structure['proxy']['reader'] = $classModelProxyAnnotation->reader;
-                if ($classModelProxyAnnotation->writer != array()) $structure['proxy']['writer'] = $classModelProxyAnnotation->writer;
+                if ($classModelProxyAnnotation->reader != array()) {
+                    $structure['proxy']['reader'] = $classModelProxyAnnotation->reader;
+                }
+                if ($classModelProxyAnnotation->writer != array()) {
+                    $structure['proxy']['writer'] = $classModelProxyAnnotation->writer;
+                }
             }
             /** @var $classExclusionPolicy ExclusionPolicy */
             $classExclusionPolicy = $this->annoReader->getClassAnnotation($classRef, 'JMS\Serializer\Annotation\ExclusionPolicy');
@@ -223,8 +227,9 @@ 
                     $association['name'] = $property->getName();
                     $association['model'] = $this->getModelName($annotation->targetEntity);
                     $association['entity'] = $annotation->targetEntity;
-                    if (!isset($association['key']) || empty($association['key']))
-                        $association['key'] = $this->tryToGetJoinColumnNameOfMappedBy($annotation);
+                    if (!isset($association['key']) || empty($association['key'])) {
+                                            $association['key'] = $this->tryToGetJoinColumnNameOfMappedBy($annotation);
+                    }
                     break;
                 case 'Doctrine\ORM\Mapping\OneToMany':
                     $association['type'] = substr(get_class($annotation), 21);
@@ -359,8 +364,11 @@ 
     protected function getAnnotation($entity, $property, $annotation) {
         $classRef = new \ReflectionClass($entity);
         $propertyRef = $classRef->getProperty($property);
-        if ($propertyRef) return $this->annoReader->getPropertyAnnotation($propertyRef, $annotation);
-        else return null;
+        if ($propertyRef) {
+            return $this->annoReader->getPropertyAnnotation($propertyRef, $annotation);
+        } else {
+            return null;
+        }
     }
 
     /**
@@ -520,9 +528,13 @@ 
         $secondList = $list;
         $duplicateList = array();
         foreach ($list as $index=>$row) {
-            if (in_array($index, $duplicateList)) continue;
+            if (in_array($index, $duplicateList)) {
+                continue;
+            }
             foreach ($secondList as $index2 => $row2) {
-                if ($index === $index2) continue;
+                if ($index === $index2) {
+                    continue;
+                }
                 if ($row == $row2) {
                     $duplicateList[] = $index2;
                 }

Resources/skeleton/controller/Controller.php

Braces +6 added lines, -2 removed lines

@@ -26,7 +26,8 @@ 
 {%- if trait %}
 
 trait
-{%- else %}
+{%- else {
+    %}
 
 class
 {%- endif %}
@@ -48,6 +49,7 @@ 
      */
     public function get{{ controller|capitalize }}Action($id) {
         $manager = $this->get("{{ manager }}");
+}
         $entity = $manager->getRepository('{{ entity_bundle }}:{{ entity }}')->find($id);
         $view = View::create($entity, 200)->setSerializationContext($this->getSerializerContext(array("get")));;
         return $this->handleView($view);
@@ -249,7 +251,9 @@ 
             array(\JMS\Serializer\Exclusion\GroupsExclusionStrategy::DEFAULT_GROUP),
             $groups
         ));
-        if ($version !== null) $serializeContext->setVersion($version);
+        if ($version !== null) {
+            $serializeContext->setVersion($version);
+        }
         return $serializeContext;
     }
 }

Listener/DeserializationAssociateRelationship.php

Braces +3 added lines, -1 removed lines

@@ -27,7 +27,9 @@
         $classMetadata = $e->getContext()->getMetadataFactory()->getMetadataForClass($className);
         /** @var PropertyMetadata $propertyMetadata */
         foreach ($classMetadata->propertyMetadata as $propertyMetadata) {
-            if ($propertyMetadata->reflection === null) continue;
+            if ($propertyMetadata->reflection === null) {
+                continue;
+            }
             /** @var JoinColumn $joinColumnAnnotation */
             $joinColumnAnnotation = $this->reader->getPropertyAnnotation(
                 $propertyMetadata->reflection,

Annotation/ModelProxy.php

Braces +12 added lines, -4 removed lines

@@ -25,10 +25,18 @@
                 "writeRecordId"=>false,
             );
         } else {
-            if (isset($values['name'])) $this->name = $values['name'];
+            if (isset($values['name'])) {
+                $this->name = $values['name'];
+            }
+        }
+        if (isset($values['option'])) {
+            $this->option = array_merge($this->option, $values['option']);
+        }
+        if (isset($values['writer'])) {
+            $this->writer = array_merge($this->writer, $values['writer']);
+        }
+        if (isset($values['reader'])) {
+            $this->reader = $values['reader'];
         }
-        if (isset($values['option'])) $this->option = array_merge($this->option, $values['option']);
-        if (isset($values['writer'])) $this->writer = array_merge($this->writer, $values['writer']);
-        if (isset($values['reader'])) $this->reader = $values['reader'];
     }
 }
\ No newline at end of file