Yes, that would probably look better, but all the other actions in the controller are written like this, and I felt it was more coherent to maintain the same style throughout the file. Of course that doesn't mean it should stay like this, but it doesn't belong in this pull request to change all the actions. Maybe it could be addressed in another pull request.

And don't worry. I think it's interesting to receive all kinds of feedback (style, performance, design, you name it).