mongo.cmd INI settings - Githubissues

colinmollenhour / mongodb-php-odm

A simple but powerful set of wrappers for using MongoDb in PHP (also a Kohana 3 module)

BSD 3-Clause "New" or "Revised" License

208 stars 50 forks source link

Closed lucasmichot closed 13 years ago

lucasmichot commented 13 years ago

Class Mongo_Collection should rely on mongo.cmd INI settings instead of $ (lines 238 to 251) :

We should have : ini_get('mongo.cmd') instead of '$' ini_get('mongo.cmd') . 'where' instead of '$where' ini_get('mongo.cmd') . 'or' instead of '$or'

colinmollenhour commented 13 years ago

I would be happy to accept a pull request for this issue. :)

lucasmichot commented 13 years ago

After testing, this change is useless : it seems PHP replaces '$' with the proper mongo.cmd setting

colinmollenhour commented 13 years ago

Good to know. Thanks for testing.