Needs review
Project:
Simplenews
Version:
7.x-1.x-dev
Component:
Code
Priority:
Normal
Category:
Feature request
Assigned:
Unassigned
Reporter:
Created:
22 Feb 2017 at 02:43 UTC
Updated:
16 Mar 2017 at 11:17 UTC
Jump to comment: Most recent, Most recent file

Comments
Comment #2
monstrfolk commentedComment #3
tfranz commentedThank you – the patch worked as expected!!
But i had a small problem with Simplenews Scheduler after applying the patch. I got a PHP - Fatal error: "Unsupported operand types".
In simplenews_scheduler.module i had to change the following lines 82ff to get it to work:
Comment #4
monstrfolk commentedcan you please give more details on what you had to fix.
Comment #5
tfranz commentedAfter applying your patch to simplenews 7.x-1.1 (with simplenews_scheduler 7.x-1.0-beta2) i received the following error visiting the page /node/[nid]/simplenews :
Fatal error: Unsupported operand types in /sites/all/modules/simplenews_scheduler/simplenews_scheduler.module on line 84
I needed to change line 84 in simplenews_scheduler.module to solve the error.
The problem seems to be the "array += array" in line 84. I simply changed it to "array_merge(array, array)".
Maybe $form['simplenews']['send']['#options'] isn't an array any more after your patch?
Attached is my patch for simplenews_scheduler.module.
Does this help you?