Problem/Motivation

Views (well, technically Node, but really it's Views) has a sloppy integration with this module: https://git.drupalcode.org/project/drupal/-/blob/10.0.x/core/modules/nod...

That's a blocker to Drupal core deprecating RDF.

Proposed resolution

Let's move that integration into this module by implementing hook_node_view_alter() and setting $node->rss_namespaces correctly.

Remaining tasks

Do that and add a (kernel?) test to prove it works.

Issue fork rdf-3307138

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

phenaproxima created an issue. See original summary.

phenaproxima opened merge request !5

phenaproxima’s picture

phenaproxima
he/him
Primary language English
Location Massachusetts
CreditAttribution: phenaproxima at Acquia commented
Status: Active » Needs review

Boom -- here's the replacement functionality with a test to prove it works as expected.

bbrala’s picture

bbrala
Location Netherlands
CreditAttribution: bbrala at SWIS commented
Status: Needs review » Reviewed & tested by the community

Tested on a fresh install, other functionalities keep working. Tested making an RSS view and see rdf present. Tests green locally (stupid testbot). So RTBC for me :)

smustgrave’s picture

smustgrave CreditAttribution: smustgrave at Mobomo commented

+1 for RTBC

bbrala’s picture

bbrala
Location Netherlands
CreditAttribution: bbrala at SWIS commented

Add credits

bbrala’s picture

bbrala
Location Netherlands
CreditAttribution: bbrala as a volunteer and at SWIS commented

bbrala’s picture

bbrala
Location Netherlands
CreditAttribution: bbrala as a volunteer and at SWIS commented
Status: Reviewed & tested by the community » Fixed

Thanks for the merge, you forgot to set to fixed. :)

Status: Fixed » Closed (fixed)

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