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 destroy
+ @message = Message.find(params[:message_id], :conditions => ["to_user_id = ? or from_user_id = ?", @user.id, @user.id ])
+ if !@message.message_read
+ flash[:notice] = 'Message not read and so not deleted'
+ redirect_to :controller => 'message', :action => 'inbox', :display_name => @user.display_name
+ else
+ flash[:notice] = "Message '#{@message.title}' deleted"
+ @message.destroy
+ redirect_to :controller => 'message', :action => 'inbox', :display_name => @user.display_name
end
end
+ def reply
+ message = Message.find(params[:message_id], :conditions => ["to_user_id = ? or from_user_id = ?", @user.id, @user.id ])
+ @body = "\n\nOn #{message.sent_on} #{message.sender.display_name} wrote:\n #{message.body}"
+ @title = "Re: #{message.title}"
+ @user_id = message.from_user_id
+ render :action => 'new'
+ rescue ActiveRecord::RecordNotFound
+ render :nothing => true, :status => :not_found
+ end
+
def read
@title = 'read message'
- if params[:message_id]
- id = params[:message_id]
- @message = Message.find_by_id(id)
- 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]
message = Message.find_by_id(id)
- message.message_read = 1
+ if params[:mark] == 'unread'
+ message_read = 0
+ mark_type = 'unread'
+ else
+ message_read = 1
+ mark_type = 'read'
+ end
+ message.message_read = message_read
if message.save
- flash[:notice] = 'Message marked as read'
+ flash[:notice] = "Message marked as #{mark_type}"
redirect_to :controller => 'message', :action => 'inbox', :display_name => @user.display_name
end
end