Closed
Bug 286008
Opened 19 years ago
Closed 19 years ago
Unable to create new users (createaccount.cgi)
Categories
(Bugzilla :: Email Notifications, defect)
Tracking
()
RESOLVED
WORKSFORME
People
(Reporter: gerv, Assigned: gerv)
Details
Attachments
(1 file)
785 bytes,
patch
|
Details | Diff | Splinter Review |
The checkin for bug 277623 broke createaccount.cgi: "Software error: Undefined subroutine &main::PerformSubsts called at CGI.pl line 199." Patch coming up. Gerv
Assignee | ||
Comment 1•19 years ago
|
||
This fixes the reported problem, although someone should probably check all other non-BugMail.pm calls to PerformSubsts() to make sure they work too. Gerv
Assignee: email-notifications → gerv
Status: NEW → ASSIGNED
Assignee | ||
Updated•19 years ago
|
Attachment #177322 -
Flags: review?(mkanat)
Assignee | ||
Updated•19 years ago
|
Severity: normal → blocker
Target Milestone: --- → Bugzilla 2.20
Comment 2•19 years ago
|
||
As MailPassword() is only called from createaccount.cgi, a better fix would be to move this function in BugMail.pm in the process of removing subs from CGI.pl. I'm sure mkanat would appreciate! ;)
Assignee | ||
Comment 3•19 years ago
|
||
We should do the minimum fix to unbreak the tree, and handle any enhancements separately, IMO. Gerv
Assignee | ||
Comment 4•19 years ago
|
||
This seems to have been fixed by some other change. Gerv
Status: ASSIGNED → RESOLVED
Closed: 19 years ago
Resolution: --- → WORKSFORME
Comment 5•19 years ago
|
||
Comment on attachment 177322 [details] [diff] [review] Patch v.1 PerformSubsts() has always been in the EXPORT section since mkanat moved it into BugMail.pm, so adding Bugzilla::BugMail does not help. By the way, I cannot reproduce this problem.
Attachment #177322 -
Flags: review?(mkanat)
Comment 6•19 years ago
|
||
clearing target of DUPLICATE/WONTFIX/INVALID/WORKSFORME so they'll show up as untriaged if they get reopened.
Target Milestone: Bugzilla 2.20 → ---
You need to log in
before you can comment on or make changes to this bug.
Description
•