Dear friends,

Thanks for this module.

I am reading source code and I can see a problem:
'$where = '(fus.date IS NULL AND (coalesce(oac.created, ru.created) + %d * 3600 < UNIX_TIMESTAMP())) AND (ru.remaining_intervals <> 0) ';

unix_timestamp() is MySQLism, it does not exist in SQL99 databases.

Instead, use PHP time() function.

Patch attached.

CommentFileSizeAuthor
uc_follow_up.patch3.17 KBgrub3

Comments

grub3’s picture

grub3 commented
Status: Active » Needs review
m.stenta’s picture

m.stenta
he/him
Primary language English
commented
Version: » 6.x-1.x-dev

Thanks, this is fixed in the uc_recurring branch now. Note that the uc_recurring branch used to be called the 6.x-1.x branch, but that was incorrect because all the releases made thusfar have been tagged on the master branch. I created a new issue for merging them: #1486508: Merge uc_recurring branch into 6.x-1.x

m.stenta’s picture

m.stenta
he/him
Primary language English
commented
Status: Needs review » Closed (fixed)