DG IZ; using group_id instead of group in post method. passing in :group_id into publisher partial
This commit is contained in:
parent
b0088c73d8
commit
055c222fc2
5 changed files with 10 additions and 6 deletions
|
|
@ -42,8 +42,8 @@ class User
|
|||
def post(class_name, options = {})
|
||||
options[:person] = self.person
|
||||
|
||||
group = options[:group]
|
||||
options.delete(:group)
|
||||
group_id = options[:group_id]
|
||||
options.delete(:group_id)
|
||||
|
||||
model_class = class_name.to_s.camelize.constantize
|
||||
post = model_class.instantiate(options)
|
||||
|
|
@ -56,7 +56,8 @@ class User
|
|||
self.raw_visible_posts << post
|
||||
self.save
|
||||
|
||||
if group
|
||||
if group_id
|
||||
group = self.groups.find_by_id(group_id)
|
||||
group.my_posts << post
|
||||
group.save
|
||||
end
|
||||
|
|
|
|||
|
|
@ -2,7 +2,7 @@
|
|||
welcome home,
|
||||
= current_user.profile.first_name
|
||||
|
||||
= render "shared/publisher"
|
||||
= render "shared/publisher", :group_id => @group.id
|
||||
|
||||
%ul#stream
|
||||
- for post in @posts
|
||||
|
|
|
|||
|
|
@ -1,4 +1,4 @@
|
|||
= render "shared/publisher"
|
||||
= render "shared/publisher", :group_id => @group.id
|
||||
%ul#stream
|
||||
- for post in @posts
|
||||
= render type_partial(post), :post => post
|
||||
|
|
|
|||
|
|
@ -3,6 +3,9 @@
|
|||
#publisher_form
|
||||
= form_for StatusMessage.new, :remote => true do |f|
|
||||
= f.error_messages
|
||||
|
||||
= f.hidden_field :group_id, :value => group_id
|
||||
|
||||
%p
|
||||
%label{:for => "status_message_message"} Message
|
||||
= f.text_area :message, :rows => 2
|
||||
|
|
|
|||
|
|
@ -57,7 +57,7 @@ describe Group do
|
|||
it 'should add post to group via post method' do
|
||||
@group = @user.group(:name => 'losers', :people => [@friend])
|
||||
|
||||
status_message = @user.post( :status_message, :message => "hey", :group => @group )
|
||||
status_message = @user.post( :status_message, :message => "hey", :group_id => @group.id )
|
||||
|
||||
@group.reload
|
||||
@group.my_posts.include?(status_message).should be true
|
||||
|
|
|
|||
Loading…
Reference in a new issue