First off. Great idea for a module. Trying to use it on one of my nonprofit sites. Having some difficulty.

Invalid argument supplied for foreach() in /home/valleyvi/public_html/modules/subscribed/subscribed.module on line 240.

I installed what I believe to be the latest versions of all the modules:
bounced_email.module, v 1.2 2005/11/03 01:16:21
enewsletter.module, v 1.17 2006/01/23 01:02:42
html2txt.module, v 1.5 2005/12/18 18:32:36
identity_hash.module, v 1.3 2005/11/13 12:38:57
publication.module, v 1.12 2006/01/23 01:01:43
schedule.module, v 1.4 2005/11/14 01:12:48
subscribed.module, v 1.5 2006/01/23 01:00:53
templates.module, v 1.4 2005/12/18 18:38:42

When I add a subscriber (user) I get an error in the log
Invalid argument supplied for foreach() in /home/valleyvi/public_html/modules/subscribed/subscribed.module on line 240.
It does this if I have selected all terms (none) or a few.

The site is a test site http:/ValleyViewer.info so it isn't a critical problem right now although I would love to use this module.

I also have not persuded it to actually send out an email. This might be downstream of the error above, not sure.
Cron Run returns:
At least 1 subscriber: YES
warning: Invalid argument supplied for foreach() in /home/valleyvi/public_html/modules/publication/publication.module on line 429.

Comments

robert castelo’s picture

robert castelo commented

Try this....

In subscribed.module, after these lines:

function subscribed_set_terms($type, $publication_id, $uid, $selected_terms) { 
  
  db_query("DELETE FROM {subscribed_terms} WHERE publication_id = '%d' AND uid = '%d' AND type = '%s'", $publication_id, $uid, $type);

Add this line:

if (!is_array($selected_terms)) return;
poisonpill’s picture

poisonpill commented

I get this similar error after running cron.php:

Time to send newsletter: newssss

At least 1 subscriber: YES

warning: Invalid argument supplied for foreach() in /home/.ashton/jeffjung/careergroup.qhc.org/modules/publication/publication.module on line 429.

I entered the code you suggested but same error comes up.

Help!

robert castelo’s picture

robert castelo commented
Status: Active » Closed (won't fix)

Try the Drupal 5 version, Drupal 4.x no longer supported.