Skip to content

Commit ffc47d4

Browse files
committed
PHPLIB-315: Remove default for mapReduce verbose option
1 parent 7231d87 commit ffc47d4

File tree

3 files changed

+1
-7
lines changed

3 files changed

+1
-7
lines changed

src/Operation/MapReduce.php

Lines changed: 1 addition & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -121,7 +121,7 @@ class MapReduce implements Executable
121121
* applied to the returned Cursor (it is not sent to the server).
122122
*
123123
* * verbose (boolean): Specifies whether to include the timing information
124-
* in the result information. The default is true.
124+
* in the result information.
125125
*
126126
* * writeConcern (MongoDB\Driver\WriteConcern): Write concern. This only
127127
* applies when results are output to a collection.
@@ -143,10 +143,6 @@ public function __construct($databaseName, $collectionName, JavascriptInterface
143143
throw InvalidArgumentException::invalidType('$out', $out, 'string or array or object');
144144
}
145145

146-
$options += [
147-
'verbose' => true,
148-
];
149-
150146
if (isset($options['bypassDocumentValidation']) && ! is_bool($options['bypassDocumentValidation'])) {
151147
throw InvalidArgumentException::invalidType('"bypassDocumentValidation" option', $options['bypassDocumentValidation'], 'boolean');
152148
}

tests/Collection/CollectionFunctionalTest.php

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -201,7 +201,6 @@ public function testMapReduce()
201201

202202
$this->assertGreaterThanOrEqual(0, $result->getExecutionTimeMS());
203203
$this->assertNotEmpty($result->getCounts());
204-
$this->assertNotEmpty($result->getTiming());
205204
}
206205

207206
/**

tests/Operation/MapReduceFunctionalTest.php

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -93,7 +93,6 @@ public function testResult()
9393
$this->assertInstanceOf('MongoDB\MapReduceResult', $result);
9494
$this->assertGreaterThanOrEqual(0, $result->getExecutionTimeMS());
9595
$this->assertNotEmpty($result->getCounts());
96-
$this->assertNotEmpty($result->getTiming());
9796
}
9897

9998
public function testResultDoesNotIncludeTimingWithoutVerboseOption()

0 commit comments

Comments
 (0)