@@ -67,7 +67,11 @@ discard block |
||
67 | 67 | $first = true; |
68 | 68 | |
69 | 69 | foreach ($owningAssoc['targetToSourceKeyColumns'] as $targetColumn => $sourceColumn) { |
70 | - if ($first) $first = false; else $sql .= ' AND '; |
|
70 | + if ($first) { |
|
71 | + $first = false; |
|
72 | + } else { |
|
73 | + $sql .= ' AND '; |
|
74 | + } |
|
71 | 75 | |
72 | 76 | $sql .= $targetTableAlias . '.' . $sourceColumn |
73 | 77 | . ' = ' |
@@ -93,7 +97,11 @@ discard block |
||
93 | 97 | $first = true; |
94 | 98 | |
95 | 99 | foreach ($joinColumns as $joinColumn) { |
96 | - if ($first) $first = false; else $sql .= ' AND '; |
|
100 | + if ($first) { |
|
101 | + $first = false; |
|
102 | + } else { |
|
103 | + $sql .= ' AND '; |
|
104 | + } |
|
97 | 105 | |
98 | 106 | $sourceColumnName = $quoteStrategy->getColumnName( |
99 | 107 | $class->fieldNames[$joinColumn['referencedColumnName']], $class, $platform |
@@ -444,7 +444,9 @@ discard block |
||
444 | 444 | foreach ($dqlAliases as $dqlAlias) { |
445 | 445 | $class = $this->queryComponents[$dqlAlias]['metadata']; |
446 | 446 | |
447 | - if ( ! $class->isInheritanceTypeSingleTable()) continue; |
|
447 | + if ( ! $class->isInheritanceTypeSingleTable()) { |
|
448 | + continue; |
|
449 | + } |
|
448 | 450 | |
449 | 451 | $conn = $this->em->getConnection(); |
450 | 452 | $values = array(); |
@@ -783,7 +785,9 @@ discard block |
||
783 | 785 | |
784 | 786 | foreach ($subClass->associationMappings as $assoc) { |
785 | 787 | // Skip if association is inherited |
786 | - if (isset($assoc['inherited'])) continue; |
|
788 | + if (isset($assoc['inherited'])) { |
|
789 | + continue; |
|
790 | + } |
|
787 | 791 | |
788 | 792 | if ($assoc['isOwningSide'] && $assoc['type'] & ClassMetadata::TO_ONE) { |
789 | 793 | $targetClass = $this->em->getClassMetadata($assoc['targetEntity']); |
@@ -227,7 +227,9 @@ |
||
227 | 227 | $found = false; |
228 | 228 | |
229 | 229 | foreach (array_merge($this->metaMappings, $this->fieldMappings) as $columnName => $columnFieldName) { |
230 | - if ( ! ($columnFieldName === $fieldName && $this->columnOwnerMap[$columnName] === $alias)) continue; |
|
230 | + if ( ! ($columnFieldName === $fieldName && $this->columnOwnerMap[$columnName] === $alias)) { |
|
231 | + continue; |
|
232 | + } |
|
231 | 233 | |
232 | 234 | $this->addIndexByColumn($alias, $columnName); |
233 | 235 | $found = true; |
@@ -1524,11 +1524,21 @@ |
||
1524 | 1524 | if ($associationMapping['cascade']) { |
1525 | 1525 | $cascades = array(); |
1526 | 1526 | |
1527 | - if ($associationMapping['isCascadePersist']) $cascades[] = '"persist"'; |
|
1528 | - if ($associationMapping['isCascadeRemove']) $cascades[] = '"remove"'; |
|
1529 | - if ($associationMapping['isCascadeDetach']) $cascades[] = '"detach"'; |
|
1530 | - if ($associationMapping['isCascadeMerge']) $cascades[] = '"merge"'; |
|
1531 | - if ($associationMapping['isCascadeRefresh']) $cascades[] = '"refresh"'; |
|
1527 | + if ($associationMapping['isCascadePersist']) { |
|
1528 | + $cascades[] = '"persist"'; |
|
1529 | + } |
|
1530 | + if ($associationMapping['isCascadeRemove']) { |
|
1531 | + $cascades[] = '"remove"'; |
|
1532 | + } |
|
1533 | + if ($associationMapping['isCascadeDetach']) { |
|
1534 | + $cascades[] = '"detach"'; |
|
1535 | + } |
|
1536 | + if ($associationMapping['isCascadeMerge']) { |
|
1537 | + $cascades[] = '"merge"'; |
|
1538 | + } |
|
1539 | + if ($associationMapping['isCascadeRefresh']) { |
|
1540 | + $cascades[] = '"refresh"'; |
|
1541 | + } |
|
1532 | 1542 | |
1533 | 1543 | if (count($cascades) === 5) { |
1534 | 1544 | $cascades = array('"all"'); |