def new
@title = 'send message'
if params[:message]
- to_user = User.find(params[:user_id])
- body = params[:message][:body]
- title = params[:message][:title]
- message = Message.new
- message.body = body
- message.title = title
- message.to_user_id = params[:user_id]
- message.from_user_id = @user.id
- message.sent_on = Time.now
+ @message = Message.new(params[:message])
+ @message.to_user_id = params[:user_id]
+ @message.from_user_id = @user.id
+ @message.sent_on = Time.now
- if message.save
+ if @message.save
flash[:notice] = 'Message sent'
+ Notifier::deliver_message_notification(@message)
redirect_to :controller => 'message', :action => 'inbox', :display_name => @user.display_name
- else
- @message.errors.add("Sending message failed")
end
-
end
end
def read
@title = 'read message'
- if params[:message_id]
- id = params[:message_id]
- @message = Message.find_by_id(id)
- @message.message_read = 1
- @message.save
- end
+ @message = Message.find(params[:message_id], :conditions => ["to_user_id = ? or from_user_id = ?", @user.id, @user.id ])
+ @message.message_read = 1 if @message.to_user_id == @user.id
+ @message.save
+ rescue ActiveRecord::RecordNotFound
+ render :nothing => true, :status => :not_found
end
def inbox
end
end
+ def outbox
+ @title = 'outbox'
+ if @user and params[:display_name] == @user.display_name
+ else
+ redirect_to :controller => 'message', :action => 'outbox', :display_name => @user.display_name
+ end
+ end
+
def mark
if params[:message_id]
id = params[:message_id]