The 2026 Drupal Business Survey is open. Your response will help shape the future of the Drupal business ecosystem.As the title says, the Cancel link on the Delete entry confirmation page incorrectly points to http://guestbook/
The 2026 Drupal Business Survey is open. Your response will help shape the future of the Drupal business ecosystem.As the title says, the Cancel link on the Delete entry confirmation page incorrectly points to http://guestbook/
Comments
Comment #1
tomws commentedThis is also a problem for the other administrative links on the guestbook entries. It's a problem with the way the links are built manually in the code. Using the drupal_get_destination() function seems to fix this. Patch attached.
Also attached in a patch that can be applied a version that has my pre-publish moderation patch already applied.
Comment #2
tomws commentedBad title change last time. It's not the links, but the destinations that are flawed. For example, they come out as
http://example.com/guestbook/delete/1?destination=/guestbookor
http://example.com/?q=guestbook/delete/1&destination=/?q=guestbookWhen propagated to the confirmation handler, that translates to a final destination http://guestbook.
The patch still works, but my description was miserable. Apologies, and hope this helps.
Comment #3
recrit commentedI had this same issue and it's even more compounded when the installation is in a subdirectory. I have re-run the patch against the latest 6.x-2.x-dev 2010-04-03. I added the pager string back in since it was missing from the patches in #1.
Comment #4
sunThanks for reporting, reviewing, and testing! Committed to HEAD.
A new development snapshot will be available within the next 12 hours. This improvement will be available in the next official release.