Hi,

I just activated the module for open graph meta tags and got the following error messages:

    warning: Invalid argument supplied for foreach() in /var/www/vhosts/mysite/httpdocs/includes/common.inc on line 927.
    warning: strtr(): The second argument is not an array in /var/www/vhosts/mysite/httpdocs/includes/common.inc on line 944.

Best,
Tobias

CommentFileSizeAuthor
#3 nodewords-n1395326.patch637 bytesdamienmckenna

Comments

damienmckenna’s picture

damienmckenna
Location TN, USA
commented
Status: Active » Postponed (maintainer needs more info)

What version of Drupal are you using? The latest 6.22? Do you get the error when you clear the site cache?

Also, was there a reason you selected "6.x-1.9" as the module version you're using? =)

jrockowitz’s picture

jrockowitz commented

The issue is the formatting of t() function is off in the nodewords_og_install() function.
http://drupalcode.org/project/nodewords.git/blob/refs/heads/6.x-1.x:/nod...

The t() function parentheses need to moved

This code...
drupal_set_message(t("Please see the README.txt file included with the Nodewords module for instructions on customizing the theme to make this work correctly.", 'warning'));

Should be...
drupal_set_message(t("Please see the README.txt file included with the Nodewords module for instructions on customizing the theme to make this work correctly."), 'warning');

damienmckenna’s picture

damienmckenna
Location TN, USA
commented
Version: 6.x-1.9 » 6.x-1.13
Status: Postponed (maintainer needs more info) » Needs review
StatusFileSize
new nodewords-n1395326.patch637 bytes

@jrockowitz: Ah, thank you for spotting that. #facepalm.

damienmckenna’s picture

damienmckenna
Location TN, USA
commented
Status: Needs review » Fixed

I've committed the fix. Thanks for spotting the problem, jrockowitz!

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.