diff options
| author | Gareth Rees <gareth@mysociety.org> | 2014-06-16 10:30:46 +0100 |
|---|---|---|
| committer | Gareth Rees <gareth@mysociety.org> | 2014-06-16 10:30:46 +0100 |
| commit | d5812d4f687c1dbe5513fde85969eecb0a87a9b8 (patch) | |
| tree | ad940008d4fea55aa8bc54e466ddd1193e188923 /app/controllers/application_controller.rb | |
| parent | 170b2508e9e07e4b430d25c6f4d7ae3930caa07d (diff) | |
| parent | 42870985da06418461c847563fa860c11b4094a1 (diff) | |
Merge branch 'replace-xxx-todo' into rails-3-develop
Diffstat (limited to 'app/controllers/application_controller.rb')
| -rw-r--r-- | app/controllers/application_controller.rb | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/app/controllers/application_controller.rb b/app/controllers/application_controller.rb index 78a82316a..0c5f5bd02 100644 --- a/app/controllers/application_controller.rb +++ b/app/controllers/application_controller.rb @@ -278,10 +278,10 @@ class ApplicationController < ActionController::Base session[:post_redirect_token] = post_redirect.token - # XXX what is the built in Ruby URI munging function that can do this + # TODO: what is the built in Ruby URI munging function that can do this # choice of & vs. ? more elegantly than this dumb if statement? if uri.include?("?") - # XXX This looks odd. What would a fragment identifier be doing server-side? + # TODO: This looks odd. What would a fragment identifier be doing server-side? # But it also looks harmless, so I’ll leave it just in case. if uri.include?("#") uri.sub!("#", "&post_redirect=1#") |