reminder.rake should ignore blank parameters (#31501).

Patch by Go MAEDA.

git-svn-id: http://svn.redmine.org/redmine/trunk@18226 e93f8b46-1217-0410-a6f0-8f06a7374b81

diff --git a/lib/tasks/reminder.rake b/lib/tasks/reminder.rake
index 52ac1c7a7530e3438d854de2b853aae75c47bd35..16c43493314149897ce92326235c27b30238dca1 100644 (file)
--- a/lib/tasks/reminder.rake
+++ b/lib/tasks/reminder.rake
@@ -32,11 +32,11 @@ END_DESC
 namespace :redmine do
   task :send_reminders => :environment do
     options = {}
-    options[:days] = ENV['days'].to_i if ENV['days']
-    options[:project] = ENV['project'] if ENV['project']
-    options[:tracker] = ENV['tracker'].to_i if ENV['tracker']
-    options[:users] = (ENV['users'] || '').split(',').each(&:strip!)
-    options[:version] = ENV['version'] if ENV['version'] 
+    options[:days] = ENV['days'].presence&.to_i
+    options[:project] = ENV['project'].presence
+    options[:tracker] = ENV['tracker'].presence&.to_i
+    options[:users] = ENV['users'].presence.to_s.split(',').each(&:strip!)
+    options[:version] = ENV['version'].presence
 
     Mailer.with_synched_deliveries do
       Mailer.reminders(options)