diff options
author | Johan Sørensen <johan@johansorensen.com> | 2009-03-23 15:58:17 +0100 |
---|---|---|
committer | Johan Sørensen <johan@johansorensen.com> | 2009-04-22 15:17:15 +0200 |
commit | 9197eb52230e4a45ce8ae45e1113e2fc570b48fd (patch) | |
tree | a4f9cfdb9819956176269477e0de5c076a12bbf1 /app/controllers/projects_controller.rb | |
parent | dc60e934369a3a5f4ab24ed2808a68025ca04e8b (diff) | |
download | gitorious-mainline-outdated-9197eb52230e4a45ce8ae45e1113e2fc570b48fd.zip gitorious-mainline-outdated-9197eb52230e4a45ce8ae45e1113e2fc570b48fd.tar.gz gitorious-mainline-outdated-9197eb52230e4a45ce8ae45e1113e2fc570b48fd.tar.bz2 |
Introduce a stale_conditional? controller helper for not setting conditional GET
headers of there's data in the flash
When used, it prevents a certain amount of foot shooting related incidents
Diffstat (limited to 'app/controllers/projects_controller.rb')
-rw-r--r-- | app/controllers/projects_controller.rb | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/app/controllers/projects_controller.rb b/app/controllers/projects_controller.rb index 8178d03..c36894e 100644 --- a/app/controllers/projects_controller.rb +++ b/app/controllers/projects_controller.rb @@ -61,8 +61,7 @@ class ProjectsController < ApplicationController @events = @project.events.top.paginate(:all, :page => params[:page], :order => "created_at desc", :include => [:user, :project]) @atom_auto_discovery_url = project_path(@project, :format => :atom) - if stale?(:etag => [@project, @events.first], - :last_modified => (@events.first || @project).created_at) + if stale_conditional?([@project, @events.first], (@events.first || @project).created_at) respond_to do |format| format.html format.xml { render :xml => @project } @@ -109,6 +108,7 @@ class ProjectsController < ApplicationController @project.attributes = params[:project] if @project.save && @project.wiki_repository.save @project.create_event(Action::UPDATE_PROJECT, @project, current_user) + flash[:success] = "Project details updated" redirect_to project_path(@project) else render :action => 'new' |