PHPORM-229 Make Query\Builder return objects instead of array to match Laravel behavior

[GromNaN]

Fix PHPORM-229

Laravel Query Builder always returns objects (source).

This is a major breaking change that requires changing all array access into property access. Returning a MongoDB\Model\BSONDocument and MongoDB\Model\BSONArray is not possible as the data is casted with native PHP cast feature in Laravel code: (object) $result and (array) $result.

Checklist

• Add tests and ensure they pass
• Add an entry to the CHANGELOG.md file
• Update documentation for new features

[GromNaN]

Shall we update the type of document to object also, to get the same type for root and embedded documents?

[alcaeus]

If the root document is returned as an object, it would make sense to return all embedded documents as objects as well.

[GromNaN]

Yes, butLaravel have sort of embedded document with "JSON" fields. They are returned as array...

[alcaeus]

Ah, I see. In that case, let's leave it as-is for consistency with Laravel, even though it seems strange in general.

[masterbater]

Is it possible that all v5 pr be merge this week? I just have an expectation that v5 will be release this week. But if not, no pressure just want those big changes to be merge so we can evaluate if we need to add rdbms or just this package can do fine itself.

[alcaeus]

data is casted with native PHP cast feature in Laravel code: (object) $result and (array) $result.

Note that we could leverage the cast_object object handler in extension classes to facilitate converting a PackedArray into an array.

[GromNaN]

data is casted with native PHP cast feature in Laravel code: (object) $result and (array) $result.

Note that we could leverage the cast_object object handler in extension classes to facilitate converting a PackedArray into an array.

Tracking in PHPC-2426