Andreas Braun

@Steveb-p I'm glad you like it - I always appreciate it reminding me about issues and PRs that I forgot to follow up on :)

We're aware of this issue and will implement the feature when we have the time. There is no need to remind us. Locking to collaborators until we start working on...

`loadReferenceManyOwningSide` now prepares the `sort` option in master: https://github.com/doctrine/mongodb-odm/blob/a443720e896dad8856e64e8031d467d3dadfc3b9/lib/Doctrine/ODM/MongoDB/Persisters/DocumentPersister.php#L771. If I understand correctly, this issue is about preparing the `$sort` option in a `$push` operator?

This can be added to the lowest supported branch, which at the moment would be 1.3.x. I've added it to the roadmap to be triaged and to keep the Bot...

I don't think a method should be marked as "internal" because it's performance sensitive, especially when it uses a cache. I suggest removing the `@internal` designation instead. @malarzm @jmikola would...

#2123 removes the designations. We will still add an interface for the document manager, similar to `EntityManagerInterface` in ORM.

> I was pretty sure we had a similar request in the past but as I can't find I'm gonna keep this open. You were thinking of #1723, which dealt...

I'll take another look at this - this should definitely work as you intend it to work.

Can you create a small test case to reproduce the issue? You can take a look at existing tests in https://github.com/doctrine/mongodb-odm/tree/master/tests/Doctrine/ODM/MongoDB/Tests/Functional/Ticket. Thanks!

I took a look at this and wasn't able to reproduce it with the following test: https://gist.github.com/alcaeus/8f91cc6ad367ed842a88ff7c5bfee3ab. Since you say the exception occurs in `CachingIterator::key`, I would assume there's something...