From 1add11d8c9765e60584b599996973e1f496ad5d0 Mon Sep 17 00:00:00 2001 From: Maja Komel Date: Mon, 1 Oct 2018 17:41:23 +0200 Subject: [PATCH] FIX: don't send multiple requests when changing category notification (#6435) --- .../components/category-notifications-button.js.es6 | 4 +++- app/models/category_user.rb | 6 +++++- 2 files changed, 8 insertions(+), 2 deletions(-) diff --git a/app/assets/javascripts/select-kit/components/category-notifications-button.js.es6 b/app/assets/javascripts/select-kit/components/category-notifications-button.js.es6 index 602369f5212..941e21f96d5 100644 --- a/app/assets/javascripts/select-kit/components/category-notifications-button.js.es6 +++ b/app/assets/javascripts/select-kit/components/category-notifications-button.js.es6 @@ -11,5 +11,7 @@ export default NotificationOptionsComponent.extend({ mutateValue(value) { this.get("category").setNotification(value); - } + }, + + deselect() {} }); diff --git a/app/models/category_user.rb b/app/models/category_user.rb index 321089335eb..bccd7cbef43 100644 --- a/app/models/category_user.rb +++ b/app/models/category_user.rb @@ -55,7 +55,11 @@ class CategoryUser < ActiveRecord::Base record.notification_level = level record.save! else - CategoryUser.create!(user: user, category_id: category_id, notification_level: level) + begin + CategoryUser.create!(user: user, category_id: category_id, notification_level: level) + rescue ActiveRecord::RecordNotUnique + # does not matter + end end auto_watch(user_id: user.id)