current HEAD doesn't redirect after a successful login.

attached is a patch from chx, that i've tested on a fresh cvs checkout, and we get the redirect back. thanks chx.

CommentFileSizeAuthor
login_redirect.patch.txt910 bytesAnonymous (not verified)

Comments

chx’s picture

chx commented
Title: no redirect on successful login » Restore default redirect on submit

The issue has nothing to do with login, it's just most apparent there.

dries’s picture

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

No one reviewed this?

Anonymous’s picture

Anonymous (not verified) commented

for what its worth, i did.

i spotted the bug => questions in #drupal => pastebin from chx => review and test from me => this issue.

RobRoy’s picture

RobRoy commented

+1 works great for forms that don't return a redirect to themselves like block_admin_display_submit().

This fixes my issue at for the aforementioned submit function. http://drupal.org/node/79441

drumm’s picture

drumm
he/him
Location NY, US
commented

I'm testing this over on http://hatfactory.net/ and it seems to be working for user logins.

webchick’s picture

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

Confirmed login fix, as well as the bug reported here: http://drupal.org/node/79809

RTBC.

hunmonk’s picture

hunmonk commented

applied the patch, and it also fixes a recent problem i've noticed at admin/build/block--where blocks are not being selected/sorted properly after form submission. +1

let's get this in!

dries’s picture

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

Committed to CVS HEAD.

Anonymous’s picture

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