Commit 8383856b authored by Jeremy Mikola's avatar Jeremy Mikola

Don't use empty() to check for empty documents

empty() is inconsistent for checking if the document is empty (i.e. empty([]) != empty(new stdClass)). Simply check that the option has been set in order to include it in the command.
parent 28e0574f
...@@ -126,11 +126,11 @@ class CreateCollection implements Executable ...@@ -126,11 +126,11 @@ class CreateCollection implements Executable
} }
} }
if ( ! empty($this->options['indexOptionDefaults'])) { if (isset($this->options['indexOptionDefaults'])) {
$cmd['indexOptionDefaults'] = (object) $this->options['indexOptionDefaults']; $cmd['indexOptionDefaults'] = (object) $this->options['indexOptionDefaults'];
} }
if ( ! empty($this->options['storageEngine'])) { if (isset($this->options['storageEngine'])) {
$cmd['storageEngine'] = (object) $this->options['storageEngine']; $cmd['storageEngine'] = (object) $this->options['storageEngine'];
} }
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment