Closed (fixed)
Project:
Site map
Version:
7.x-1.x-dev
Component:
User interface
Priority:
Normal
Category:
Feature request
Assigned:
Issue tags:
Reporter:
Created:
22 Nov 2011 at 16:22 UTC
Updated:
28 Jul 2017 at 18:14 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
mrfelton commentedI wonder if this could be implemented as a Bean plugin. http://drupal.org/project/bean_twitter_pull is a good example of using Bean to allow users to create and configure their own blocks that integrate with a module and its settings.
Comment #2
darrell_ulm commentedInteresting for a feature request. This may just work with a module like,
http://drupal.org/project/nodesinblock
Have not tested, as /sitemap link may not be treated as a normal 'node.'
The block option could be created for site_map, with some more code.
Comment #3
headdragon commentedIf I could link it into a panels existing node under add content
Comment #4
darrell_ulm commented@headdragon Yes, worth a look. Support for that would likely be quick. Seems like this should be tested, and assuming if not there, just another hook to get it to show up.
Thank you,
Comment #5
headdragon commentedHave the actual sitemap not the more show up in the block would be great.
Comment #6
darrell_ulm commentedRight, that was what I was thinking also, although may need a way to selectively limit the data shown.
Comment #7
darrell_ulm commentedChanged to active until there is a patch.
Comment #8
carolpettirossi commentedI've created the block using hook_block_info. So, the block is available on admin/structure/block named as Site map. The block content shows the same content as site_map_page.
Hope that works for everyone.
Comment #9
carolpettirossi commentedComment #10
carolpettirossi commentedComment #11
rafaolf commented@carolpettirossi, I found two issues, please check them below:
- the block has no title;
- if the block is added to any region, the node title will be overriden with sitemap's one (considering that there aren't modules which higher weight than this one).
Comment #12
carolpettirossi commented@rafaolf thanks for the review.
I fixed the issues reported.
Please, could you test the new patch?
Comment #13
carolpettirossi commentedComment #16
rodrigoeg commentedThe last patch was missing a "return" in the function of the page content, resulting in a blank page for /sitemap (it was breaking the tests).
Comment #17
renatog commentedHi people.
Good catch! I applied the patch and works good for me.
P.s. Rafa's Points in #11 was fixed.
Good Job!
RTBC
Comment #19
renatog commentedFixed.
Commited in dev branch. Soon we will have a new release.
Thank you all for contributions.
Good Work and Good Weekend.
Regards.