nocebo/diaspora
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

notification eagerly load the target

Browse source
This commit is contained in:
zhitomirskiyi 2011-01-18 17:22:48 -08:00
parent 5ac194911f
commit 21bccedf9b
2 changed files with 3 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

3
app/controllers/notifications_controller.rb
View file

@ -18,7 +18,8 @@ class NotificationsController < ApplicationController
end end
def index def index
@notifications = Notification.for(current_user).include(:target).paginate :page => params[:page], :per_page => 25 @notifications = Notification.find(:all, :conditions => {:recipient_id => current_user.id},
:order => 'created_at desc', :include => [:target]).paginate :page => params[:page], :per_page => 25
@group_days = @notifications.group_by{|note| note.created_at.strftime("%B %d") } @group_days = @notifications.group_by{|note| note.created_at.strftime("%B %d") }
respond_with @notifications respond_with @notifications
end end

2
app/models/notification.rb
View file

@ -11,7 +11,7 @@ class Notification < ActiveRecord::Base
belongs_to :target, :polymorphic => true belongs_to :target, :polymorphic => true
def self.for(recipient, opts={}) def self.for(recipient, opts={})
self.where(opts.merge!(:recipient_id => recipient.id)).order('created_at desc').include(:target) self.where(opts.merge!(:recipient_id => recipient.id)).order('created_at desc')
end end
def self.notify(recipient, target, actor) def self.notify(recipient, target, actor)