Send reminder.service fix
Merge request reports
Activity
@its4you Can you please share the case scenario for this fix? so that we can link to an issue. Possibly please compress the commits on this Merge request to one with the fix.
Send reminder is a scheduled service. If you check "Send Email Reminder Before" in the Calendar module, then an Email notification should be sent. In SendReminder.service file $to_addr='' variable is defined as String and later use as array
while($user_result_row =
adb->fetch_array(user_result)) { if($user_result_row['email1']!='' || $user_result_row['email1'] !=NULL) {to_addr[user_result_row['smid']] = $user_result_row['email1']; } $invitedUsersList[] = $user_result_row['smid']; }Then the script ends with fatal error and the scheduled reminder doesn't work.
@its4you Thanks! for the shared observations and fix.
Reassigned to @vtiger
@its4you Thanks! for the contribution, Fix has been pulled #1494 (closed)