Don't require doctrine/mongodb

[plfort]

We should not put "doctrine/mongodb" in the "require" section of composer.json, but in "require-dev" and in "suggest".

[DonCallisto]

Well, why?
If this bundle supports mongodb - as it does - it should have the same requirements of doctrine/orm.
What am I missing?

[plfort]

Because doctrine/mongodb requires "ext-mongo". I don't want to install it if I don't use it.

[DonCallisto]

So, the same should be true for doctrine/orm, isn't it?
I mean, what if I use mongo and I don't want to use doctrine/orm: AFAIK doctrine/orm can't handle directly mongo, am I wrong?

[plfort]

Yes

[DonCallisto]

Well, it makes sense but I still be doubtful about constraint versions of those packages: how can we handle them?

[plfort]

I don't know exactly how to handle it, maybe the best way would be to split the project into 3 part (common,orm,odm)
But for now we can just remove these 2 dependencies of the "require" section, add it in "require-dev" and let the user handle this manually (as before)

[DonCallisto]

But for now we can just remove these 2 dependencies of the "require" section, add it in "require-dev" and let the user handle this manually (as before)

Yes, maybe we should document it in the README

Moreover Travis CI should be fixed and doctrine/orm should be required "manually"

[daum]

Just ran into this issue as well we don't use mongodb but now can't get to the latest version of this because it required mongo.

[DonCallisto]

@plfort will you fix this or do I need to assign this issue to me?

I can fix it tomorrow if you want, otherwise, as this suggestion comes from you, you can take it directly.
Let me know.

[plfort]

I wouldn't have time to deal with it this weekend, you can take it

[DonCallisto]

I will ask you for a review ASAP

[DonCallisto]

@daum @plfort read this https://matthiasnoback.nl/2014/04/theres-no-such-thing-as-an-optional-dependency/

I totally agree with Matthias: we should split our project. As long as it lives this way, we cannot get a rid of mongo or doctrine.

Can I close this?

[plfort]

This is exactly what I said:

I don't know exactly how to handle it, maybe the best way would be to split the project into 3 parts (common,orm,odm)

But in a first time we must remove this deps and let users handle it, as it