Completed
Pull Request — master (#84)
by Andreas
03:52
created
lib/Mongo/MongoCollection.php 1 patch
Spacing   +32 added lines, -32 removed lines patch added patch discarded remove patch
@@ -130,14 +130,14 @@  discard block
 block discarded – undo
130 130
      */
131 131
     public function aggregate(array $pipeline, array $op = [])
132 132
     {
133
-        if (! TypeConverter::isNumericArray($pipeline)) {
133
+        if ( ! TypeConverter::isNumericArray($pipeline)) {
134 134
             $pipeline = [];
135 135
             $options = [];
136 136
 
137 137
             $i = 0;
138 138
             foreach (func_get_args() as $operator) {
139 139
                 $i++;
140
-                if (! is_array($operator)) {
140
+                if ( ! is_array($operator)) {
141 141
                     trigger_error("Argument $i is not an array", E_WARNING);
142 142
                     return;
143 143
                 }
@@ -169,7 +169,7 @@  discard block
 block discarded – undo
169 169
                 'waitedMS' => 0,
170 170
             ];
171 171
         } catch (\MongoDB\Driver\Exception\Exception $e) {
172
-            throw ExceptionConverter::toLegacy($e,'MongoResultException');
172
+            throw ExceptionConverter::toLegacy($e, 'MongoResultException');
173 173
         }
174 174
     }
175 175
 
@@ -190,7 +190,7 @@  discard block
 block discarded – undo
190 190
         ];
191 191
 
192 192
         // Convert cursor option
193
-        if (! isset($options['cursor'])) {
193
+        if ( ! isset($options['cursor'])) {
194 194
             $options['cursor'] = new \stdClass();
195 195
         }
196 196
 
@@ -276,12 +276,12 @@  discard block
 block discarded – undo
276 276
      */
277 277
     public function insert(&$a, array $options = [])
278 278
     {
279
-        if (! $this->ensureDocumentHasMongoId($a)) {
279
+        if ( ! $this->ensureDocumentHasMongoId($a)) {
280 280
             trigger_error(sprintf('%s(): expects parameter %d to be an array or object, %s given', __METHOD__, 1, gettype($a)), E_USER_WARNING);
281 281
             return;
282 282
         }
283 283
 
284
-        if (! count((array)$a)) {
284
+        if ( ! count((array) $a)) {
285 285
             throw new \MongoException('document must be an array or object');
286 286
         }
287 287
 
@@ -294,7 +294,7 @@  discard block
 block discarded – undo
294 294
             throw ExceptionConverter::toLegacy($e);
295 295
         }
296 296
 
297
-        if (! $result->isAcknowledged()) {
297
+        if ( ! $result->isAcknowledged()) {
298 298
             return true;
299 299
         }
300 300
 
@@ -325,7 +325,7 @@  discard block
 block discarded – undo
325 325
 
326 326
         foreach ($a as $key => $item) {
327 327
             try {
328
-                if (! $this->ensureDocumentHasMongoId($a[$key])) {
328
+                if ( ! $this->ensureDocumentHasMongoId($a[$key])) {
329 329
                     if ($continueOnError) {
330 330
                         unset($a[$key]);
331 331
                     } else {
@@ -349,7 +349,7 @@  discard block
 block discarded – undo
349 349
             throw ExceptionConverter::toLegacy($e, 'MongoResultException');
350 350
         }
351 351
 
352
-        if (! $result->isAcknowledged()) {
352
+        if ( ! $result->isAcknowledged()) {
353 353
             return true;
354 354
         }
355 355
 
@@ -373,7 +373,7 @@  discard block
 block discarded – undo
373 373
      * @throws MongoCursorException
374 374
      * @return boolean
375 375
      */
376
-    public function update(array $criteria , array $newobj, array $options = [])
376
+    public function update(array $criteria, array $newobj, array $options = [])
377 377
     {
378 378
         $multiple = isset($options['multiple']) ? $options['multiple'] : false;
379 379
         $isReplace = ! \MongoDB\is_first_key_operator($newobj);
@@ -397,7 +397,7 @@  discard block
 block discarded – undo
397 397
             throw ExceptionConverter::toLegacy($e);
398 398
         }
399 399
 
400
-        if (! $result->isAcknowledged()) {
400
+        if ( ! $result->isAcknowledged()) {
401 401
             return true;
402 402
         }
403 403
 
@@ -424,7 +424,7 @@  discard block
 block discarded – undo
424 424
      */
425 425
     public function remove(array $criteria = [], array $options = [])
426 426
     {
427
-        $multiple = isset($options['justOne']) ? !$options['justOne'] : true;
427
+        $multiple = isset($options['justOne']) ? ! $options['justOne'] : true;
428 428
         $method = $multiple ? 'deleteMany' : 'deleteOne';
429 429
 
430 430
         try {
@@ -437,7 +437,7 @@  discard block
 block discarded – undo
437 437
             throw ExceptionConverter::toLegacy($e);
438 438
         }
439 439
 
440
-        if (! $result->isAcknowledged()) {
440
+        if ( ! $result->isAcknowledged()) {
441 441
             return true;
442 442
         }
443 443
 
@@ -515,7 +515,7 @@  discard block
 block discarded – undo
515 515
 
516 516
                 $options['projection'] = is_array($fields) ? TypeConverter::fromLegacy($fields) : [];
517 517
 
518
-                if (! \MongoDB\is_first_key_operator($update)) {
518
+                if ( ! \MongoDB\is_first_key_operator($update)) {
519 519
                     $document = $this->collection->findOneAndReplace($query, $update, $options);
520 520
                 } else {
521 521
                     $document = $this->collection->findOneAndUpdate($query, $update, $options);
@@ -546,7 +546,7 @@  discard block
 block discarded – undo
546 546
     public function findOne($query = [], array $fields = [], array $options = [])
547 547
     {
548 548
         // Can't typehint for array since MongoGridFS extends and accepts strings
549
-        if (! is_array($query)) {
549
+        if ( ! is_array($query)) {
550 550
             trigger_error(sprintf('MongoCollection::findOne(): expects parameter 1 to be an array or object, %s given', gettype($query)), E_WARNING);
551 551
             return;
552 552
         }
@@ -586,11 +586,11 @@  discard block
 block discarded – undo
586 586
             $keys = (array) $keys;
587 587
         }
588 588
 
589
-        if (! is_array($keys) || ! count($keys)) {
589
+        if ( ! is_array($keys) || ! count($keys)) {
590 590
             throw new MongoException('index specification has no elements');
591 591
         }
592 592
 
593
-        if (! isset($options['name'])) {
593
+        if ( ! isset($options['name'])) {
594 594
             $options['name'] = \MongoDB\generate_index_name($keys);
595 595
         }
596 596
 
@@ -625,14 +625,14 @@  discard block
 block discarded – undo
625 625
         }
626 626
 
627 627
         $result = [
628
-            'createdCollectionAutomatically' => !$collectionExists,
628
+            'createdCollectionAutomatically' => ! $collectionExists,
629 629
             'numIndexesBefore' => $indexCount,
630 630
             'numIndexesAfter' => $indexCount,
631 631
             'note' => 'all indexes already exist',
632 632
             'ok' => 1.0,
633 633
         ];
634 634
 
635
-        if (! $indexExists) {
635
+        if ( ! $indexExists) {
636 636
             $result['numIndexesAfter']++;
637 637
             unset($result['note']);
638 638
         }
@@ -665,7 +665,7 @@  discard block
 block discarded – undo
665 665
     {
666 666
         if (is_string($keys)) {
667 667
             $indexName = $keys;
668
-            if (! preg_match('#_-?1$#', $indexName)) {
668
+            if ( ! preg_match('#_-?1$#', $indexName)) {
669 669
                 $indexName .= '_1';
670 670
             }
671 671
         } elseif (is_array($keys)) {
@@ -728,7 +728,7 @@  discard block
 block discarded – undo
728 728
     {
729 729
         try {
730 730
             // Handle legacy mode - limit and skip as second and third parameters, respectively
731
-            if (! is_array($options)) {
731
+            if ( ! is_array($options)) {
732 732
                 $limit = $options;
733 733
                 $options = [];
734 734
 
@@ -775,7 +775,7 @@  discard block
 block discarded – undo
775 775
                 $this->convertWriteConcernOptions($options)
776 776
             );
777 777
 
778
-            if (! $result->isAcknowledged()) {
778
+            if ( ! $result->isAcknowledged()) {
779 779
                 return true;
780 780
             }
781 781
 
@@ -809,13 +809,13 @@  discard block
 block discarded – undo
809 809
         if ($document_or_id instanceof \MongoId) {
810 810
             $id = $document_or_id;
811 811
         } elseif (is_object($document_or_id)) {
812
-            if (! isset($document_or_id->_id)) {
812
+            if ( ! isset($document_or_id->_id)) {
813 813
                 return null;
814 814
             }
815 815
 
816 816
             $id = $document_or_id->_id;
817 817
         } elseif (is_array($document_or_id)) {
818
-            if (! isset($document_or_id['_id'])) {
818
+            if ( ! isset($document_or_id['_id'])) {
819 819
                 return null;
820 820
             }
821 821
 
@@ -858,14 +858,14 @@  discard block
 block discarded – undo
858 858
         $command = [
859 859
             'group' => [
860 860
                 'ns' => $this->name,
861
-                '$reduce' => (string)$reduce,
861
+                '$reduce' => (string) $reduce,
862 862
                 'initial' => $initial,
863 863
                 'cond' => $condition,
864 864
             ],
865 865
         ];
866 866
 
867 867
         if ($keys instanceof MongoCode) {
868
-            $command['group']['$keyf'] = (string)$keys;
868
+            $command['group']['$keyf'] = (string) $keys;
869 869
         } else {
870 870
             $command['group']['key'] = $keys;
871 871
         }
@@ -874,7 +874,7 @@  discard block
 block discarded – undo
874 874
         }
875 875
         if (array_key_exists('finalize', $condition)) {
876 876
             if ($condition['finalize'] instanceof MongoCode) {
877
-                $condition['finalize'] = (string)$condition['finalize'];
877
+                $condition['finalize'] = (string) $condition['finalize'];
878 878
             }
879 879
             $command['group']['finalize'] = $condition['finalize'];
880 880
         }
@@ -928,11 +928,11 @@  discard block
 block discarded – undo
928 928
             $options['w'] = ($options['safe']) ? 1 : 0;
929 929
         }
930 930
 
931
-        if (isset($options['wtimeout']) && !isset($options['wTimeoutMS'])) {
931
+        if (isset($options['wtimeout']) && ! isset($options['wTimeoutMS'])) {
932 932
             $options['wTimeoutMS'] = $options['wtimeout'];
933 933
         }
934 934
 
935
-        if (isset($options['w']) || !isset($options['wTimeoutMS'])) {
935
+        if (isset($options['w']) || ! isset($options['wTimeoutMS'])) {
936 936
             $collectionWriteConcern = $this->getWriteConcern();
937 937
             $writeConcern = $this->createWriteConcernFromParameters(
938 938
                 isset($options['w']) ? $options['w'] : $collectionWriteConcern['w'],
@@ -965,7 +965,7 @@  discard block
 block discarded – undo
965 965
         };
966 966
 
967 967
         if (is_array($document)) {
968
-            if (! isset($document['_id'])) {
968
+            if ( ! isset($document['_id'])) {
969 969
                 $document['_id'] = new \MongoId();
970 970
             }
971 971
 
@@ -975,12 +975,12 @@  discard block
 block discarded – undo
975 975
         } elseif (is_object($document)) {
976 976
             $reflectionObject = new \ReflectionObject($document);
977 977
             foreach ($reflectionObject->getProperties() as $property) {
978
-                if (! $property->isPublic()) {
978
+                if ( ! $property->isPublic()) {
979 979
                     throw new \MongoException('zero-length keys are not allowed, did you use $ with double quotes?');
980 980
                 }
981 981
             }
982 982
 
983
-            if (! isset($document->_id)) {
983
+            if ( ! isset($document->_id)) {
984 984
                 $document->_id = new \MongoId();
985 985
             }
986 986
 
Please login to merge, or discard this patch.