diff options
author | jessib <jessib@riseup.net> | 2013-12-24 13:27:22 -0800 |
---|---|---|
committer | jessib <jessib@riseup.net> | 2013-12-24 13:27:22 -0800 |
commit | 7f12c795207ac818bffac42aa581bf1165f9e424 (patch) | |
tree | 59dbe67df59793ec58a8b1521fe39caf9e37a712 /users/app/controllers/v1 | |
parent | e4390e2ee5b2df20038f12865db462cf1e208ee6 (diff) |
Catching some corner cases & new tests.
Diffstat (limited to 'users/app/controllers/v1')
-rw-r--r-- | users/app/controllers/v1/messages_controller.rb | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/users/app/controllers/v1/messages_controller.rb b/users/app/controllers/v1/messages_controller.rb index fa98042..d49b161 100644 --- a/users/app/controllers/v1/messages_controller.rb +++ b/users/app/controllers/v1/messages_controller.rb @@ -10,14 +10,13 @@ module V1 render json: (user ? user.messages : [] ) end - # routes ensure this is only for PUT def mark_read user_message = UserMessage.find_by_user_id_and_message_id([params[:user_id], params[:message_id]]) - user_message.seen = true + user_message.seen = true if user_message # TODO what to return? - if user_message.save + if user_message and user_message.save render json: true else render json: false |