diff options
Diffstat (limited to 'app')
| -rw-r--r-- | app/controllers/admin_public_body_controller.rb | 12 | ||||
| -rw-r--r-- | app/helpers/link_to_helper.rb | 7 | ||||
| -rw-r--r-- | app/views/layouts/admin.rhtml | 3 |
3 files changed, 11 insertions, 11 deletions
diff --git a/app/controllers/admin_public_body_controller.rb b/app/controllers/admin_public_body_controller.rb index bc2dd90fa..9803c1f65 100644 --- a/app/controllers/admin_public_body_controller.rb +++ b/app/controllers/admin_public_body_controller.rb @@ -4,7 +4,7 @@ # Copyright (c) 2007 UK Citizens Online Democracy. All rights reserved. # Email: francis@mysociety.org; WWW: http://www.mysociety.org/ # -# $Id: admin_public_body_controller.rb,v 1.4 2007-11-13 12:59:53 francis Exp $ +# $Id: admin_public_body_controller.rb,v 1.5 2007-12-14 13:24:04 francis Exp $ class AdminPublicBodyController < ApplicationController layout "admin" @@ -14,10 +14,6 @@ class AdminPublicBodyController < ApplicationController render :action => 'list' end - # GETs should be safe (see http://www.w3.org/2001/tag/doc/whenToUseGet.html) - verify :method => :post, :only => [ :destroy, :create, :update ], - :redirect_to => { :action => :list } - def list @public_bodies = PublicBody.paginate :order => "name", :page => params[:page], :per_page => 10 end @@ -35,7 +31,7 @@ class AdminPublicBodyController < ApplicationController @public_body = PublicBody.new(params[:public_body]) if @public_body.save flash[:notice] = 'PublicBody was successfully created.' - redirect_to :action => 'list' + redirect_to admin_url('body/list') else render :action => 'new' end @@ -51,7 +47,7 @@ class AdminPublicBodyController < ApplicationController @public_body = PublicBody.find(params[:id]) if @public_body.update_attributes(params[:public_body]) flash[:notice] = 'PublicBody was successfully updated.' - redirect_to :action => 'show', :id => @public_body + redirect_to admin_url('body/show/' + @public_body.id.to_s) else render :action => 'edit' end @@ -59,7 +55,7 @@ class AdminPublicBodyController < ApplicationController def destroy PublicBody.find(params[:id]).destroy - redirect_to :action => 'list' + redirect_to admin_url('body/list') end private diff --git a/app/helpers/link_to_helper.rb b/app/helpers/link_to_helper.rb index 24ee8e578..8eeed45b5 100644 --- a/app/helpers/link_to_helper.rb +++ b/app/helpers/link_to_helper.rb @@ -5,7 +5,7 @@ # Copyright (c) 2007 UK Citizens Online Democracy. All rights reserved. # Email: francis@mysociety.org; WWW: http://www.mysociety.org/ # -# $Id: link_to_helper.rb,v 1.2 2007-11-06 16:05:29 francis Exp $ +# $Id: link_to_helper.rb,v 1.3 2007-12-14 13:24:04 francis Exp $ module LinkToHelper @@ -41,6 +41,11 @@ module LinkToHelper text.gsub!(/[^a-z0-9_-]/, "") text end + + def admin_url(relative_path) + admin_url_prefix = MySociety::Config.get("ADMIN_BASE_URL", "/admin/") + return admin_url_prefix + relative_path + end end diff --git a/app/views/layouts/admin.rhtml b/app/views/layouts/admin.rhtml index b856632d5..ad30f4924 100644 --- a/app/views/layouts/admin.rhtml +++ b/app/views/layouts/admin.rhtml @@ -11,8 +11,7 @@ <body> <p><b>FOIFA admin interface:</b> -<% admin_url_prefix = MySociety::Config.get("ADMIN_BASE_URL", "/admin/") %> -<%= link_to 'Public bodies', admin_url_prefix + 'body/list' %> +<%= link_to 'Public bodies', admin_url('body/list') %> </p> <p style="color: green"><%= flash[:notice] %></p> |