User-Agent: Mozilla/5.0 (X11; U; Linux i686; fr; rv:1.7.6) Gecko/20050406 Firefox/1.0.2 (Debian package 1.0.2-3) Build Identifier: Mozilla/5.0 (X11; U; Linux i686; fr; rv:1.7.6) Gecko/20050406 Firefox/1.0.2 (Debian package 1.0.2-3) A query crash the process while creating a new user if the edituser.cgi script. Reproducible: Always Steps to Reproduce: 1. Select "add a user" in the administration of bugzilla 2. Fill-in the form and leave "Disable text" field blank 3. Submit the form Actual Results: Bugzilla crashes with following message: DBD::Pg::st execute failed: ERROR: function locate("unknown", character varying) does not exist HINT: No function matches the given name and argument types. You may need to add explicit type casts. [for Statement "SELECT eventdata FROM tokens WHERE tokentype = 'emailold' AND SUBSTRING(eventdata, 1, (LOCATE(':', eventdata) - 1)) = ? OR SUBSTRING(eventdata, (LOCATE(':', eventdata) + 1)) = ?" with ParamValues: firstname.lastname@example.org', email@example.com'] at Bugzilla/User.pm line 1142 Bugzilla::User::is_available_username('firstname.lastname@example.org') called at /home/httpd/htdocs/bugzilla/editusers.cgi line 175
Hrm, I wonder if that's specific to 8.0, or if LOCATE is not ANSI.
OK, yeah, we should be using POSITION (or really, sql_position, I think) instead of LOCATE. That's a pretty easy fix.
Created attachment 182830 [details] [diff] [review] Fix User Account Creation OK, there was one other problem, which was that refreshed_when is NOT NULL and was missing from the INSERT.
(In reply to comment #3) > Created an attachment (id=182830)  > Fix User Account Creation > > OK, there was one other problem, which was that refreshed_when is NOT NULL and > was missing from the INSERT. I have tried your patch and it works just fine for me.
Comment on attachment 182830 [details] [diff] [review] Fix User Account Creation refreshed_when should not be now() unless you derived groups. Users should start out in need of refresh.
Created attachment 183786 [details] [diff] [review] v2 OK, I changed it to be January, 1901 by default. In another patch, we should probably give refreshed_when a default value. I think that would be a cleaner long-term solution. (Something for 2.22.)
Comment on attachment 183786 [details] [diff] [review] v2 r=joel if you've tested it It does seem like there should have been a simpler way to do this in the first place.
Yep, I just tested it, and it works. :-)
Checking in Bugzilla/User.pm; /cvsroot/mozilla/webtools/bugzilla/Bugzilla/User.pm,v <-- User.pm new revision: 1.55; previous revision: 1.54 done
*** Bug 296075 has been marked as a duplicate of this bug. ***
*** Bug 299880 has been marked as a duplicate of this bug. ***