doctrine / orm

lib/Doctrine/ORM/Query/AST/Functions/SizeFunction.php

Braces +10 added lines, -2 removed lines

@@ -67,7 +67,11 @@ 
             $first = true;
 
             foreach ($owningAssoc['targetToSourceKeyColumns'] as $targetColumn => $sourceColumn) {
-                if ($first) $first = false; else $sql .= ' AND ';
+                if ($first) {
+                    $first = false;
+                } else {
+                    $sql .= ' AND ';
+                }
 
                 $sql .= $targetTableAlias . '.' . $sourceColumn
                       . ' = '
@@ -93,7 +97,11 @@ 
             $first = true;
 
             foreach ($joinColumns as $joinColumn) {
-                if ($first) $first = false; else $sql .= ' AND ';
+                if ($first) {
+                    $first = false;
+                } else {
+                    $sql .= ' AND ';
+                }
 
                 $sourceColumnName = $quoteStrategy->getColumnName(
                     $class->fieldNames[$joinColumn['referencedColumnName']], $class, $platform

lib/Doctrine/ORM/Query/ResultSetMapping.php

Braces +3 added lines, -1 removed lines

@@ -227,7 +227,9 @@
         $found = false;
 
         foreach (array_merge($this->metaMappings, $this->fieldMappings) as $columnName => $columnFieldName) {
-            if ( ! ($columnFieldName === $fieldName && $this->columnOwnerMap[$columnName] === $alias)) continue;
+            if ( ! ($columnFieldName === $fieldName && $this->columnOwnerMap[$columnName] === $alias)) {
+                continue;
+            }
 
             $this->addIndexByColumn($alias, $columnName);
             $found = true;

lib/Doctrine/ORM/Query/SqlWalker.php

Braces +6 added lines, -2 removed lines

@@ -444,7 +444,9 @@ 
         foreach ($dqlAliases as $dqlAlias) {
             $class = $this->queryComponents[$dqlAlias]['metadata'];
 
-            if ( ! $class->isInheritanceTypeSingleTable()) continue;
+            if ( ! $class->isInheritanceTypeSingleTable()) {
+                continue;
+            }
 
             $conn   = $this->em->getConnection();
             $values = [];
@@ -782,7 +784,9 @@ 
 
                 foreach ($subClass->associationMappings as $assoc) {
                     // Skip if association is inherited
-                    if (isset($assoc['inherited'])) continue;
+                    if (isset($assoc['inherited'])) {
+                        continue;
+                    }
 
                     if ($assoc['isOwningSide'] && $assoc['type'] & ClassMetadata::TO_ONE) {
                         $targetClass = $this->em->getClassMetadata($assoc['targetEntity']);

lib/Doctrine/ORM/Tools/EntityGenerator.php

Braces +15 added lines, -5 removed lines

@@ -1525,11 +1525,21 @@
             if ($associationMapping['cascade']) {
                 $cascades = [];
 
-                if ($associationMapping['isCascadePersist']) $cascades[] = '"persist"';
-                if ($associationMapping['isCascadeRemove']) $cascades[] = '"remove"';
-                if ($associationMapping['isCascadeDetach']) $cascades[] = '"detach"';
-                if ($associationMapping['isCascadeMerge']) $cascades[] = '"merge"';
-                if ($associationMapping['isCascadeRefresh']) $cascades[] = '"refresh"';
+                if ($associationMapping['isCascadePersist']) {
+                    $cascades[] = '"persist"';
+                }
+                if ($associationMapping['isCascadeRemove']) {
+                    $cascades[] = '"remove"';
+                }
+                if ($associationMapping['isCascadeDetach']) {
+                    $cascades[] = '"detach"';
+                }
+                if ($associationMapping['isCascadeMerge']) {
+                    $cascades[] = '"merge"';
+                }
+                if ($associationMapping['isCascadeRefresh']) {
+                    $cascades[] = '"refresh"';
+                }
 
                 if (count($cascades) === 5) {
                     $cascades = ['"all"'];

tests/Doctrine/Tests/OrmFunctionalTestCase.php

Braces +6 added lines, -1 removed lines

@@ -787,7 +787,12 @@
             $queries = "";
             $last25queries = array_slice(array_reverse($this->_sqlLoggerStack->queries, true), 0, 25, true);
             foreach ($last25queries as $i => $query) {
-                $params = array_map(function($p) { if (is_object($p)) return get_class($p); else return var_export($p, true); }, $query['params'] ?: []
+                $params = array_map(function($p) { if (is_object($p)) {
+                    return get_class($p);
+                } else {
+                    return var_export($p, true);
+                }
+                }, $query['params'] ?: []
                 );
                 $queries .= $i.". SQL: '".$query['sql']."' Params: ".implode(", ", $params).PHP_EOL;
             }

tests/Doctrine/Tests/Mocks/HydratorMockStatement.php

Braces +3 added lines, -1 removed lines

@@ -46,7 +46,9 @@
     public function fetchColumn($columnNumber = 0)
     {
         $row = current($this->_resultSet);
-        if ( ! is_array($row)) return false;
+        if ( ! is_array($row)) {
+            return false;
+        }
         $val = array_shift($row);
         return $val !== null ? $val : false;
     }