no longer works with Drupal 6.12

CommentFileSizeAuthor
#4 search404_request.patch494 bytesjcmarco

Comments

Anonymous’s picture

Anonymous (not verified) commented

Same here. Upgraded today.

We get Page Not Found
and a Search box

superdiva’s picture

superdiva commented

I'm in the same boat, but I was in process of updating modules.

kendouglass’s picture

kendouglass commented

I agree - no longer works with Drupal 6.12.

jcmarco’s picture

jcmarco commented
Status: Active » Needs review
StatusFileSize
new search404_request.patch494 bytes

Umm! I have just discovered it right now!
It seems that #292565: Regression: Can't set /user to 403 page (was: login destination is broken) in Drupal 6.12 "sanitized" the $_REQUEST['redirect'] in common.inc
Here is the patch, I have tested it and it is working with the search404 solr patch as well, but there shouldn't be any change with the standard version.
Please test it.
But it would be nice have a new dev version with this patch and the solr_patch as well together, ;-)

Anonymous’s picture

Anonymous (not verified) commented

Patch sems to work,

Thanks

cwatt’s picture

cwatt commented

Confirms as working for me using 6.12

netbjarne’s picture

netbjarne commented

confirmed fixed functionality with drupal 6.12

dbeall’s picture

dbeall commented

thought i missed a setting, just started using this.
confirm, patch works very nice, thanks

zyxware’s picture

zyxware commented

Hello All

We have fixed the issue in the latest commit, do check it out and let us know if things are ok.

Regards
Zyxware

zyxware’s picture

zyxware commented
Status: Needs review » Fixed

Marking this as fixed there there has been no activity.

Feel free to open the issue in case it is not resolved.

Regards
Santhosh Raju

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.