Fix PHPDoc @return types in various places about mongodb-odm

[epicwhale]

Another better example is in \Doctrine\MongoDB\Query\Builder

the return types for functions like field(..), equals(..), where(..) is @return Query instead of @return Builder.

[kriswallsmith]

Would you be willing to submit a PR to fix these?

[epicwhale]

I'm sorry, but what does PR stand for?

[kriswallsmith]

Pull request.

No apology necessary :)

[epicwhale]

ha, stupid me. Should be an obvious term on git. I'm still on the SVN bandwagon, so figuring out the git mechanisms will take me a while. I'm going to try my hands around a little. But does there happen to be an easier for me to contribute towards this issue? Since I'm going to be using ODM heavily this month, I can fix any broken PHPDoc blocks.

[kriswallsmith]

Sounds like an excellent opportunity to learn some basic Git, actually. Fork the repository in GitHub, clone your fork to your computer, make the changes, commit the changes, push the changes, submit a PR.

[epicwhale]

Sounds good. Should I submit a PR frequently or only once I've covered a substantial amount of PHPDoc fixes?

[kriswallsmith]

One PR with everything would be awesome.

[epicwhale]

Ok, I'll try my best :)

p.s: closed that issue by mistake.

[epicwhale]

@kriswallsmith

Tell me if a fix like this would be acceptable? epicwhale/mongodb@eec4105#lib/Doctrine/MongoDB/Query/Builder.php (that belongs from the mongodb library, but just use it as an example)

I just want to make sure I'm doing the right thing and the final push will actually be accepted into the master branch.

[epicwhale]

I've been fixing more of it. Could you tell me if there are any objections to my PHPDoc improvements?

epicwhale@570c1da#lib/Doctrine/ODM/MongoDB/Query/Builder.php

[kriswallsmith]

Your work looks good to me.

[epicwhale]

Great. I'll do the same for the mongodb library as well and update you when I'm done. It's going to take a while. Any major changes to the API planned in the coming weeks? I don't want to be witnessing conflicts when I pull.

[epicwhale]

@kriswallsmith

The most important bits are already covered (since these will be used by end-developers). Here's the pull request: #128

[jwage]

if you want to make more changes please send a PR.