Patch coming.

CommentFileSizeAuthor
#1 886878.logintoboggan.registration-403.patch1.33 KBjoachim

Comments

joachim’s picture

joachim commented
Status: Active » Needs review
StatusFileSize
new 886878.logintoboggan.registration-403.patch1.33 KB

I wanted to change the 'Present login form on access denied (403):' radio into a set with 3 options, but the way that it's an actual 'toboggan/denied' path that is stored makes it somewhat complicated. I did figure this could be done by storing either toboggan/denied/login or toboggan/denied/register, but then all the checks for toboggan/denied would have to be changed to substr() and it would get quite messy.

Here's a patch on my SVN which just adds an extra admin setting.

hunmonk’s picture

hunmonk commented
Status: Needs review » Closed (won't fix)

not a fan of this approach, i'd rather see the 403 respect this feature: #264332: Create unified login/register page, which i hope to get into 7.x

not accepting new features for 6.x anyways.