Fix - Prevent bulk delete action on resources with childs

[lguichard]

No description provided.

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Comments	Updated (UTC)
lunar-docs	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	May 23, 2024 8:08am

[alecritson]

Thanks @lguichard the only issue I can potentially see with this is calling the relationship like ->attributes->count() is going to lazy load them for each row which is going to result in quite a lot of heavy queries potentially.

Would it be more efficient to modify the query and load the count like $query->withCount(['attributes']) and then change the check to ->attributes_count instead?

[lguichard]

@alecritson Thanks for the feedback. It was so obvious that I forgot it 😵‍💫

[alecritson]

@alecritson Thanks for the feedback. It was so obvious that I forgot it 😵‍💫

Thanks @lguichard I've made one little tweak to the statement logic, hopefully that's alright and makes sense :)