If the "Edit node" link that is added by plugins/content_types/node/node.inc is used to eventually delete a node, the browser will end up at a 404 page at the end because it is being redirected back to the page it just deleted. This happens because CTools unconditionally sets the destination= query string for that link, which is inconsistent with the standard behaviors of the node module. The attached patch corrects this behavior by only setting the query parameter if $_REQUEST['destination'] is set, similar to how node_form_delete_submit() works in node.pages.inc.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

mgriego’s picture

Also discovered that the same thing was happening in plugins/content_types/node_context/node_content.inc. Another patch attached.

mgriego’s picture

I should actually mention that this is related to the admin_links.

merlinofchaos’s picture

Version: 6.x-1.3 » 7.x-1.x-dev
Status: Needs review » Patch (to be ported)

Makes sense to me. Committed. THanks!

merlinofchaos’s picture

Status: Patch (to be ported) » Closed (fixed)

All of the 'to be ported' patches got merged in during the porting sprint in September.