update requested

jjancel - May 18, 2009 - 22:40
Project:Search 404
Version:6.x-1.4
Component:Code
Category:bug report
Priority:critical
Assigned:jjancel
Status:needs review
Description

no longer works with Drupal 6.12

#1

midkemia - May 19, 2009 - 06:42

Same here. Upgraded today.

We get Page Not Found
and a Search box

#2

superdiva - May 19, 2009 - 08:57

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

#3

K.D. - May 21, 2009 - 00:55

I agree - no longer works with Drupal 6.12.

#4

jcmarco - May 24, 2009 - 14:32
Status:active» needs review

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, ;-)

AttachmentSize
search404_request.patch 494 bytes

#5

midkemia - May 24, 2009 - 18:18

Patch sems to work,

Thanks

#6

cwatt - May 26, 2009 - 11:50

Confirms as working for me using 6.12

#7

netbjarne - June 2, 2009 - 13:01

confirmed fixed functionality with drupal 6.12

#8

dbeall - June 14, 2009 - 16:33

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

#9

zyxware - June 15, 2009 - 08:27

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

 
 

Drupal is a registered trademark of Dries Buytaert.