Not all links login will change to /bakery, they need to be redirected.

CommentFileSizeAuthor
#3 bakery.diff1.05 KBdrumm
#2 bakery.diff617 bytesdrumm
bakery.diff433 bytesdrumm

Comments

drumm’s picture

That doesn't work if there is a ?destination=.. since drupal_goto() uses the destination if present. This strategy may be okay. However, the destination does get encoded one more time than it is decoded.

I want this for comment links, which are kinda hard to alter, #532682: Comment links arenot passed to hook_alter()

drumm’s picture

StatusFileSize
new617 bytes

Option 2 attached

drumm’s picture

StatusFileSize
new1.05 KB

Option 3, path aliases actually work kinda well.

killes@www.drop.org’s picture

Status: Needs review » Fixed

everybody needs to resave the settings form on slave sites now.

Status: Fixed » Closed (fixed)

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

greggles’s picture

Status: Closed (fixed) » Active

We could use some code comments on this to reference why we create them.

Also, why is this in the form rather than a hook_update?

juliangb’s picture

I'd just like to link from here to #787090: Use hook_menu_alter rather than aliases, which is an issue to move from using path aliases to using hook_menu_alter to handle the user/login and user/register paths.

Are there any comments from the original authors as to why this might / might not work / the original reasoning?

greggles’s picture

Status: Active » Closed (fixed)

I'm moving this back to fixed, but think we should implement the changes in #787090: Use hook_menu_alter rather than aliases which basically revert this in favor of using hook_menu_alter.