bogus integration with alt_login [#287496]

You guys managed to commit something that doesn't work at all !

Issues:
- the logic for alt_login requires name to be defined! Since you just cut&pasted one of my lines, the result doesn't make sense.
The select never works.

- if you don't patch the auto-select part as well, the patch is of little use anyways, as you have to enter alt_login stuff manually...

Comment	File	Size	Author
#10	287496_alt_login_integration_10.patch	2.16 KB	deviantintegral
#10
#8	287496_alt_login_integration.patch	1.8 KB	deviantintegral
#8
#2	masquerade.patch	2.7 KB	espie
#2
#1	masq.diff	2.81 KB	espie
#1

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Comment #1

espie CreditAttribution: espie commented 26 July 2008 at 13:35

File	Size
masq.diff	2.81 KB

patch attached, that makes it work with 1.2...

Comment #2

espie CreditAttribution: espie commented 27 July 2009 at 12:59

Version:

5.x-1.2

» 6.x-1.0

File	Size
masquerade.patch	2.7 KB

Fixed patch for 6.x, since I switched.

With this one, both auto-complete and actual changes work.

I think it's simpler than what's currently in.

Comment #3

deekayen CreditAttribution: deekayen commented 30 July 2009 at 14:16

Status:

Active

» Postponed

#521444: Store user ID's instead of user names for quick switch links is a prereq to this

Comment #4

deviantintegral CreditAttribution: deviantintegral commented 31 August 2009 at 17:54

Version:	6.x-1.0	» 6.x-1.x-dev
Status:	Postponed	» Needs work

Patch no longer applies.

Comment #5

espie CreditAttribution: espie commented 1 September 2009 at 22:32

sigh, I'll work on another patch tomorrow.
But please stop postponing it or applying it in weird ways, it's a bit like having to fix the same bug over three times. It is getting annoying...