Collection::ensureIndex() should convert write options #168

Closed
jmikola opened this Issue Apr 9, 2014 · 0 comments

Projects

None yet

1 participant

@jmikola
Member
jmikola commented Apr 9, 2014

Options should be run through convertWriteConcern() if the safe option is set, much like the do methods for other write methods.

@jmikola jmikola self-assigned this Apr 9, 2014
@jmikola jmikola added a commit to doctrine/mongodb-odm that referenced this issue Apr 9, 2014
@jmikola jmikola Set default "w" or "safe" option when creating indexes
Depending on the driver version, "w" or "safe" should be set by default. doctrine/mongodb#168 may make this unnecessary, but there's no reason ODM cannot be explicit.
b9e571f
@jmikola jmikola added a commit to doctrine/mongodb-odm that referenced this issue Apr 9, 2014
@jmikola jmikola Set default "w" or "safe" option when creating indexes
Depending on the driver version, "w" or "safe" should be set by default. doctrine/mongodb#168 may make this unnecessary, but there's no reason ODM cannot be explicit.

Fixes #655.
334e6cd
@jmikola jmikola added the bug label Apr 10, 2014
@jmikola jmikola added this to the 1.0.9 milestone Apr 10, 2014
@jmikola jmikola closed this in 30aa713 Apr 10, 2014
@jmikola jmikola added a commit to doctrine/mongodb-odm that referenced this issue Apr 10, 2014
@jmikola jmikola Set default "w" or "safe" option when creating indexes
Depending on the driver version, "w" or "safe" should be set by default. doctrine/mongodb#168 may make this unnecessary, but there's no reason ODM cannot be explicit.

Fixes #655.
deb6a38
@crimsonronin crimsonronin pushed a commit to crimsonronin/mongodb-odm that referenced this issue Apr 28, 2014
@jmikola jmikola Set default "w" or "safe" option when creating indexes
Depending on the driver version, "w" or "safe" should be set by default. doctrine/mongodb#168 may make this unnecessary, but there's no reason ODM cannot be explicit.

Fixes #655.
e2ed903
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment