Add a note to the README.txt to mention that there can be OG validation problems when the core RDF module is enabled.

CommentFileSizeAuthor
#12 metatag-n1978708-12.patch2.58 KBDamienMcKenna
#9 1978708_9.patch2.87 KBscor
#3 metatag-n1978708-3.patch2.16 KBDamienMcKenna
#2 metatag-n1978708-2.patch0 bytesDamienMcKenna
#1 metatag-n1978708-1.patch1.35 KBDamienMcKenna
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

DamienMcKenna’s picture

DamienMcKenna
Location NH, USA
CreditAttribution: DamienMcKenna commented
Status: Active » Needs review
FileSize
metatag-n1978708-1.patch1.35 KB

This also adds a new metatag_opengraph_install that mentions the possible conflict if RDF is enabled.

DamienMcKenna’s picture

DamienMcKenna
Location NH, USA
CreditAttribution: DamienMcKenna commented
FileSize
metatag-n1978708-2.patch0 bytes

An updated patch that standardizes the language and adds a hook_requirements note.

DamienMcKenna’s picture

DamienMcKenna
Location NH, USA
CreditAttribution: DamienMcKenna commented
FileSize
metatag-n1978708-3.patch2.16 KB

Doh.

DamienMcKenna’s picture

DamienMcKenna
Location NH, USA
CreditAttribution: DamienMcKenna commented
Status: Needs review » Fixed

Committed.

Status: Fixed » Closed (fixed)

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

scor’s picture

scor CreditAttribution: scor commented
Title: Add a README.txt note about OG validation problems from core RDF module » Revert: Add a README.txt note about OG validation problems from core RDF module
Issue summary: View changes
Status: Closed (fixed) » Postponed

This patch should be reverted once these issues have been committed to Drupal 7:
- #1323830: Place title RDFa metadata inside entity HTML element
- #2130673: Place number of comments metadata inside node template

DamienMcKenna’s picture

DamienMcKenna
Location NH, USA
CreditAttribution: DamienMcKenna commented
Related issues: +#1323830: Place title RDFa metadata inside entity HTML element, +#2130673: Place number of comments metadata inside node template
DamienMcKenna’s picture

DamienMcKenna
Location NH, USA
CreditAttribution: DamienMcKenna commented
Status: Postponed » Active

It seems like 7.33 will include the core fixes, so I'm reopening this.

scor’s picture

scor CreditAttribution: scor commented
Status: Active » Needs review
FileSize
1978708_9.patch2.87 KB

Indeed, it's ready to be shipped in 7.33!

DamienMcKenna’s picture

DamienMcKenna
Location NH, USA
CreditAttribution: DamienMcKenna commented
Status: Needs review » Needs work

How about updating hook_requirements to note the fix in 7.33?

DamienMcKenna’s picture

DamienMcKenna
Location NH, USA
CreditAttribution: DamienMcKenna commented
Parent issue: » #2376921: META: Plan for Metatag 7.x-1.5 release
DamienMcKenna’s picture

DamienMcKenna
Location NH, USA
CreditAttribution: DamienMcKenna commented
FileSize
metatag-n1978708-12.patch2.58 KB

Updated patch that only shows a warning on older versions of core.

DamienMcKenna’s picture

DamienMcKenna
Location NH, USA
CreditAttribution: DamienMcKenna commented
Status: Needs work » Needs review

  • DamienMcKenna committed 4eefca5 on 7.x-1.x 
    Issue #1978708 by DamienMcKenna, scor: Updated warning about...
DamienMcKenna’s picture

DamienMcKenna
Location NH, USA
CreditAttribution: DamienMcKenna commented
Status: Needs review » Fixed

Committed! Thanks for the help, scor!

Status: Fixed » Closed (fixed)

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