This path was the current user page and should be so.

CommentFileSizeAuthor
#13 user-fixup-172765-13.patch774 bytespwolanin
#3 user_page_2.patch1.17 KBwebchick
#2 user_page_1.patch823 byteschx
#1 user_page_0.patch884 byteschx
user_page.patch20.7 KBchx

Comments

chx’s picture

chx commented
StatusFileSize
new user_page_0.patch884 bytes

Someone get that aggregator issue out of my hair...

chx’s picture

chx commented
StatusFileSize
new user_page_1.patch823 bytes

small is beautiful.

webchick’s picture

webchick
she/they
Primary language English
Location Vancouver 🇨🇦
commented
StatusFileSize
new user_page_2.patch1.17 KB

Tested and works great. This is exactly the same patch, except with some PHPDoc for the function, since it's not immediately obvious why we have it. Please don't credit me for the patch.

chx’s picture

chx commented

webchick's is the right one. The smaller one breaks tabs.

webchick’s picture

webchick
she/they
Primary language English
Location Vancouver 🇨🇦
commented
Status: Needs review » Reviewed & tested by the community

Meant to mark this RTBC.

pwolanin’s picture

pwolanin commented
Status: Reviewed & tested by the community » Needs work

This should use menu_set_active_item() - but that needs this patch: http://drupal.org/node/148677

chx’s picture

chx commented
Status: Needs work » Reviewed & tested by the community

This is good to go. When that one is in, we can change this one line.

pwolanin’s picture

pwolanin commented

ok - agreed.

webchick’s picture

webchick
she/they
Primary language English
Location Vancouver 🇨🇦
commented
Title: path 'user' returns 403 » Beta-breaker: path 'user' returns 403

upgrading this to a beta-breaker... lots of themes have menus pointing to ?q=user as a convenient way of handling the "Login"/"My Account" link.

walkah’s picture

walkah commented

I'd just like to +1 this - as discussed in IRC with chx - this also fixes current behaviour with openid.module where you get 403'ed despite being successfully logged in (since you get returned to /user).

dries’s picture

dries commented
Status: Reviewed & tested by the community » Fixed

Committed to CVS HEAD. Thanks.

Anonymous’s picture

(not verified) commented
Status: Fixed » Closed (fixed)
pwolanin’s picture

pwolanin commented
Status: Closed (fixed) » Reviewed & tested by the community
StatusFileSize
new user-fixup-172765-13.patch774 bytes

Part of this patch was reverted by the user module split. Bah! /user returns 403 again.

see attached fix-up patch.

gábor hojtsy’s picture

gábor hojtsy
he/him
Primary language Hungarian
Location Hungary
commented
Status: Reviewed & tested by the community » Fixed

Thanks, committed.

Anonymous’s picture

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