Skip to content
Snippets Groups Projects

Send reminder.service fix

Closed its4you requested to merge its4you/vtigercrm:SendReminder.servicefix into 7.4.0

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • its4you Title changed from Send reminder.servicefix to Send reminder.service fix

    Title changed from Send reminder.servicefix to Send reminder.service fix

  • @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.

  • Author Contributor

    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.

  • Uma Reassigned to @vtiger

    Reassigned to @vtiger

  • @its4you Thanks! for the contribution, Fix has been pulled #1494 (closed)

  • Uma Status changed to closed

    Status changed to closed

Please register or sign in to reply
Loading