Title says it all. Please remove the connect option from line 34 in mongodb.module

CommentFileSizeAuthor
#1 connect_option_removed-1866622-0.patch605 bytestungscit

Comments

tungscit’s picture

tungscit commented
StatusFileSize
new connect_option_removed-1866622-0.patch605 bytes

Here is the patch

tungscit’s picture

tungscit commented
Status: Needs work » Needs review

Status: Needs review » Needs work

The last submitted patch, connect_option_removed-1866622-0.patch, failed testing.

tungscit’s picture

tungscit commented
Version: 7.x-1.0-rc2 » 7.x-1.x-dev
fgm’s picture

fgm
Primary language French
Location Paris, France
commented

This would need some rationale: the module needs to support not only recent versions but, to a certain extent, older ones.

Will this soon-to-be-deprecated option cause an error if used on a recent server, or just be ignored ? And will its removal actually cause a problem on older versions ? If that's the case, the patch needs to account for both situations.

Anonymous’s picture

Anonymous (not verified) commented
Issue summary: View changes
Status: Needs work » Closed (cannot reproduce)

The "connect" option is passed to the MongoClient constructor. And I don't see it as being deprecated.

Please provide more info if you consider this to be the case and I missed something in the docs.