codeodor/How-to-avoid-becoming-a-formerly-employed-rails-developer-standing-in-line-at-the-OOP-kitchen — Issue #3: registrations_controller#create does too much, contains business logic, and violates "thin controllers" rule
Author: codeodor
Created: 2012-02-16 00:59:22+00:00
State: open

This method is doing four things:

1. creating the registration
2. setting the fee in case the user switched it in the params sent in the request
3. creating a study group
4. calculating the scholarship discount

To be highly cohesive, this method should do one thing, not four.