Changes to the drupal_goto() function in 4.7 HEAD require that query elements in a URL be passed in as a separate parameter. If the URL is concantenated and passed in as a single parameter, the query elements are URLencoded and fail to parse properly.

A complete solution probably requires reworking the various link construction functions, but this gets the module working under HEAD.

CommentFileSizeAuthor
freelinking.module.patch1.71 KBeaton

Comments

eaton’s picture

eaton commented

Er, scratch that. How strange. It was working on my site for a moment but it seems to have gone back to the old behavior.

Instead of:

http://www.havana-mod.com/node/add/page?edit[title]=Foo

I'm getting:

http://www.havana-mod.com/node/add/page%3Fedit%5Btitle%5D%3DFoo

It certainly does seem to be related to the drupal_goto changes, but I'm still investigating the 'proper' fix.

eaton’s picture

eaton commented

After some investigation, it appears that the problem lies in the the creation of the 'freelinks' page. The URLs that it caches are not clean, so all links on the page generate the problem. With this patch in place, however, the actual links in the node bodies resolve correctly and prepopulate the node/add form.

eafarris’s picture

eafarris commented
Assigned: Unassigned » eafarris
Status: Active » Fixed

Committed an alternative fix to HEAD. Thanks, Jeff, for the report.

Please verify and close.

Anonymous’s picture

(not verified) commented
Status: Fixed » Closed (fixed)