90933 - different handling of disabledtext

Status: RESOLVED FIXED

(Bugzilla :: Bugzilla-General, defect)

Description

[Lukas Gruetzmacher]

creating a user by using editusers.cgi (administrative) the field disabled text 
will be set to '' (because the edit field is empty)
Using the login screen (results in globals.pl:InsertNewUser) the field will not 
be set and keeps NULL

This results in hang of edit user created in the second way - the details 
screen (editusers.cgi?action=edit&user=...) will never be opened

You can fix this by setting disabled text to '' in globals.pl

Comment 1

[Jacob Steenhagen]

Hmm... I haven't seen this particular issue with editusers not loading, but
there does seem to be some inconsitancy w/the way disabledtext is handled, even
just within checksetup.pl.  At line 1012 where it is defined in the table
declaration, it's simply "disabledtext mediumtext", but at line 1828 where the
AddField is called to add it to a pre-existing database, it's defined as
"disabledtext mediumtext not null".  I think the proper declaration should be
"not null" which would make an it "" when empty rather than null.

Patch to checksetup.pl forthcoming.  If this is causing the problems described
here, it should be included in 2.14.

Comment 2

[Jacob Steenhagen]

Attachment: Make disabledtext be not null

Comment 3

[Dave Miller [:justdave]]

r= justdave

checked in

Comment 4

[Dave Miller [:justdave]]

Moving to Bugzilla product