- New routes were defined (/player/:id/game_type_cd) when a query
parameter is the better fit. Game types are a filter condition
on the resource, not the resource itself, thus should not be in
the route.
- Pagination was broken. Passing search_query to route_url makes
the query parameters persist, although I pass query parameters
in an awful way that needs to be fixed.
- Multiple lines just to show the game type description when an if
statement would do the same thing.
- No need to pass the player_url to the template. You can reverse-
generate URLs to routes with route_url just fine.
- Remove the extra link back to the player_info page, as the
pattern throughout the rest of the application is to click the
nick itself.