Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.Just tried installing 7.x.2.0 and turning on the modal module. This doesn't seem to work at all. It implements a hook hook_xmlsitemap_operation_link_alter() which is not invoked anywhere. The hook hook_xmlsitemap_link_alter() is documented, and looks like it should be the right one to use, but I think there is more to the problem than that: changing the name of the function in the modal does not seem to be enough on its own.
| Comment | File | Size | Author |
|---|---|---|---|
| #1 | xmlsitemap--modal-2307573-1.patch | 412 bytes | loopduplicate |
Comments
Comment #1
loopduplicateThe latest commit in the dev branch is meant to solve this (well, according to the commit message, "Make xmlsitemap_modal kinda work."). I noticed that the new code doesn't use drupal_exit() but instead just uses exit(). This means that modules can't implement hook_exit().
Per the drupal_exit() docs: "There should rarely be a reason to call exit instead of drupal_exit();"
I'm attaching a patch for this.
Comment #2
Chris Matthews CreditAttribution: Chris Matthews commentedThe 4 year old patch to xmlsitemap_modal.module applied cleanly to the latest xmlsitemap 7.x-2.x-dev and if still applicable needs review.
Comment #3
renatog#1 really works good
Comment #5
renatog