Trying to add the requisite search engine verification tags is extraordinarily complicated for something that should be incredibly simple and this is because Nodewords is really designed for tags that should appear on multiple pages, not just on the homepage. I really think we should just remove this functionality and promote a separate module.
| Comment | File | Size | Author |
|---|---|---|---|
| #14 | nodewords-n1027086-14.patch | 11.36 KB | damienmckenna |
| #13 | nodewords-n1028086-13.patch | 3.97 KB | damienmckenna |
| #11 | nodewords-n1028086-11.patch | 15.41 KB | damienmckenna |
| #9 | nodewords-n1028086-2.patch | 15.41 KB | damienmckenna |
| #6 | nodewords-n1028086.patch | 15.24 KB | damienmckenna |
Comments
Comment #1
dave reidMaybe Site verifications should implement the Nodewords API?
Comment #2
damienmckennaSounds good.
So what do we need? I was thinking:
Whaddya think?
Comment #3
damienmckennaGiven that Site_Verify has conversion scripts for other modules already, I'm going to write a patch for it to import the Nodewords settings.
Comment #4
damienmckennaSite_Verify issue for the conversion script: #1028246: Import script for Nodewords settings
Comment #5
damienmckennaI've written a patch that converts Nodewords settings, both as an update script and during installation.
Comment #6
damienmckennaHere's a patch that removes the nodewords_verification_tags module and adds a small update script to both remove the now unnecessary variables but also notify users of the change.
Comment #7
damienmckennaComment #8
damienmckennaI think this should be in the next release.
Comment #9
damienmckennaSmall update to the patch that fixes variable_get() calls and removes the record from the {system} table.
Comment #11
damienmckennaSmall SQL bug in the last patch.
Comment #13
damienmckennaRerolled the patch.
Comment #14
damienmckennaRerolled for 6.x-1.x, lets see if it passes this time.
Comment #15
damienmckennaCommitted.