Redirecting when called from Drush breaks Drush requests. Such redirects can occur when the site_frontpage variable is set to a space.

The patch that was applied to Global Redirect in #594326: Breaks Drush is a good starting place.

CommentFileSizeAuthor
#1 982708-1-spaces-drush-fix.patch1.01 KBnedjo
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

nedjo’s picture

nedjo
he/him/his
Primary language English
CreditAttribution: nedjo commented
Status: Active » Needs review
FileSize
982708-1-spaces-drush-fix.patch1.01 KB

Moved this bug from #936234: call to router during menu access check causes infinite loop.

Attached patch is based on #594326: Breaks Drush by smk-ka and Dave Reid, see http://drupal.org/cvs?commit=336592.

hefox’s picture

hefox CreditAttribution: hefox commented
Status: Needs review » Reviewed & tested by the community

Works as expected and looks good

nedjo’s picture

nedjo
he/him/his
Primary language English
CreditAttribution: nedjo commented
Version: 6.x-3.0 » 7.x-3.x-dev
Status: Reviewed & tested by the community » Needs work

Thanks for reviewing. I guess though this should be for D7 first, using drupal_is_cli().

hefox’s picture

hefox CreditAttribution: hefox commented

Bugs can go into 6.x and up-ported to 7.x to my knowledge.