tighten / tlint

Tighten linter for Laravel conventions.

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

[Idea] Disallow mass assignment rule

Wulfheart opened this issue · comments

commented

Some Laravel code guides discourage the usage of mass assignment. Would you be willing to merge a rule which lints the use of it?

Personally, no. I much prefer mass assignment with create() and update() over the individual property assignment in that example, and we already have a rule that lints $request->all() to encourage being explicit about passing around request input. Open to hearing what others think though.

I agree with @bakerkretzmar here, thanks for the suggestion @Wulfheart.
This can always be added to a custom preset for your own use 👍