Sutty/trabajo-afectivo
5
0
Fork 0
Code Issues Pull requests Projects Releases Wiki Activity

Fixed reminder notifications.

Browse source
This commit is contained in:
Martin Edenhofer 2016-02-22 13:54:28 +01:00
parent 50971e4425
commit 2d4b7dab56
2 changed files with 5 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
app/models/observer/ticket/notification/background_job.rb
View file

@ -140,7 +140,7 @@ class Observer::Ticket::Notification::BackgroundJob
next if history['type'] != 'notification'
next if history['value_to'] !~ /\(#{Regexp.escape(@p[:type])}:/
next if history['value_to'] !~ /#{Regexp.escape(identifier)}\(/
next if history['created_at'].today?
next if !history['created_at'].today?
already_notified = true
}
next if already_notified
@ -154,7 +154,7 @@ class Observer::Ticket::Notification::BackgroundJob
# delete old notifications
if @p[:type] == 'reminder_reached' || @p[:type] == 'escalation'
seen = false
OnlineNotification.remove_by_type('Ticket', ticket.id, @p[:type])
OnlineNotification.remove_by_type('Ticket', ticket.id, @p[:type], user)
# on updates without state changes create unseen messages
elsif @p[:type] != 'create' && (!@p[:changes] || @p[:changes].empty? || !@p[:changes]['state_id'])

5
app/models/online_notification.rb
View file

@ -88,17 +88,18 @@ remove whole online notifications of an object
remove whole online notifications of an object by type
OnlineNotification.remove_by_type('Ticket', 123, type)
OnlineNotification.remove_by_type('Ticket', 123, type, user)
=end
def self.remove_by_type(object_name, o_id, type)
def self.remove_by_type(object_name, o_id, type, user)
object_id = ObjectLookup.by_name(object_name)
type_id = TypeLookup.by_name(type)
OnlineNotification.where(
object_lookup_id: object_id,
type_lookup_id: type_id,
o_id: o_id,
user_id: user.id,
).destroy_all
end