Completed
Pull Request — master (#193)
by
unknown
04:51 queued 56s
created
lib/Mongo/MongoCollection.php 1 patch
Spacing   +33 added lines, -33 removed lines patch added patch discarded remove patch
@@ -130,7 +130,7 @@  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
             $operators = func_get_args();
135 135
             $pipeline = [];
136 136
             $options = [];
@@ -138,7 +138,7 @@  discard block
 block discarded – undo
138 138
             $i = 0;
139 139
             foreach ($operators as $operator) {
140 140
                 $i++;
141
-                if (! is_array($operator)) {
141
+                if ( ! is_array($operator)) {
142 142
                     trigger_error("Argument $i is not an array", E_USER_WARNING);
143 143
                     return;
144 144
                 }
@@ -191,7 +191,7 @@  discard block
 block discarded – undo
191 191
         ];
192 192
 
193 193
         // Convert cursor option
194
-        if (! isset($options['cursor'])) {
194
+        if ( ! isset($options['cursor'])) {
195 195
             $options['cursor'] = new \stdClass();
196 196
         }
197 197
 
@@ -277,12 +277,12 @@  discard block
 block discarded – undo
277 277
      */
278 278
     public function insert(&$a, array $options = [])
279 279
     {
280
-        if (! $this->ensureDocumentHasMongoId($a)) {
280
+        if ( ! $this->ensureDocumentHasMongoId($a)) {
281 281
             trigger_error(sprintf('%s(): expects parameter %d to be an array or object, %s given', __METHOD__, 1, gettype($a)), E_USER_WARNING);
282 282
             return;
283 283
         }
284 284
 
285
-        if (! count((array)$a)) {
285
+        if ( ! count((array) $a)) {
286 286
             throw new \MongoException('document must be an array or object');
287 287
         }
288 288
 
@@ -295,7 +295,7 @@  discard block
 block discarded – undo
295 295
             throw ExceptionConverter::toLegacy($e);
296 296
         }
297 297
 
298
-        if (! $result->isAcknowledged()) {
298
+        if ( ! $result->isAcknowledged()) {
299 299
             return true;
300 300
         }
301 301
 
@@ -326,7 +326,7 @@  discard block
 block discarded – undo
326 326
 
327 327
         foreach ($a as $key => $item) {
328 328
             try {
329
-                if (! $this->ensureDocumentHasMongoId($a[$key])) {
329
+                if ( ! $this->ensureDocumentHasMongoId($a[$key])) {
330 330
                     if ($continueOnError) {
331 331
                         unset($a[$key]);
332 332
                     } else {
@@ -335,7 +335,7 @@  discard block
 block discarded – undo
335 335
                     }
336 336
                 }
337 337
             } catch (MongoException $e) {
338
-                if (! $continueOnError) {
338
+                if ( ! $continueOnError) {
339 339
                     throw $e;
340 340
                 }
341 341
             }
@@ -350,7 +350,7 @@  discard block
 block discarded – undo
350 350
             throw ExceptionConverter::toLegacy($e, 'MongoResultException');
351 351
         }
352 352
 
353
-        if (! $result->isAcknowledged()) {
353
+        if ( ! $result->isAcknowledged()) {
354 354
             return true;
355 355
         }
356 356
 
@@ -400,7 +400,7 @@  discard block
 block discarded – undo
400 400
             throw ExceptionConverter::toLegacy($e);
401 401
         }
402 402
 
403
-        if (! $result->isAcknowledged()) {
403
+        if ( ! $result->isAcknowledged()) {
404 404
             return true;
405 405
         }
406 406
 
@@ -427,7 +427,7 @@  discard block
 block discarded – undo
427 427
      */
428 428
     public function remove(array $criteria = [], array $options = [])
429 429
     {
430
-        $multiple = isset($options['justOne']) ? !$options['justOne'] : true;
430
+        $multiple = isset($options['justOne']) ? ! $options['justOne'] : true;
431 431
         $method = $multiple ? 'deleteMany' : 'deleteOne';
432 432
 
433 433
         try {
@@ -440,7 +440,7 @@  discard block
 block discarded – undo
440 440
             throw ExceptionConverter::toLegacy($e);
441 441
         }
442 442
 
443
-        if (! $result->isAcknowledged()) {
443
+        if ( ! $result->isAcknowledged()) {
444 444
             return true;
445 445
         }
446 446
 
@@ -518,7 +518,7 @@  discard block
 block discarded – undo
518 518
 
519 519
                 $options['projection'] = TypeConverter::convertProjection($fields);
520 520
 
521
-                if (! \MongoDB\is_first_key_operator($update)) {
521
+                if ( ! \MongoDB\is_first_key_operator($update)) {
522 522
                     $document = $this->collection->findOneAndReplace($query, $update, $options);
523 523
                 } else {
524 524
                     $document = $this->collection->findOneAndUpdate($query, $update, $options);
@@ -549,7 +549,7 @@  discard block
 block discarded – undo
549 549
     public function findOne($query = [], array $fields = [], array $options = [])
550 550
     {
551 551
         // Can't typehint for array since MongoGridFS extends and accepts strings
552
-        if (! is_array($query)) {
552
+        if ( ! is_array($query)) {
553 553
             trigger_error(sprintf('MongoCollection::findOne(): expects parameter 1 to be an array or object, %s given', gettype($query)), E_USER_WARNING);
554 554
             return;
555 555
         }
@@ -589,11 +589,11 @@  discard block
 block discarded – undo
589 589
             $keys = (array) $keys;
590 590
         }
591 591
 
592
-        if (! is_array($keys) || ! count($keys)) {
592
+        if ( ! is_array($keys) || ! count($keys)) {
593 593
             throw new MongoException('index specification has no elements');
594 594
         }
595 595
 
596
-        if (! isset($options['name'])) {
596
+        if ( ! isset($options['name'])) {
597 597
             $options['name'] = \MongoDB\generate_index_name($keys);
598 598
         }
599 599
 
@@ -628,14 +628,14 @@  discard block
 block discarded – undo
628 628
         }
629 629
 
630 630
         $result = [
631
-            'createdCollectionAutomatically' => !$collectionExists,
631
+            'createdCollectionAutomatically' => ! $collectionExists,
632 632
             'numIndexesBefore' => $indexCount,
633 633
             'numIndexesAfter' => $indexCount,
634 634
             'note' => 'all indexes already exist',
635 635
             'ok' => 1.0,
636 636
         ];
637 637
 
638
-        if (! $indexExists) {
638
+        if ( ! $indexExists) {
639 639
             $result['numIndexesAfter']++;
640 640
             unset($result['note']);
641 641
         }
@@ -668,7 +668,7 @@  discard block
 block discarded – undo
668 668
     {
669 669
         if (is_string($keys)) {
670 670
             $indexName = $keys;
671
-            if (! preg_match('#_-?1$#', $indexName)) {
671
+            if ( ! preg_match('#_-?1$#', $indexName)) {
672 672
                 $indexName .= '_1';
673 673
             }
674 674
         } elseif (is_array($keys)) {
@@ -707,7 +707,7 @@  discard block
 block discarded – undo
707 707
      */
708 708
     public function getIndexInfo()
709 709
     {
710
-        $convertIndex = function (\MongoDB\Model\IndexInfo $indexInfo) {
710
+        $convertIndex = function(\MongoDB\Model\IndexInfo $indexInfo) {
711 711
             $infos = [
712 712
                 'v' => $indexInfo->getVersion(),
713 713
                 'key' => $indexInfo->getKey(),
@@ -731,7 +731,7 @@  discard block
 block discarded – undo
731 731
             ];
732 732
 
733 733
             foreach ($additionalKeys as $key) {
734
-                if (! isset($indexInfo[$key])) {
734
+                if ( ! isset($indexInfo[$key])) {
735 735
                     continue;
736 736
                 }
737 737
 
@@ -756,7 +756,7 @@  discard block
 block discarded – undo
756 756
     {
757 757
         try {
758 758
             // Handle legacy mode - limit and skip as second and third parameters, respectively
759
-            if (! is_array($options)) {
759
+            if ( ! is_array($options)) {
760 760
                 $limit = $options;
761 761
                 $options = [];
762 762
 
@@ -803,7 +803,7 @@  discard block
 block discarded – undo
803 803
                 $this->convertWriteConcernOptions($options)
804 804
             );
805 805
 
806
-            if (! $result->isAcknowledged()) {
806
+            if ( ! $result->isAcknowledged()) {
807 807
                 return true;
808 808
             }
809 809
 
@@ -837,13 +837,13 @@  discard block
 block discarded – undo
837 837
         if ($document_or_id instanceof \MongoId) {
838 838
             $id = $document_or_id;
839 839
         } elseif (is_object($document_or_id)) {
840
-            if (! isset($document_or_id->_id)) {
840
+            if ( ! isset($document_or_id->_id)) {
841 841
                 return null;
842 842
             }
843 843
 
844 844
             $id = $document_or_id->_id;
845 845
         } elseif (is_array($document_or_id)) {
846
-            if (! isset($document_or_id['_id'])) {
846
+            if ( ! isset($document_or_id['_id'])) {
847 847
                 return null;
848 848
             }
849 849
 
@@ -886,14 +886,14 @@  discard block
 block discarded – undo
886 886
         $command = [
887 887
             'group' => [
888 888
                 'ns' => $this->name,
889
-                '$reduce' => (string)$reduce,
889
+                '$reduce' => (string) $reduce,
890 890
                 'initial' => $initial,
891 891
                 'cond' => $condition,
892 892
             ],
893 893
         ];
894 894
 
895 895
         if ($keys instanceof MongoCode) {
896
-            $command['group']['$keyf'] = (string)$keys;
896
+            $command['group']['$keyf'] = (string) $keys;
897 897
         } else {
898 898
             $command['group']['key'] = $keys;
899 899
         }
@@ -902,7 +902,7 @@  discard block
 block discarded – undo
902 902
         }
903 903
         if (array_key_exists('finalize', $condition)) {
904 904
             if ($condition['finalize'] instanceof MongoCode) {
905
-                $condition['finalize'] = (string)$condition['finalize'];
905
+                $condition['finalize'] = (string) $condition['finalize'];
906 906
             }
907 907
             $command['group']['finalize'] = $condition['finalize'];
908 908
         }
@@ -956,11 +956,11 @@  discard block
 block discarded – undo
956 956
             $options['w'] = ($options['safe']) ? 1 : 0;
957 957
         }
958 958
 
959
-        if (isset($options['wtimeout']) && !isset($options['wTimeoutMS'])) {
959
+        if (isset($options['wtimeout']) && ! isset($options['wTimeoutMS'])) {
960 960
             $options['wTimeoutMS'] = $options['wtimeout'];
961 961
         }
962 962
 
963
-        if (isset($options['w']) || !isset($options['wTimeoutMS'])) {
963
+        if (isset($options['w']) || ! isset($options['wTimeoutMS'])) {
964 964
             $collectionWriteConcern = $this->getWriteConcern();
965 965
             $writeConcern = $this->createWriteConcernFromParameters(
966 966
                 isset($options['w']) ? $options['w'] : $collectionWriteConcern['w'],
@@ -994,7 +994,7 @@  discard block
 block discarded – undo
994 994
     private function ensureDocumentHasMongoId(&$document)
995 995
     {
996 996
         if (is_array($document)) {
997
-            if (! isset($document['_id'])) {
997
+            if ( ! isset($document['_id'])) {
998 998
                 $document['_id'] = new \MongoId();
999 999
             }
1000 1000
 
@@ -1004,12 +1004,12 @@  discard block
 block discarded – undo
1004 1004
         } elseif (is_object($document)) {
1005 1005
             $reflectionObject = new \ReflectionObject($document);
1006 1006
             foreach ($reflectionObject->getProperties() as $property) {
1007
-                if (! $property->isPublic()) {
1007
+                if ( ! $property->isPublic()) {
1008 1008
                     throw new \MongoException('zero-length keys are not allowed, did you use $ with double quotes?');
1009 1009
                 }
1010 1010
             }
1011 1011
 
1012
-            if (! isset($document->_id)) {
1012
+            if ( ! isset($document->_id)) {
1013 1013
                 $document->_id = new \MongoId();
1014 1014
             }
1015 1015
 
Please login to merge, or discard this patch.