From aec391621b6af7af1e0ba61c993bbdd88624eac3 Mon Sep 17 00:00:00 2001 From: Michael Koziarski Date: Tue, 7 Oct 2008 21:09:07 +0200 Subject: Make sure last_modified! works with <= rather than just equality. --- actionpack/lib/action_controller/base.rb | 4 +++- actionpack/test/controller/render_test.rb | 6 ++++++ 2 files changed, 9 insertions(+), 1 deletion(-) diff --git a/actionpack/lib/action_controller/base.rb b/actionpack/lib/action_controller/base.rb index 413f6d48e5..3ede681253 100644 --- a/actionpack/lib/action_controller/base.rb +++ b/actionpack/lib/action_controller/base.rb @@ -969,7 +969,9 @@ module ActionController #:nodoc: # If-Modified-Since request header is <= last modified. def last_modified!(utc_time) response.last_modified= utc_time - head(:not_modified) if response.last_modified == request.if_modified_since + if request.if_modified_since && request.if_modified_since <= utc_time + head(:not_modified) + end end # Sets the ETag response header. Returns 304 Not Modified if the diff --git a/actionpack/test/controller/render_test.rb b/actionpack/test/controller/render_test.rb index 5a6ca98b2e..7b8bb6856b 100644 --- a/actionpack/test/controller/render_test.rb +++ b/actionpack/test/controller/render_test.rb @@ -1441,6 +1441,12 @@ class LastModifiedRenderTest < Test::Unit::TestCase get :conditional_hello_with_bangs assert_response :not_modified end + + def test_last_modified_works_with_less_than_too + @request.if_modified_since = 5.years.ago.httpdate + get :conditional_hello_with_bangs + assert_response :not_modified + end end class RenderingLoggingTest < Test::Unit::TestCase -- cgit v1.2.3