@@ -29,7 +29,7 @@ |
||
29 | 29 | static public function matchingAssocationFieldRequiresObject($class, $associationName) |
30 | 30 | { |
31 | 31 | return new self(sprintf( |
32 | - "Cannot match on %s::%s with a non-object value. Matching objects by id is " . |
|
32 | + "Cannot match on %s::%s with a non-object value. Matching objects by id is ". |
|
33 | 33 | "not compatible with matching on an in-memory collection, which compares objects by reference.", |
34 | 34 | $class, $associationName |
35 | 35 | )); |
@@ -147,9 +147,9 @@ |
||
147 | 147 | return (bool) $persister->count($criteria); |
148 | 148 | } |
149 | 149 | |
150 | - /** |
|
151 | - * {@inheritdoc} |
|
152 | - */ |
|
150 | + /** |
|
151 | + * {@inheritdoc} |
|
152 | + */ |
|
153 | 153 | public function contains(PersistentCollection $collection, $element) |
154 | 154 | { |
155 | 155 | if ( ! $this->isValidEntityState($element)) { |
@@ -217,8 +217,8 @@ discard block |
||
217 | 217 | $parameters[] = $identifier[$sourceClass->getFieldForColumn($joinColumn['referencedColumnName'])]; |
218 | 218 | } |
219 | 219 | |
220 | - $statement = 'DELETE FROM ' . $this->quoteStrategy->getTableName($targetClass, $this->platform) |
|
221 | - . ' WHERE ' . implode(' = ? AND ', $columns) . ' = ?'; |
|
220 | + $statement = 'DELETE FROM '.$this->quoteStrategy->getTableName($targetClass, $this->platform) |
|
221 | + . ' WHERE '.implode(' = ? AND ', $columns).' = ?'; |
|
222 | 222 | |
223 | 223 | return $this->conn->executeUpdate($statement, $parameters); |
224 | 224 | } |
@@ -255,18 +255,18 @@ discard block |
||
255 | 255 | ]; |
256 | 256 | } |
257 | 257 | |
258 | - $statement = $this->platform->getCreateTemporaryTableSnippetSQL() . ' ' . $tempTable |
|
259 | - . ' (' . $this->platform->getColumnDeclarationListSQL($columnDefinitions) . ')'; |
|
258 | + $statement = $this->platform->getCreateTemporaryTableSnippetSQL().' '.$tempTable |
|
259 | + . ' ('.$this->platform->getColumnDeclarationListSQL($columnDefinitions).')'; |
|
260 | 260 | |
261 | 261 | $this->conn->executeUpdate($statement); |
262 | 262 | |
263 | 263 | // 2) Build insert table records into temporary table |
264 | 264 | $query = $this->em->createQuery( |
265 | - ' SELECT t0.' . implode(', t0.', $rootClass->getIdentifierFieldNames()) |
|
266 | - . ' FROM ' . $targetClass->name . ' t0 WHERE t0.' . $mapping['mappedBy'] . ' = :owner' |
|
265 | + ' SELECT t0.'.implode(', t0.', $rootClass->getIdentifierFieldNames()) |
|
266 | + . ' FROM '.$targetClass->name.' t0 WHERE t0.'.$mapping['mappedBy'].' = :owner' |
|
267 | 267 | )->setParameter('owner', $collection->getOwner()); |
268 | 268 | |
269 | - $statement = 'INSERT INTO ' . $tempTable . ' (' . $idColumnList . ') ' . $query->getSQL(); |
|
269 | + $statement = 'INSERT INTO '.$tempTable.' ('.$idColumnList.') '.$query->getSQL(); |
|
270 | 270 | $parameters = array_values($sourceClass->getIdentifierValues($collection->getOwner())); |
271 | 271 | $numDeleted = $this->conn->executeUpdate($statement, $parameters); |
272 | 272 | |
@@ -275,8 +275,8 @@ discard block |
||
275 | 275 | |
276 | 276 | foreach (array_reverse($classNames) as $className) { |
277 | 277 | $tableName = $this->quoteStrategy->getTableName($this->em->getClassMetadata($className), $this->platform); |
278 | - $statement = 'DELETE FROM ' . $tableName . ' WHERE (' . $idColumnList . ')' |
|
279 | - . ' IN (SELECT ' . $idColumnList . ' FROM ' . $tempTable . ')'; |
|
278 | + $statement = 'DELETE FROM '.$tableName.' WHERE ('.$idColumnList.')' |
|
279 | + . ' IN (SELECT '.$idColumnList.' FROM '.$tempTable.')'; |
|
280 | 280 | |
281 | 281 | $this->conn->executeUpdate($statement); |
282 | 282 | } |
@@ -47,17 +47,17 @@ |
||
47 | 47 | * Get all queued inserts. |
48 | 48 | * |
49 | 49 | * @return array |
50 | - */ |
|
50 | + */ |
|
51 | 51 | public function getInserts(); |
52 | 52 | |
53 | - /** |
|
54 | - * @TODO - It should not be here. |
|
55 | - * But its necessary since JoinedSubclassPersister#executeInserts invoke the root persister. |
|
56 | - * |
|
57 | - * Gets the INSERT SQL used by the persister to persist a new entity. |
|
58 | - * |
|
59 | - * @return string |
|
60 | - */ |
|
53 | + /** |
|
54 | + * @TODO - It should not be here. |
|
55 | + * But its necessary since JoinedSubclassPersister#executeInserts invoke the root persister. |
|
56 | + * |
|
57 | + * Gets the INSERT SQL used by the persister to persist a new entity. |
|
58 | + * |
|
59 | + * @return string |
|
60 | + */ |
|
61 | 61 | public function getInsertSQL(); |
62 | 62 | |
63 | 63 | /** |
@@ -57,7 +57,7 @@ |
||
57 | 57 | $rootClass = $this->em->getClassMetadata($this->class->rootEntityName); |
58 | 58 | $tableAlias = $this->getSQLTableAlias($rootClass->name); |
59 | 59 | |
60 | - // Append discriminator column |
|
60 | + // Append discriminator column |
|
61 | 61 | $discrColumn = $this->class->discriminatorColumn['name']; |
62 | 62 | $discrColumnType = $this->class->discriminatorColumn['type']; |
63 | 63 |
@@ -61,7 +61,7 @@ discard block |
||
61 | 61 | $discrColumn = $this->class->discriminatorColumn['name']; |
62 | 62 | $discrColumnType = $this->class->discriminatorColumn['type']; |
63 | 63 | |
64 | - $columnList[] = $tableAlias . '.' . $discrColumn; |
|
64 | + $columnList[] = $tableAlias.'.'.$discrColumn; |
|
65 | 65 | |
66 | 66 | $resultColumnName = $this->platform->getSQLResultCasing($discrColumn); |
67 | 67 | |
@@ -142,7 +142,7 @@ discard block |
||
142 | 142 | $conditionSql .= ' AND '; |
143 | 143 | } |
144 | 144 | |
145 | - return $conditionSql . $this->getSelectConditionDiscriminatorValueSQL(); |
|
145 | + return $conditionSql.$this->getSelectConditionDiscriminatorValueSQL(); |
|
146 | 146 | } |
147 | 147 | |
148 | 148 | /** |
@@ -156,7 +156,7 @@ discard block |
||
156 | 156 | $conditionSql .= ' AND '; |
157 | 157 | } |
158 | 158 | |
159 | - return $conditionSql . $this->getSelectConditionDiscriminatorValueSQL(); |
|
159 | + return $conditionSql.$this->getSelectConditionDiscriminatorValueSQL(); |
|
160 | 160 | } |
161 | 161 | |
162 | 162 | /** |
@@ -180,7 +180,7 @@ discard block |
||
180 | 180 | $discColumn = $this->class->discriminatorColumn['name']; |
181 | 181 | $tableAlias = $this->getSQLTableAlias($this->class->name); |
182 | 182 | |
183 | - return $tableAlias . '.' . $discColumn . ' IN (' . $values . ')'; |
|
183 | + return $tableAlias.'.'.$discColumn.' IN ('.$values.')'; |
|
184 | 184 | } |
185 | 185 | |
186 | 186 | /** |
@@ -33,6 +33,6 @@ |
||
33 | 33 | */ |
34 | 34 | public static function noDispatchForNode($node) |
35 | 35 | { |
36 | - return new self("Double-dispatch for node " . get_class($node) . " is not supported."); |
|
36 | + return new self("Double-dispatch for node ".get_class($node)." is not supported."); |
|
37 | 37 | } |
38 | 38 | } |
@@ -74,7 +74,7 @@ discard block |
||
74 | 74 | foreach ($props as $name => $prop) { |
75 | 75 | $ident += 4; |
76 | 76 | $str .= str_repeat(' ', $ident) . '"' . $name . '": ' |
77 | - . $this->dump($prop) . ',' . PHP_EOL; |
|
77 | + . $this->dump($prop) . ',' . PHP_EOL; |
|
78 | 78 | $ident -= 4; |
79 | 79 | } |
80 | 80 | |
@@ -86,7 +86,7 @@ discard block |
||
86 | 86 | |
87 | 87 | foreach ($obj as $k => $v) { |
88 | 88 | $str .= PHP_EOL . str_repeat(' ', $ident) . '"' |
89 | - . $k . '" => ' . $this->dump($v) . ','; |
|
89 | + . $k . '" => ' . $this->dump($v) . ','; |
|
90 | 90 | $some = true; |
91 | 91 | } |
92 | 92 |
@@ -68,32 +68,32 @@ |
||
68 | 68 | $str = ''; |
69 | 69 | |
70 | 70 | if ($obj instanceof Node) { |
71 | - $str .= get_class($obj) . '(' . PHP_EOL; |
|
71 | + $str .= get_class($obj).'('.PHP_EOL; |
|
72 | 72 | $props = get_object_vars($obj); |
73 | 73 | |
74 | 74 | foreach ($props as $name => $prop) { |
75 | 75 | $ident += 4; |
76 | - $str .= str_repeat(' ', $ident) . '"' . $name . '": ' |
|
77 | - . $this->dump($prop) . ',' . PHP_EOL; |
|
76 | + $str .= str_repeat(' ', $ident).'"'.$name.'": ' |
|
77 | + . $this->dump($prop).','.PHP_EOL; |
|
78 | 78 | $ident -= 4; |
79 | 79 | } |
80 | 80 | |
81 | - $str .= str_repeat(' ', $ident) . ')'; |
|
81 | + $str .= str_repeat(' ', $ident).')'; |
|
82 | 82 | } else if (is_array($obj)) { |
83 | 83 | $ident += 4; |
84 | 84 | $str .= 'array('; |
85 | 85 | $some = false; |
86 | 86 | |
87 | 87 | foreach ($obj as $k => $v) { |
88 | - $str .= PHP_EOL . str_repeat(' ', $ident) . '"' |
|
89 | - . $k . '" => ' . $this->dump($v) . ','; |
|
88 | + $str .= PHP_EOL.str_repeat(' ', $ident).'"' |
|
89 | + . $k.'" => '.$this->dump($v).','; |
|
90 | 90 | $some = true; |
91 | 91 | } |
92 | 92 | |
93 | 93 | $ident -= 4; |
94 | - $str .= ($some ? PHP_EOL . str_repeat(' ', $ident) : '') . ')'; |
|
94 | + $str .= ($some ? PHP_EOL.str_repeat(' ', $ident) : '').')'; |
|
95 | 95 | } else if (is_object($obj)) { |
96 | - $str .= 'instanceof(' . get_class($obj) . ')'; |
|
96 | + $str .= 'instanceof('.get_class($obj).')'; |
|
97 | 97 | } else { |
98 | 98 | $str .= var_export($obj, true); |
99 | 99 | } |
@@ -86,9 +86,9 @@ |
||
86 | 86 | $tableName = $sqlWalker->getEntityManager()->getClassMetadata($assoc['sourceEntity'])->getTableName(); |
87 | 87 | |
88 | 88 | $tableAlias = $sqlWalker->getSQLTableAlias($tableName, $dqlAlias); |
89 | - $columnName = $quoteStrategy->getJoinColumnName($joinColumn, $targetEntity, $platform); |
|
89 | + $columnName = $quoteStrategy->getJoinColumnName($joinColumn, $targetEntity, $platform); |
|
90 | 90 | |
91 | - return $tableAlias . '.' . $columnName; |
|
91 | + return $tableAlias.'.'.$columnName; |
|
92 | 92 | } |
93 | 93 | |
94 | 94 | /** |
@@ -70,8 +70,8 @@ discard block |
||
70 | 70 | if ($first) $first = false; else $sql .= ' AND '; |
71 | 71 | |
72 | 72 | $sql .= $targetTableAlias . '.' . $sourceColumn |
73 | - . ' = ' |
|
74 | - . $sourceTableAlias . '.' . $quoteStrategy->getColumnName($class->fieldNames[$targetColumn], $class, $platform); |
|
73 | + . ' = ' |
|
74 | + . $sourceTableAlias . '.' . $quoteStrategy->getColumnName($class->fieldNames[$targetColumn], $class, $platform); |
|
75 | 75 | } |
76 | 76 | } else { // many-to-many |
77 | 77 | $targetClass = $sqlWalker->getEntityManager()->getClassMetadata($assoc['targetEntity']); |
@@ -100,8 +100,8 @@ discard block |
||
100 | 100 | ); |
101 | 101 | |
102 | 102 | $sql .= $joinTableAlias . '.' . $joinColumn['name'] |
103 | - . ' = ' |
|
104 | - . $sourceTableAlias . '.' . $sourceColumnName; |
|
103 | + . ' = ' |
|
104 | + . $sourceTableAlias . '.' . $sourceColumnName; |
|
105 | 105 | } |
106 | 106 | } |
107 | 107 |
@@ -60,7 +60,7 @@ discard block |
||
60 | 60 | $targetTableAlias = $sqlWalker->getSQLTableAlias($targetClass->getTableName()); |
61 | 61 | $sourceTableAlias = $sqlWalker->getSQLTableAlias($class->getTableName(), $dqlAlias); |
62 | 62 | |
63 | - $sql .= $quoteStrategy->getTableName($targetClass, $platform) . ' ' . $targetTableAlias . ' WHERE '; |
|
63 | + $sql .= $quoteStrategy->getTableName($targetClass, $platform).' '.$targetTableAlias.' WHERE '; |
|
64 | 64 | |
65 | 65 | $owningAssoc = $targetClass->associationMappings[$assoc['mappedBy']]; |
66 | 66 | |
@@ -69,9 +69,9 @@ discard block |
||
69 | 69 | foreach ($owningAssoc['targetToSourceKeyColumns'] as $targetColumn => $sourceColumn) { |
70 | 70 | if ($first) $first = false; else $sql .= ' AND '; |
71 | 71 | |
72 | - $sql .= $targetTableAlias . '.' . $sourceColumn |
|
72 | + $sql .= $targetTableAlias.'.'.$sourceColumn |
|
73 | 73 | . ' = ' |
74 | - . $sourceTableAlias . '.' . $quoteStrategy->getColumnName($class->fieldNames[$targetColumn], $class, $platform); |
|
74 | + . $sourceTableAlias.'.'.$quoteStrategy->getColumnName($class->fieldNames[$targetColumn], $class, $platform); |
|
75 | 75 | } |
76 | 76 | } else { // many-to-many |
77 | 77 | $targetClass = $sqlWalker->getEntityManager()->getClassMetadata($assoc['targetEntity']); |
@@ -84,7 +84,7 @@ discard block |
||
84 | 84 | $sourceTableAlias = $sqlWalker->getSQLTableAlias($class->getTableName(), $dqlAlias); |
85 | 85 | |
86 | 86 | // join to target table |
87 | - $sql .= $quoteStrategy->getJoinTableName($owningAssoc, $targetClass, $platform) . ' ' . $joinTableAlias . ' WHERE '; |
|
87 | + $sql .= $quoteStrategy->getJoinTableName($owningAssoc, $targetClass, $platform).' '.$joinTableAlias.' WHERE '; |
|
88 | 88 | |
89 | 89 | $joinColumns = $assoc['isOwningSide'] |
90 | 90 | ? $joinTable['joinColumns'] |
@@ -99,13 +99,13 @@ discard block |
||
99 | 99 | $class->fieldNames[$joinColumn['referencedColumnName']], $class, $platform |
100 | 100 | ); |
101 | 101 | |
102 | - $sql .= $joinTableAlias . '.' . $joinColumn['name'] |
|
102 | + $sql .= $joinTableAlias.'.'.$joinColumn['name'] |
|
103 | 103 | . ' = ' |
104 | - . $sourceTableAlias . '.' . $sourceColumnName; |
|
104 | + . $sourceTableAlias.'.'.$sourceColumnName; |
|
105 | 105 | } |
106 | 106 | } |
107 | 107 | |
108 | - return '(' . $sql . ')'; |
|
108 | + return '('.$sql.')'; |
|
109 | 109 | } |
110 | 110 | |
111 | 111 | /** |
@@ -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 |
@@ -44,9 +44,9 @@ |
||
44 | 44 | */ |
45 | 45 | public function getSql(\Doctrine\ORM\Query\SqlWalker $sqlWalker) |
46 | 46 | { |
47 | - return 'ABS(' . $sqlWalker->walkSimpleArithmeticExpression( |
|
47 | + return 'ABS('.$sqlWalker->walkSimpleArithmeticExpression( |
|
48 | 48 | $this->simpleArithmeticExpression |
49 | - ) . ')'; |
|
49 | + ).')'; |
|
50 | 50 | } |
51 | 51 | |
52 | 52 | /** |