RDFa 1.1 was published last month and requires namespaces to be serialized in a prefix attribute. The profile attribute is also obsolete. patch forthcoming.

This effectively backports the RDFa changes which happened in Drupal 8 at #1077566: Convert html.tpl.php to HTML5.

CommentFileSizeAuthor
#5 1674246-html-rdf-output-5-1.patch1.27 KBacbramley
#1 1674246_1_zen_rdfa11.patch1.23 KBscor

Comments

scor’s picture

scor commented
Status: Active » Needs review
StatusFileSize
new 1674246_1_zen_rdfa11.patch1.23 KB

This patch changes the way the namespace prefixes are serialized in HTML. xmlns is not longer used, the new prefix attribute is used instead. The profile attribute also deprecated and removed from html.tpl.php.

johnalbin’s picture

johnalbin
they/他
Primary language English
Location Taipei, Taiwan
commented
Status: Needs review » Fixed

Thanks for the patch, Scor! But I'm not a big fan of rerunning the module_invoke_all('rdf_namespaces'); from within rdf_get_namespaces(); template_preprocess_html() already runs that once.

It seems to me it would be faster to run some native-PHP string manipulations on $rdf_namespaces then it would be to call several functions for the hook invoke.

I've committed this change: http://drupalcode.org/project/zen.git/commitdiff/34afaf5

Let me know if you think this is the wrong approach, scor. I've given you a commit credit since I just modified your patch. :-)

scor’s picture

scor commented

Thanks John! That works and you're refactoring is indeed slightly faster.

Status: Fixed » Closed (fixed)

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

acbramley’s picture

acbramley commented
StatusFileSize
new 1674246-html-rdf-output-5-1.patch1.27 KB

Rolled a patch against 5.1 based on the commit diff

acbramley’s picture

acbramley commented
Issue summary: View changes

add note about D8 backport