A quick patch is attached which
- enables the module to work under D7 (it's not a proper upgrade, but it works (for me))
- adds titles* to the sitemaps
- cleans up a couple of odd bits
* Not required but strongly recommended: http://www.google.com/support/news_pub/bin/answer.py?hl=en&answer=74288
This is intended as a stopgap until anything happens at #787578: Figure out how to handle the global 'type' context and then #451234: Support for extended sitemap formats (e.g. video, news, image, mobile). However, it is marked 'needs review' for a reason: use at your own risk.
| Comment | File | Size | Author |
|---|---|---|---|
| #1 | googlenews-1040798.patch | 5.42 KB | willmoy |
| googlenews-d7-crude-upgrade.patch | 3.5 KB | willmoy |
Comments
Comment #1
willmoy commentedThis version tested and confirmed working, now with minimal updates to README.txt.
Comment #2
halo123 commentedHi Willmoy,
I tried the patch from #1 but I can't get it working.
I used 6.x-1.x-dev and 6.x-1.4 to patch, but both releases are not working after patching. I used everal programs to patch this module to Drupal 7, but all are giving errors. I've been working on this issue for hours now, but I can't seem to get it to work in any way.
Can you please upload the already patched version? This module is really important for my website.
Thanks in advance.
Comment #3
dave reidPatch was not against 6.x-1.4 or 6.x-1.x-dev. Please make sure to always use the latest code when patching otherwise it is not useful at all to other users. The module is now ported to 7.x properly and a 7.x-1.4 release should be available soon to hold people over until this functionality is merged into xmlsitemap.module.
Comment #5
Cybergrass commentedFYI, it doesn't work with DP 7.8.
No XML is generated. No errors. No warnings. No admin content types settings.
It is in admin>modules however.
Comment #6
Cybergrass commentedremoving the 'ad' advertising module resulted in googlenews working.
Problem resolved