]> de.git.xonotic.org Git - xonotic/xonstat.git/commit
Fix the broken things in player_game_index.
authorAnt Zucaro <azucaro@gmail.com>
Fri, 19 Apr 2013 00:30:55 +0000 (20:30 -0400)
committerAnt Zucaro <azucaro@gmail.com>
Fri, 19 Apr 2013 00:30:55 +0000 (20:30 -0400)
commit371e4d7ce59cbe02aa409604ecc7e70d5626d965
treecddab3843546fb725cfdacb5d0d11d7cec025dc6
parent67770f6a980ef9bc98f4f1c39fad39669816e9a0
Fix the broken things in player_game_index.

- 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.
xonstat/__init__.py
xonstat/templates/player_game_index.mako
xonstat/views/player.py