Closed (duplicate)
Project:
XML sitemap
Version:
7.x-2.x-dev
Component:
Other
Priority:
Normal
Category:
Task
Assigned:
Unassigned
Reporter:
Created:
12 May 2011 at 13:15 UTC
Updated:
13 Nov 2011 at 15:42 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
dave reidThis features has *already* been planned on being added to XML sitemap. As the maintainer I'd like to see work being put into the main module.
See:
#451234: Support for extended sitemap formats (e.g. video, news, image, mobile)
#1154062: Submit as a patch to the main XMLSitemap module
Comment #2
dave reidComment #3
tim.plunkettClosing, feel free to re-open if this was a mistake.
Comment #4
leanazulyoro commentedhello,
I've taken the liberty to get the code from the sandbox and change several things, while also port it to drupal 6, I also changed a little the patch for xmlsitemap.
The result is good, it basically searches for imagefields in the node and adds tags for each image in them.
I also added a "hook_xmlsitemap_images_file_links" , to allow other modules to modify the list of images before the markup gets built. I did this because in my case I have a content type that has a nodereference field that referenes nodes that contain an imagefield. So I implemented that hook from another module and added the images from those referenced nodes to the node that referenced them.
To activate the functionallity I added a checkbox "Add image references to each node" in the XMLsitemap fieldset in content types' edit forms, so you can activate it per content type.
I'm attaching the modified module, it also contains the modified xmlsitemap.patch.
Comment #5
leanazulyoro commentedComment #6
elc commentedHow about we move this over to the xmlsitemap module issue queue since that's where things seem to be heading. It requires a patch to xml sitemap, and it's a direct addition that is already planned.
Also, @leanazulyoro please only submit patches. Full code dumps like that are not very useful as someone has to carefully go through and pick out the changes manually rather than actually using the code.
Please move this back to App Queue if needed.
Comment #7
Anonymous (not verified) commentedSubmit your patches to #451234: Support for extended sitemap formats (e.g. video, news, image, mobile)