search

rollerworks-graveyard / RollerworksMultiUserBundle

Multi user management for the FOSUserBundle - DISCONTINUED!!
MIT License
56 stars 21 forks source link

Fix the user-system option in FOSUserBundle commands #75

Closed gfreeau closed 10 years ago

gfreeau commented 10 years ago

The current FOSUserBundle commands do not seem to work as documented. "user-system" should be an option instead of ordered argument. After the changes, it works as documented.

sstok commented 10 years ago

Can you also update the tests? Thanks :+1:

gfreeau commented 10 years ago

I have updated the tests and they are now passing.

I had a problem running the tests though, I had to disable any mention of mongo to get the functional tests to run. I saw the note about requiring mongo in your README, however the composer.json does not list mongodbbundle as a dev dependency.

Anyway, this second commit is only for the fosuserbundle tests.

sstok commented 10 years ago

Its only needed when testing multiple storage engines, so for these tests are not related. Some of the CI's had a problem with this dependency so its not installed by default.

I will merge this later today.

sstok commented 10 years ago

Thank you @gfreeau