From 9d23b0d00c874a8c55cfe2f2d0237277f59baea3 Mon Sep 17 00:00:00 2001 From: Tom Hughes Date: Sat, 23 Mar 2013 17:35:10 +0000 Subject: [PATCH 1/1] Improve selection logic in DiaryEntry#rss --- app/controllers/diary_entry_controller.rb | 27 +++++++++++++---------- 1 file changed, 15 insertions(+), 12 deletions(-) diff --git a/app/controllers/diary_entry_controller.rb b/app/controllers/diary_entry_controller.rb index 12f3fdcc8..777034468 100644 --- a/app/controllers/diary_entry_controller.rb +++ b/app/controllers/diary_entry_controller.rb @@ -141,30 +141,33 @@ class DiaryEntryController < ApplicationController end def rss - @entries = DiaryEntry.includes(:user).order("created_at DESC").limit(20) - if params[:display_name] user = User.active.find_by_display_name(params[:display_name]) if user - @entries = user.diary_entries.visible + @entries = user.diary_entries @title = I18n.t('diary_entry.feed.user.title', :user => user.display_name) @description = I18n.t('diary_entry.feed.user.description', :user => user.display_name) @link = "http://#{SERVER_URL}/user/#{user.display_name}/diary" else render :nothing => true, :status => :not_found end - elsif params[:language] - @entries = @entries.visible.where(:language_code => params[:language]).joins(:user).where(:users => { :status => ["active", "confirmed"] }) - @title = I18n.t('diary_entry.feed.language.title', :language_name => Language.find(params[:language]).english_name) - @description = I18n.t('diary_entry.feed.language.description', :language_name => Language.find(params[:language]).english_name) - @link = "http://#{SERVER_URL}/diary/#{params[:language]}" else - @entries = @entries.visible.joins(:user).where(:users => { :status => ["active", "confirmed"] }) - @title = I18n.t('diary_entry.feed.all.title') - @description = I18n.t('diary_entry.feed.all.description') - @link = "http://#{SERVER_URL}/diary" + @entries = DiaryEntry.joins(:user).where(:users => { :status => ["active", "confirmed"] }) + + if params[:language] + @entries = @entries.where(:language_code => params[:language]) + @title = I18n.t('diary_entry.feed.language.title', :language_name => Language.find(params[:language]).english_name) + @description = I18n.t('diary_entry.feed.language.description', :language_name => Language.find(params[:language]).english_name) + @link = "http://#{SERVER_URL}/diary/#{params[:language]}" + else + @title = I18n.t('diary_entry.feed.all.title') + @description = I18n.t('diary_entry.feed.all.description') + @link = "http://#{SERVER_URL}/diary" + end end + + @entries = @entries.visible.includes(:user).order("created_at DESC").limit(20) end def view -- 2.39.5