diff options
author | Matthew Landauer <matthew@openaustralia.org> | 2013-01-31 12:44:29 +1100 |
---|---|---|
committer | Matthew Landauer <matthew@openaustralia.org> | 2013-01-31 12:44:29 +1100 |
commit | d85a11d1a6072abe48ff75cc47624e104fa42c4a (patch) | |
tree | 9f2398856f1c325fe3d7644ed3383400b7c1ddbc | |
parent | d5327fc2fe691ee2bc427dc5f25af47508044be0 (diff) |
Fix rendering of atom feed and set correct content type
-rw-r--r-- | app/controllers/track_controller.rb | 2 | ||||
-rw-r--r-- | spec/support/validate_html.rb | 2 |
2 files changed, 2 insertions, 2 deletions
diff --git a/app/controllers/track_controller.rb b/app/controllers/track_controller.rb index 51e081c88..9b4746bde 100644 --- a/app/controllers/track_controller.rb +++ b/app/controllers/track_controller.rb @@ -157,10 +157,10 @@ class TrackController < ApplicationController def atom_feed_internal @xapian_object = perform_search([InfoRequestEvent], @track_thing.track_query, @track_thing.params[:feed_sortby], nil, 25, 1) respond_to do |format| - format.atom { render :template => 'track/atom_feed' } format.json { render :json => @xapian_object.results.map { |r| r[:model].json_for_api(true, lambda { |t| @template.highlight_and_excerpt(t, @xapian_object.words_to_highlight, 150) } ) } } + format.any { render :template => 'track/atom_feed.atom', :layout => false, :content_type => :atom } end end diff --git a/spec/support/validate_html.rb b/spec/support/validate_html.rb index 93031ef46..56f384537 100644 --- a/spec/support/validate_html.rb +++ b/spec/support/validate_html.rb @@ -45,7 +45,7 @@ if $tempfilecount.nil? # don't validate auto-generated HTML return if @request.query_parameters["action"] == "get_attachment_as_html" # XXX Is there a better way to check this than calling a private method? - return if @response.body.empty? + return if @response.body.strip.empty? # And then if HTML, not a redirect (302, 301) if @response.content_type == "text/html" && ! [301,302,401].include?(@response.response_code) if !is_fragment |