Fat models are slightly better than having that crud in your views, but they're still an antipattern. Models should describe the relationships between entities in your business, not be the junk drawer for stuff that doesn't go anywhere else.Fat models are already a code smell, but passing the request object to your model should really set off alarm bells. You've now made it extremely messy to use