Hi,

Thanks for the great module. I've been playing around with it and most of the stuff works great but adding feeds tosses this error message:

warning: implode() [function.implode]: Bad arguments. in /home/testsite/public_html/groups/modules/mysite/plugins/types/feed.inc on line 338.

I'm using 4.7.4 with mysite 4.7.3. Hummmm, is that my problem?

The feeds I've added work.

Jim

Comments

agentrickard’s picture

agentrickard
he/him
Primary language English
Location Georgia (US)
commented

No, the 4.7.x modules work with any version of 4.7 Drupal.

I think the problem there is that you haven't configured the feed categories.

This bug has been reported before, and I think it's administrator error. Certainly the module could do error capture here, but I think getting this error is ok while installing the module.

This is covered in the INSTALL.txt and the help documentation:

http://example.com/?q=admin/help/mysite

To fix, go to:

http://example.com/?q=admin/settings/mysite/type/feed

Be sure to select at least one Allowed Feed Category, save the settings, and then select a Default Feed Category.

If you don't have any Aggregator Categories defined, you must do so at:

http://examle.com/?q=admin/aggregator/add/category

If this doesn't fix the problem, then we'll have to take a deeper look.

jwilde’s picture

jwilde commented

Hi,

My mistake. I rushed through the docs, which are some of the best I've seen for a module, and missed setting the categories.

Thank you for being so graceful and patient with your response. I appreciate it, a lot.

Kind regards,

Jim

agentrickard’s picture

agentrickard
he/him
Primary language English
Location Georgia (US)
commented
Status: Active » Postponed

No problem.

I've seen this 'bug' before and have left it in intentionally. If I had infinite time, I'd have the module send you a nicer warning.

As it stands, I think it's good that admins see the error, but better error handling is certainly needed.

agentrickard’s picture

agentrickard
he/him
Primary language English
Location Georgia (US)
commented
Status: Postponed » Fixed

I'm marking this as 'fixed,' with the intention of improving the error handling for admins.

Anonymous’s picture

(not verified) commented
Status: Fixed » Closed (fixed)
agentrickard’s picture

agentrickard
he/him
Primary language English
Location Georgia (US)
commented

The 5.x.2 release (currently in HEAD) adds a mysite_type_{name}_active() hook that allows includes to report configuration errors.