-
Notifications
You must be signed in to change notification settings - Fork 230
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Update configuration for 1.4.x driver options #219
Conversation
Changed VariableNode to prototyped ArrayNode Listing all options for connection in Configuration Removed \MongoClient consts Changed readPreferenceTags to arrayNode
->end() | ||
->end() | ||
->booleanNode('j')->end() | ||
->scalarNode('timeout')->end() |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
shouldn't this be an integer node ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The bundle still supports Symfony 2.1, so those aren't yet available. Perhaps we can bump it to 2.2 for 3.0. I've created #220 to track the idea.
Also refactors the ConfigurationTest class. Removed strict checking of default_commit_options (and tests), added missing options. Fixed handling of readPreferenceTags option for XML. As a change, "server" and "options" no longer have default values; however, existing code in Configuration should handle this fine.
This is currently supported in AbstractDoctrineExtension.
Update configuration for 1.4.x driver options
Glad I could help :) |
Supersedes @malarzm's work in #218.
Fixes #203, #214, and #215.