Passed
Pull Request — 2.7 (#7909)
by Luís
07:38
created
lib/Doctrine/ORM/EntityManager.php 1 patch
Spacing   +11 added lines, -11 removed lines patch added patch discarded remove patch
@@ -227,8 +227,8 @@  discard block
 block discarded – undo
227 227
      */
228 228
     public function transactional($func)
229 229
     {
230
-        if (!is_callable($func)) {
231
-            throw new \InvalidArgumentException('Expected argument of type "callable", got "' . gettype($func) . '"');
230
+        if ( ! is_callable($func)) {
231
+            throw new \InvalidArgumentException('Expected argument of type "callable", got "'.gettype($func).'"');
232 232
         }
233 233
 
234 234
         $this->conn->beginTransaction();
@@ -358,7 +358,7 @@  discard block
 block discarded – undo
358 358
     {
359 359
         if ($entity !== null) {
360 360
             @trigger_error(
361
-                'Calling ' . __METHOD__ . '() with any arguments to flush specific entities is deprecated and will not be supported in Doctrine 3.0.',
361
+                'Calling '.__METHOD__.'() with any arguments to flush specific entities is deprecated and will not be supported in Doctrine 3.0.',
362 362
                 E_USER_DEPRECATED
363 363
             );
364 364
         }
@@ -558,7 +558,7 @@  discard block
 block discarded – undo
558 558
 
559 559
         if ($entityName !== null) {
560 560
             @trigger_error(
561
-                'Calling ' . __METHOD__ . '() with any arguments to clear specific entities is deprecated and will not be supported in Doctrine 3.0.',
561
+                'Calling '.__METHOD__.'() with any arguments to clear specific entities is deprecated and will not be supported in Doctrine 3.0.',
562 562
                 E_USER_DEPRECATED
563 563
             );
564 564
         }
@@ -670,7 +670,7 @@  discard block
 block discarded – undo
670 670
      */
671 671
     public function detach($entity)
672 672
     {
673
-        @trigger_error('Method ' . __METHOD__ . '() is deprecated and will be removed in Doctrine 3.0.', E_USER_DEPRECATED);
673
+        @trigger_error('Method '.__METHOD__.'() is deprecated and will be removed in Doctrine 3.0.', E_USER_DEPRECATED);
674 674
 
675 675
         if ( ! is_object($entity)) {
676 676
             throw ORMInvalidArgumentException::invalidObject('EntityManager#detach()', $entity);
@@ -695,7 +695,7 @@  discard block
 block discarded – undo
695 695
      */
696 696
     public function merge($entity)
697 697
     {
698
-        @trigger_error('Method ' . __METHOD__ . '() is deprecated and will be removed in Doctrine 3.0.', E_USER_DEPRECATED);
698
+        @trigger_error('Method '.__METHOD__.'() is deprecated and will be removed in Doctrine 3.0.', E_USER_DEPRECATED);
699 699
 
700 700
         if ( ! is_object($entity)) {
701 701
             throw ORMInvalidArgumentException::invalidObject('EntityManager#merge()', $entity);
@@ -711,7 +711,7 @@  discard block
 block discarded – undo
711 711
      */
712 712
     public function copy($entity, $deep = false)
713 713
     {
714
-        @trigger_error('Method ' . __METHOD__ . '() is deprecated and will be removed in Doctrine 3.0.', E_USER_DEPRECATED);
714
+        @trigger_error('Method '.__METHOD__.'() is deprecated and will be removed in Doctrine 3.0.', E_USER_DEPRECATED);
715 715
 
716 716
         throw new \BadMethodCallException("Not implemented.");
717 717
     }
@@ -785,7 +785,7 @@  discard block
 block discarded – undo
785 785
      */
786 786
     public function isOpen()
787 787
     {
788
-        return (!$this->closed);
788
+        return ( ! $this->closed);
789 789
     }
790 790
 
791 791
     /**
@@ -896,7 +896,7 @@  discard block
 block discarded – undo
896 896
                 sprintf(
897 897
                     'Invalid $connection argument of type %s given%s.',
898 898
                     is_object($connection) ? get_class($connection) : gettype($connection),
899
-                    is_object($connection) ? '' : ': "' . $connection . '"'
899
+                    is_object($connection) ? '' : ': "'.$connection.'"'
900 900
                 )
901 901
             );
902 902
         }
@@ -946,13 +946,13 @@  discard block
 block discarded – undo
946 946
     {
947 947
         switch ($lockMode) {
948 948
             case LockMode::OPTIMISTIC:
949
-                if (!$class->isVersioned) {
949
+                if ( ! $class->isVersioned) {
950 950
                     throw OptimisticLockException::notVersioned($class->name);
951 951
                 }
952 952
                 break;
953 953
             case LockMode::PESSIMISTIC_READ:
954 954
             case LockMode::PESSIMISTIC_WRITE:
955
-                if (!$this->getConnection()->isTransactionActive()) {
955
+                if ( ! $this->getConnection()->isTransactionActive()) {
956 956
                     throw TransactionRequiredException::transactionRequired();
957 957
                 }
958 958
         }
Please login to merge, or discard this patch.
lib/Doctrine/ORM/Mapping/UnderscoreNamingStrategy.php 1 patch
Spacing   +6 added lines, -6 removed lines patch added patch discarded remove patch
@@ -60,9 +60,9 @@  discard block
 block discarded – undo
60 60
      */
61 61
     public function __construct($case = CASE_LOWER, bool $numberAware = false)
62 62
     {
63
-        if (! $numberAware) {
63
+        if ( ! $numberAware) {
64 64
             @trigger_error(
65
-                'Creating ' . self::class . ' without making it number aware is deprecated and will be removed in Doctrine 3.0.',
65
+                'Creating '.self::class.' without making it number aware is deprecated and will be removed in Doctrine 3.0.',
66 66
                 E_USER_DEPRECATED
67 67
             );
68 68
         }
@@ -125,7 +125,7 @@  discard block
 block discarded – undo
125 125
      */
126 126
     public function referenceColumnName()
127 127
     {
128
-        return $this->case === CASE_UPPER ?  'ID' : 'id';
128
+        return $this->case === CASE_UPPER ? 'ID' : 'id';
129 129
     }
130 130
 
131 131
     /**
@@ -133,7 +133,7 @@  discard block
 block discarded – undo
133 133
      */
134 134
     public function joinColumnName($propertyName, $className = null)
135 135
     {
136
-        return $this->underscore($propertyName) . '_' . $this->referenceColumnName();
136
+        return $this->underscore($propertyName).'_'.$this->referenceColumnName();
137 137
     }
138 138
 
139 139
     /**
@@ -141,7 +141,7 @@  discard block
 block discarded – undo
141 141
      */
142 142
     public function joinTableName($sourceEntity, $targetEntity, $propertyName = null)
143 143
     {
144
-        return $this->classToTableName($sourceEntity) . '_' . $this->classToTableName($targetEntity);
144
+        return $this->classToTableName($sourceEntity).'_'.$this->classToTableName($targetEntity);
145 145
     }
146 146
 
147 147
     /**
@@ -149,7 +149,7 @@  discard block
 block discarded – undo
149 149
      */
150 150
     public function joinKeyColumnName($entityName, $referencedColumnName = null)
151 151
     {
152
-        return $this->classToTableName($entityName) . '_' .
152
+        return $this->classToTableName($entityName).'_'.
153 153
                 ($referencedColumnName ?: $this->referenceColumnName());
154 154
     }
155 155
 
Please login to merge, or discard this patch.