diff options
author | Azul <azul@leap.se> | 2014-02-10 14:23:38 +0100 |
---|---|---|
committer | Azul <azul@leap.se> | 2014-02-10 14:23:38 +0100 |
commit | bd867c51c4d9e3d4c6b4c55d326eb9b13b89288b (patch) | |
tree | 9c03e20ec93093e9bb2ec47e982bdfcaec5c055e /users/app/controllers/v1/messages_controller.rb | |
parent | bcdde2f6bfb4ed3a1535bd2e50ab47529a9141e2 (diff) |
minor: move some logic from message controller into model
Diffstat (limited to 'users/app/controllers/v1/messages_controller.rb')
-rw-r--r-- | users/app/controllers/v1/messages_controller.rb | 5 |
1 files changed, 1 insertions, 4 deletions
diff --git a/users/app/controllers/v1/messages_controller.rb b/users/app/controllers/v1/messages_controller.rb index 1b994ca..55292ff 100644 --- a/users/app/controllers/v1/messages_controller.rb +++ b/users/app/controllers/v1/messages_controller.rb @@ -13,10 +13,7 @@ module V1 def update message = Message.find(params[:id]) if (message and current_user) - message.user_ids_to_show.delete(current_user.id) - # is it necessary to keep track of what users have already seen it? - message.user_ids_have_shown << current_user.id if !message.user_ids_have_shown.include?(current_user.id) - # TODO: is it quicker to call uniq! after adding rather than check if it is already included? + message.mark_as_read_by(current_user) message.save render json: true else |