diff options
author | MrYummy <elemental428@gmail.com> | 2017-06-16 22:27:03 +0200 |
---|---|---|
committer | MrYummy <elemental428@gmail.com> | 2017-06-16 22:27:03 +0200 |
commit | 6a0eedc58541e9cf73f25ceef535ff7defde210c (patch) | |
tree | afd9d223baa478c2a22b7bf66c95e7c39be1c9f8 | |
parent | 4e8d94a7b63dc9e4c0002404e8a385e9bf7fa1a2 (diff) |
Fixed compatability issues in methods for thread-like messages
-rw-r--r-- | app/controllers/messagereplies_controller.rb | 1 | ||||
-rw-r--r-- | app/controllers/messages_controller.rb | 2 | ||||
-rw-r--r-- | app/models/message.rb | 6 | ||||
-rw-r--r-- | app/views/messagereplies/_new.html.erb | 1 | ||||
-rw-r--r-- | app/views/messages/index.html.erb | 2 |
5 files changed, 6 insertions, 6 deletions
diff --git a/app/controllers/messagereplies_controller.rb b/app/controllers/messagereplies_controller.rb index 193850a..45c5785 100644 --- a/app/controllers/messagereplies_controller.rb +++ b/app/controllers/messagereplies_controller.rb @@ -19,6 +19,7 @@ class MessagerepliesController < ApplicationController if false @reply.send_new_message_reply_mail end + Message.find(params[:message_id]).update_attributes(user_hidden: nil, user_unread_id: current_user.id) position = message.replies.count - 1 page = position / Kaminari.config.default_per_page + 1 redirect_to message_path(@reply.message, page: page) + "#reply-#{@reply.id}", notice: 'Reply created!' diff --git a/app/controllers/messages_controller.rb b/app/controllers/messages_controller.rb index bcc57cc..5b1d7aa 100644 --- a/app/controllers/messages_controller.rb +++ b/app/controllers/messages_controller.rb @@ -17,7 +17,7 @@ class MessagesController < ApplicationController end def show - Message.find(@message.id).update_attributes(user_unread: nil) if @message.user_unread && @message.user_target.is?(current_user) + Message.find(@message.id).update_attributes(user_unread: nil) unless @message.user_unread == current_user @replies = @message.replies.page(params[:page]) end diff --git a/app/models/message.rb b/app/models/message.rb index 2bbe4cb..5d61557 100644 --- a/app/models/message.rb +++ b/app/models/message.rb @@ -21,11 +21,11 @@ class Message < ActiveRecord::Base before_destroy :do_destroy? def do_destroy? - unless user_hidden || user_sender == user_target + if user_hidden || user_sender == user_target + return true + else update_attributes(user_hidden: User.current) return false - else - return true end end diff --git a/app/views/messagereplies/_new.html.erb b/app/views/messagereplies/_new.html.erb index a882235..a7b7780 100644 --- a/app/views/messagereplies/_new.html.erb +++ b/app/views/messagereplies/_new.html.erb @@ -1,6 +1,5 @@ <%= form_for [reply.get_message, reply] do |f| %> <%= render partial: "md_editor", locals: {name: "messagereply[text]", content: reply.text} %> - <%= f.hidden_field :user_unread, value: current_user %> <p><%= f.submit "Reply", class: "btn blue" %></p> <% end %> diff --git a/app/views/messages/index.html.erb b/app/views/messages/index.html.erb index 54a9348..2cc3df8 100644 --- a/app/views/messages/index.html.erb +++ b/app/views/messages/index.html.erb @@ -26,7 +26,7 @@ <%= render partial: "users/username", locals: { user: user } %> <span style="font-size:16px"> - <span class="<%= "bold" if message.user_unread && message.user_unread != current_user %>"><%= link_to message.subject, message %></span> + <span class="<%= "bold" if message.user_unread_id && message.user_unread != current_user %>"><%= link_to message.subject, message %></span> | </span> <%= ago message.created_at %> |