ACK – Now, it works great ;)

--
Benjamin LAN-SUN-LUK



Le 30/11/08 17:56, « Darryl L. Pierce » <mcpierce@gmail.com> a écrit :

This patch fixes the nil error raised for users with no tasks to report.

Added a new column to the notifications table to toggle sending
the notifications.

Updated the user's profile page to include an option to toggle this
on and off. Also changed the notifications view to be more intuitive.

When daily updates are collected, then all affected products are CC'd
with the user's daily update. CC'ing product lists requires the user
to have daily updates enabled.

Signed-off-by: Darryl L. Pierce <mcpierce@gmail.com>
---
 config/initializers/schedules.rb |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/config/initializers/schedules.rb b/config/initializers/schedules.rb
index 83be588..b1fc4f2 100644
--- a/config/initializers/schedules.rb
+++ b/config/initializers/schedules.rb
@@ -63,7 +63,7 @@ threads["daily updates"] = Thread.new do
     end

     User.find(:all).each do |user|
-      if user.notifications.daily_updates
+      if user.notifications.daily_updates && tasks_performed[user]
         cc_list = []

         if user.notifications.updates_to_product_list
--
1.6.0.2

_______________________________________________
projxp-devel mailing list
projxp-devel@lists.fedorahosted.org
https://fedorahosted.org/mailman/listinfo/projxp-devel