Implemented Change TO_MANY @property definitions to include \XF\Mvc\Entity\ArrayCollection

DragonByte Tech

Well-known member
This is clearly the most important suggestion of all time and should be given maximum priority 😛

At the moment, phpStorm doesn't recognise that our entities' relations are ArrayCollections:

1f8tOjw.png


If I then change the relevant comment to * @property \DBTech\Credits\Entity\ChargePurchase[]|\XF\Mvc\Entity\ArrayCollection Purchases

ACBJB9F.png


It's not just me being anal retentive about seeing warnings in my code... okay it's mostly not about that... okay it's partially not about that; it might help new developers realise that you can actually perform certain operations on TO_MANY relations as they will be considered ArrayCollections, and not just an array of entities.


Fillip
 
Upvote 0
This suggestion has been implemented. Votes are no longer accepted.
Top Bottom