As discussed and tested here and by myself the following HTML output
<meta property="fb:admins" content="foo,bar" />
should be:

<meta property="fb:admins" content="foo" />
<meta property="fb:admins" content="bar" />

so if I fill the formfield with "foo,bar,person3" the module should print out 3 different HTML-Tags, not only one whith the list. It's only a warning by the debugger, but I think the module should be OG-Spec compliant ;)

Facebook-Tool to check an OG page
Example to the OG-Spec "Array"

Sorry, I'm not speaking english as fluent as I like to do.

CommentFileSizeAuthor
#4 metatag-n1915926-4.patch596 bytesdamienmckenna

Comments

kn8’s picture

Priority: Normal » Minor

I confirm this minor issue. Although facebook will recognize and parse the wrong format, it will throw a warning message in linter.

Solution would be to explode the string in fb:admins setting by "," and then print out all

<meta property="fb:admins" content="foo" />

as separate lines.

damienmckenna’s picture

Version: 7.x-1.0-beta4 » 7.x-1.x-dev
Category: task » feature
Status: Active » Postponed

This is dependent upon #1282636: Support meta tags that allow multiple values (repeated tags), a needed enhancement to the API.

damienmckenna’s picture

Title: Output of fb:admins isn't proper to OpenGraph Array Standard » Allow multiple fb:admins tags
Issue summary: View changes
Status: Postponed » Active

The other issue is unblocked, so now this can be done.

damienmckenna’s picture

Status: Active » Needs review
StatusFileSize
new596 bytes

Just took a single line to add this.

damienmckenna’s picture

Title: Allow multiple fb:admins tags » Allow multiple fb:admins values
damienmckenna’s picture

Status: Needs review » Fixed

Committed.

  • DamienMcKenna committed fb5fb54 on 7.x-1.x
    Issue #1915926 by DamienMcKenna: Allow multiple fb:admins values.
    

Status: Fixed » Closed (fixed)

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