Cron error

[shawnb61]

A proper link to line 1413 was provided earlier- see reply 4 above.

[shawnb61]

SOoL -
I sub'd a PR to fix this, found here: https://github.com/SimpleMachines/SMF/pull/8610

In ScheduledTasks.php, find this:

Code Select Expand

// Send the actual email.
if ($notifyPrefs[$row['id_member']] & 0x02)
sendmail($row['email_address'], $emaildata['subject'], $emaildata['body'], null, 'paid_sub_remind', $emaildata['is_html'], 2);

if ($notifyPrefs[$row['id_member']] & 0x01)

Replace with this:

Code Select Expand

// Check notification prefs.
$subs_notify = isset($notifyPrefs[$row['id_member']]['paidsubs_expiring']) ? $notifyPrefs[$row['id_member']]['paidsubs_expiring'] : 0;

// Send the actual email.
if ($subs_notify & 0x02)
sendmail($row['email_address'], $emaildata['subject'], $emaildata['body'], null, 'paid_sub_remind', $emaildata['is_html'], 2);

if ($subs_notify & 0x01)
Let me know how it goes.

[Sir Osis of Liver]

Made the change, will check error log tomorrow.

[shawnb61]

Any updates?

[Sir Osis of Liver]

Last error was logged April 24, 2025, 07:20:46 PM, so that worked.  Will notification emails still be sent?

[shawnb61]

No, because those particular users have no notification preferences set.

I.e., they haven't opted in to receiving any email at all.

[Sir Osis of Liver]

Aren't default prefs set for all members in 2.0.19?

[shawnb61]

Yes, same in 2.1. 

But ever since GDPR, in 2.0 & in 2.1, email requires an opt-in.

[Sir Osis of Liver]

Ok, we'll see what happens.  It's not running in production, the migration to new server went fine but they're getting 403s on some (not all) phone browsers, and host hasn't been able to fix it.