Domain Meta Data is implementing a hook that versions after 6.x-1.0 doesn't use anymore.
Probably a similar hook will be used again, but it will be called hook_nodewords_tags_output_alter().

CommentFileSizeAuthor
#8 domain_meta-n635982-8.patch1.25 KBdamienmckenna

Comments

avpaderno’s picture

avpaderno
he/him
Primary language Italian
Location Brescia, 🇮🇹 🇪🇺
commented

It would also be better if the code would check the object associated with the meta tags.
Nodewords handles the meta tags for many pages, but Domain Meta Data offers only one set of settings.

scarr’s picture

scarr commented

I'm also interested in this issue - any way I can help, or chance it can get updated soon?

nonsie’s picture

nonsie
she/her
Primary language Estonian
commented
Status: Active » Postponed

I don't currently have a need for this feature on any of my sites nor time to work on it. Patches are more than welcome though.

damienmckenna’s picture

damienmckenna
Location TN, USA
commented

Marked #693042: Support for Nodewords v6.x-1.13 as a duplicate.

damienmckenna’s picture

damienmckenna
Location TN, USA
commented
Title: Module doesn't work with Nodewords > 6.x-1.0 » Update to work with Nodewords v6.x-1.13
Status: Postponed » Active

Lets refocus this issue on ensuring that it works with the latest version, maybe even make it required due to how broken earlier versions were.

damienmckenna’s picture

damienmckenna
Location TN, USA
commented
Assigned: Unassigned » damienmckenna

I'll work on this later this week.

damienmckenna’s picture

damienmckenna
Location TN, USA
commented
Status: Active » Fixed

I've tested it out and it appears to work just fine.

damienmckenna’s picture

damienmckenna
Location TN, USA
commented
Status: Fixed » Needs review
StatusFileSize
new domain_meta-n635982-8.patch1.25 KB

All the same, here's a patch that adds a hook_requirements check to ensure Nodewords v6.x-1.13 (or newer) is installed.

damienmckenna’s picture

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

Committed. I also added a note to the README.txt to say that v6.x-1.13 was required.

Status: Fixed » Closed (fixed)

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

damienmckenna’s picture

damienmckenna
Location TN, USA
commented
Assigned: damienmckenna » Unassigned
Issue summary: View changes