Completed
Pull Request — master (#159)
by
unknown
04:33
created
lib/Mongo/MongoId.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -48,7 +48,7 @@
 block discarded – undo
48 48
     {
49 49
         if ($value instanceof ObjectID || $value instanceof MongoId) {
50 50
             return true;
51
-        } elseif (! is_string($value)) {
51
+        } elseif ( ! is_string($value)) {
52 52
             return false;
53 53
         }
54 54
 
Please login to merge, or discard this patch.
lib/Mongo/MongoCursor.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -221,7 +221,7 @@  discard block
 block discarded – undo
221 221
      */
222 222
     public function hasNext()
223 223
     {
224
-        if (! $this->startedIterating) {
224
+        if ( ! $this->startedIterating) {
225 225
             $this->ensureIterator();
226 226
             $this->startedIterating = true;
227 227
             $this->storeIteratorState();
@@ -405,7 +405,7 @@  discard block
 block discarded – undo
405 405
      */
406 406
     protected function convertCursorType()
407 407
     {
408
-        if (! $this->tailable) {
408
+        if ( ! $this->tailable) {
409 409
             return null;
410 410
         }
411 411
 
@@ -446,7 +446,7 @@  discard block
 block discarded – undo
446 446
     protected function wrapTraversable(\Traversable $traversable)
447 447
     {
448 448
         foreach ($traversable as $key => $value) {
449
-            if (isset($value->_id) && ($value->_id instanceof \MongoDB\BSON\ObjectID || !is_object($value->_id))) {
449
+            if (isset($value->_id) && ($value->_id instanceof \MongoDB\BSON\ObjectID || ! is_object($value->_id))) {
450 450
                 $key = (string) $value->_id;
451 451
             }
452 452
             yield $key => $value;
Please login to merge, or discard this patch.
lib/Mongo/MongoWriteBatch.php 1 patch
Spacing   +7 added lines, -7 removed lines patch added patch discarded remove patch
@@ -83,7 +83,7 @@  discard block
 block discarded – undo
83 83
     public function add($item)
84 84
     {
85 85
         if (is_object($item)) {
86
-            $item = (array)$item;
86
+            $item = (array) $item;
87 87
         }
88 88
 
89 89
         $this->validate($item);
@@ -102,7 +102,7 @@  discard block
 block discarded – undo
102 102
     final public function execute(array $writeOptions = [])
103 103
     {
104 104
         $writeOptions += $this->writeOptions;
105
-        if (! count($this->items)) {
105
+        if ( ! count($this->items)) {
106 106
             return ['ok' => true];
107 107
         }
108 108
 
@@ -167,7 +167,7 @@  discard block
 block discarded – undo
167 167
                 break;
168 168
         }
169 169
 
170
-        if (! $ok) {
170
+        if ( ! $ok) {
171 171
             // Exception code is hardcoded to the value in ext-mongo, see
172 172
             // https://github.com/mongodb/mongo-php-driver-legacy/blob/ab4bc0d90e93b3f247f6bcb386d0abc8d2fa7d74/batch/write.c#L428
173 173
             throw new \MongoWriteConcernException('Failed write', 911, null, $resultDocument);
@@ -180,19 +180,19 @@  discard block
 block discarded – undo
180 180
     {
181 181
         switch ($this->batchType) {
182 182
             case self::COMMAND_UPDATE:
183
-                if (! isset($item['q'])) {
183
+                if ( ! isset($item['q'])) {
184 184
                     throw new Exception("Expected \$item to contain 'q' key");
185 185
                 }
186
-                if (! isset($item['u'])) {
186
+                if ( ! isset($item['u'])) {
187 187
                     throw new Exception("Expected \$item to contain 'u' key");
188 188
                 }
189 189
                 break;
190 190
 
191 191
             case self::COMMAND_DELETE:
192
-                if (! isset($item['q'])) {
192
+                if ( ! isset($item['q'])) {
193 193
                     throw new Exception("Expected \$item to contain 'q' key");
194 194
                 }
195
-                if (! isset($item['limit'])) {
195
+                if ( ! isset($item['limit'])) {
196 196
                     throw new Exception("Expected \$item to contain 'limit' key");
197 197
                 }
198 198
                 break;
Please login to merge, or discard this patch.
lib/Alcaeus/MongoDbAdapter/TypeConverter.php 1 patch
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -103,16 +103,16 @@
 block discarded – undo
103 103
      */
104 104
     public static function convertProjection($fields)
105 105
     {
106
-        if (! is_array($fields) || $fields === []) {
106
+        if ( ! is_array($fields) || $fields === []) {
107 107
             return [];
108 108
         }
109 109
 
110
-        if (! TypeConverter::isNumericArray($fields)) {
110
+        if ( ! TypeConverter::isNumericArray($fields)) {
111 111
             $projection = TypeConverter::fromLegacy($fields);
112 112
         } else {
113 113
             $projection = array_fill_keys(
114
-                array_map(function ($field) {
115
-                    if (!is_string($field)) {
114
+                array_map(function($field) {
115
+                    if ( ! is_string($field)) {
116 116
                         throw new \MongoException('field names must be strings', 8);
117 117
                     }
118 118
 
Please login to merge, or discard this patch.
lib/Alcaeus/MongoDbAdapter/AbstractCursor.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -160,7 +160,7 @@  discard block
 block discarded – undo
160 160
      */
161 161
     public function next()
162 162
     {
163
-        if (! $this->startedIterating) {
163
+        if ( ! $this->startedIterating) {
164 164
             $this->ensureIterator();
165 165
             $this->startedIterating = true;
166 166
         } else {
@@ -402,7 +402,7 @@  discard block
 block discarded – undo
402 402
      */
403 403
     protected function storeIteratorState()
404 404
     {
405
-        if (! $this->startedIterating) {
405
+        if ( ! $this->startedIterating) {
406 406
             $this->current = null;
407 407
             $this->key = null;
408 408
             $this->valid = false;
Please login to merge, or discard this patch.
lib/Alcaeus/MongoDbAdapter/Helper/WriteConcernConverter.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -29,7 +29,7 @@
 block discarded – undo
29 29
             $wstring = (int) $wstring;
30 30
         }
31 31
 
32
-        if (! is_string($wstring) && ! is_int($wstring)) {
32
+        if ( ! is_string($wstring) && ! is_int($wstring)) {
33 33
             trigger_error("w for WriteConcern must be a string or integer", E_USER_WARNING);
34 34
             return false;
35 35
         }
Please login to merge, or discard this patch.
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
                 }
@@ -170,7 +170,7 @@  discard block
 block discarded – undo
170 170
                 'waitedMS' => 0,
171 171
             ];
172 172
         } catch (\MongoDB\Driver\Exception\Exception $e) {
173
-            throw ExceptionConverter::toLegacy($e,'MongoResultException');
173
+            throw ExceptionConverter::toLegacy($e, 'MongoResultException');
174 174
         }
175 175
     }
176 176
 
@@ -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 {
@@ -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
 
@@ -374,7 +374,7 @@  discard block
 block discarded – undo
374 374
      * @throws MongoCursorException
375 375
      * @return boolean
376 376
      */
377
-    public function update(array $criteria , array $newobj, array $options = [])
377
+    public function update(array $criteria, array $newobj, array $options = [])
378 378
     {
379 379
         $multiple = isset($options['multiple']) ? $options['multiple'] : false;
380 380
         $isReplace = ! \MongoDB\is_first_key_operator($newobj);
@@ -398,7 +398,7 @@  discard block
 block discarded – undo
398 398
             throw ExceptionConverter::toLegacy($e);
399 399
         }
400 400
 
401
-        if (! $result->isAcknowledged()) {
401
+        if ( ! $result->isAcknowledged()) {
402 402
             return true;
403 403
         }
404 404
 
@@ -425,7 +425,7 @@  discard block
 block discarded – undo
425 425
      */
426 426
     public function remove(array $criteria = [], array $options = [])
427 427
     {
428
-        $multiple = isset($options['justOne']) ? !$options['justOne'] : true;
428
+        $multiple = isset($options['justOne']) ? ! $options['justOne'] : true;
429 429
         $method = $multiple ? 'deleteMany' : 'deleteOne';
430 430
 
431 431
         try {
@@ -438,7 +438,7 @@  discard block
 block discarded – undo
438 438
             throw ExceptionConverter::toLegacy($e);
439 439
         }
440 440
 
441
-        if (! $result->isAcknowledged()) {
441
+        if ( ! $result->isAcknowledged()) {
442 442
             return true;
443 443
         }
444 444
 
@@ -516,7 +516,7 @@  discard block
 block discarded – undo
516 516
 
517 517
                 $options['projection'] = TypeConverter::convertProjection($fields);
518 518
 
519
-                if (! \MongoDB\is_first_key_operator($update)) {
519
+                if ( ! \MongoDB\is_first_key_operator($update)) {
520 520
                     $document = $this->collection->findOneAndReplace($query, $update, $options);
521 521
                 } else {
522 522
                     $document = $this->collection->findOneAndUpdate($query, $update, $options);
@@ -547,7 +547,7 @@  discard block
 block discarded – undo
547 547
     public function findOne($query = [], array $fields = [], array $options = [])
548 548
     {
549 549
         // Can't typehint for array since MongoGridFS extends and accepts strings
550
-        if (! is_array($query)) {
550
+        if ( ! is_array($query)) {
551 551
             trigger_error(sprintf('MongoCollection::findOne(): expects parameter 1 to be an array or object, %s given', gettype($query)), E_USER_WARNING);
552 552
             return;
553 553
         }
@@ -587,11 +587,11 @@  discard block
 block discarded – undo
587 587
             $keys = (array) $keys;
588 588
         }
589 589
 
590
-        if (! is_array($keys) || ! count($keys)) {
590
+        if ( ! is_array($keys) || ! count($keys)) {
591 591
             throw new MongoException('index specification has no elements');
592 592
         }
593 593
 
594
-        if (! isset($options['name'])) {
594
+        if ( ! isset($options['name'])) {
595 595
             $options['name'] = \MongoDB\generate_index_name($keys);
596 596
         }
597 597
 
@@ -626,14 +626,14 @@  discard block
 block discarded – undo
626 626
         }
627 627
 
628 628
         $result = [
629
-            'createdCollectionAutomatically' => !$collectionExists,
629
+            'createdCollectionAutomatically' => ! $collectionExists,
630 630
             'numIndexesBefore' => $indexCount,
631 631
             'numIndexesAfter' => $indexCount,
632 632
             'note' => 'all indexes already exist',
633 633
             'ok' => 1.0,
634 634
         ];
635 635
 
636
-        if (! $indexExists) {
636
+        if ( ! $indexExists) {
637 637
             $result['numIndexesAfter']++;
638 638
             unset($result['note']);
639 639
         }
@@ -666,7 +666,7 @@  discard block
 block discarded – undo
666 666
     {
667 667
         if (is_string($keys)) {
668 668
             $indexName = $keys;
669
-            if (! preg_match('#_-?1$#', $indexName)) {
669
+            if ( ! preg_match('#_-?1$#', $indexName)) {
670 670
                 $indexName .= '_1';
671 671
             }
672 672
         } elseif (is_array($keys)) {
@@ -729,7 +729,7 @@  discard block
 block discarded – undo
729 729
             ];
730 730
 
731 731
             foreach ($additionalKeys as $key) {
732
-                if (! isset($indexInfo[$key])) {
732
+                if ( ! isset($indexInfo[$key])) {
733 733
                     continue;
734 734
                 }
735 735
 
@@ -754,7 +754,7 @@  discard block
 block discarded – undo
754 754
     {
755 755
         try {
756 756
             // Handle legacy mode - limit and skip as second and third parameters, respectively
757
-            if (! is_array($options)) {
757
+            if ( ! is_array($options)) {
758 758
                 $limit = $options;
759 759
                 $options = [];
760 760
 
@@ -801,7 +801,7 @@  discard block
 block discarded – undo
801 801
                 $this->convertWriteConcernOptions($options)
802 802
             );
803 803
 
804
-            if (! $result->isAcknowledged()) {
804
+            if ( ! $result->isAcknowledged()) {
805 805
                 return true;
806 806
             }
807 807
 
@@ -835,13 +835,13 @@  discard block
 block discarded – undo
835 835
         if ($document_or_id instanceof \MongoId) {
836 836
             $id = $document_or_id;
837 837
         } elseif (is_object($document_or_id)) {
838
-            if (! isset($document_or_id->_id)) {
838
+            if ( ! isset($document_or_id->_id)) {
839 839
                 return null;
840 840
             }
841 841
 
842 842
             $id = $document_or_id->_id;
843 843
         } elseif (is_array($document_or_id)) {
844
-            if (! isset($document_or_id['_id'])) {
844
+            if ( ! isset($document_or_id['_id'])) {
845 845
                 return null;
846 846
             }
847 847
 
@@ -884,14 +884,14 @@  discard block
 block discarded – undo
884 884
         $command = [
885 885
             'group' => [
886 886
                 'ns' => $this->name,
887
-                '$reduce' => (string)$reduce,
887
+                '$reduce' => (string) $reduce,
888 888
                 'initial' => $initial,
889 889
                 'cond' => $condition,
890 890
             ],
891 891
         ];
892 892
 
893 893
         if ($keys instanceof MongoCode) {
894
-            $command['group']['$keyf'] = (string)$keys;
894
+            $command['group']['$keyf'] = (string) $keys;
895 895
         } else {
896 896
             $command['group']['key'] = $keys;
897 897
         }
@@ -900,7 +900,7 @@  discard block
 block discarded – undo
900 900
         }
901 901
         if (array_key_exists('finalize', $condition)) {
902 902
             if ($condition['finalize'] instanceof MongoCode) {
903
-                $condition['finalize'] = (string)$condition['finalize'];
903
+                $condition['finalize'] = (string) $condition['finalize'];
904 904
             }
905 905
             $command['group']['finalize'] = $condition['finalize'];
906 906
         }
@@ -954,11 +954,11 @@  discard block
 block discarded – undo
954 954
             $options['w'] = ($options['safe']) ? 1 : 0;
955 955
         }
956 956
 
957
-        if (isset($options['wtimeout']) && !isset($options['wTimeoutMS'])) {
957
+        if (isset($options['wtimeout']) && ! isset($options['wTimeoutMS'])) {
958 958
             $options['wTimeoutMS'] = $options['wtimeout'];
959 959
         }
960 960
 
961
-        if (isset($options['w']) || !isset($options['wTimeoutMS'])) {
961
+        if (isset($options['w']) || ! isset($options['wTimeoutMS'])) {
962 962
             $collectionWriteConcern = $this->getWriteConcern();
963 963
             $writeConcern = $this->createWriteConcernFromParameters(
964 964
                 isset($options['w']) ? $options['w'] : $collectionWriteConcern['w'],
@@ -991,7 +991,7 @@  discard block
 block discarded – undo
991 991
         };
992 992
 
993 993
         if (is_array($document)) {
994
-            if (! isset($document['_id'])) {
994
+            if ( ! isset($document['_id'])) {
995 995
                 $document['_id'] = new \MongoId();
996 996
             }
997 997
 
@@ -1001,12 +1001,12 @@  discard block
 block discarded – undo
1001 1001
         } elseif (is_object($document)) {
1002 1002
             $reflectionObject = new \ReflectionObject($document);
1003 1003
             foreach ($reflectionObject->getProperties() as $property) {
1004
-                if (! $property->isPublic()) {
1004
+                if ( ! $property->isPublic()) {
1005 1005
                     throw new \MongoException('zero-length keys are not allowed, did you use $ with double quotes?');
1006 1006
                 }
1007 1007
             }
1008 1008
 
1009
-            if (! isset($document->_id)) {
1009
+            if ( ! isset($document->_id)) {
1010 1010
                 $document->_id = new \MongoId();
1011 1011
             }
1012 1012
 
Please login to merge, or discard this patch.
lib/Mongo/MongoGridFSFile.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -78,7 +78,7 @@
 block discarded – undo
78 78
             $filename = 'file';
79 79
         }
80 80
 
81
-        if (! $handle = fopen($filename, 'w')) {
81
+        if ( ! $handle = fopen($filename, 'w')) {
82 82
             trigger_error('Can not open the destination file', E_USER_ERROR);
83 83
             return 0;
84 84
         }
Please login to merge, or discard this patch.