diff options
| author | Patrick J Cherry <patrick@bytemark.co.uk> | 2011-06-15 19:47:00 +0100 | 
|---|---|---|
| committer | Patrick J Cherry <patrick@bytemark.co.uk> | 2011-06-15 19:47:00 +0100 | 
| commit | 3576923edf788eecb4320e034f5afaaf625bd62e (patch) | |
| tree | 9c3593ae713a7cec39d0886ee0988c39737cf41f /lib | |
| parent | 42f14de6ab50d0e05f49f038e40b94b686701bf1 (diff) | |
Missed from last commit
Diffstat (limited to 'lib')
| -rw-r--r-- | lib/mauve/alert_changed.rb | 26 | 
1 files changed, 14 insertions, 12 deletions
| diff --git a/lib/mauve/alert_changed.rb b/lib/mauve/alert_changed.rb index 4668fd7..68a0a00 100644 --- a/lib/mauve/alert_changed.rb +++ b/lib/mauve/alert_changed.rb @@ -52,10 +52,6 @@ module Mauve       Log4r::Logger.new self.class.to_s      end -##    def initialize -#     logger = Log4r::Logger.new self.class.to_s -#    end -      ## Checks to see if a raise was send to the person.      #      # @TODO: Recurence is broken in ruby, change this so that it does not  @@ -100,14 +96,20 @@ module Mauve          save        else          saved = false -         -        alert_group.notifications.each do |notification| -          notification.people.each do |person| -            if person.username == self.person -              person.remind(alert, level) -              self.remind_at = notification.remind_at_next(alert) -              save -              saved = true + +        unless alert_group.notifications.nil? + +          alert_group.notifications.each do |notification| +            notification.people.each do |person| +              # Not interested in nil people. +              next if person.nil? +   +              if person.username == self.person +                person.remind(alert, level) +                self.remind_at = notification.remind_at_next(alert) +                save +                saved = true +              end              end            end          end | 
