Closed Bug 286008 Opened 20 years ago Closed 20 years ago

Unable to create new users (createaccount.cgi)

Categories

(Bugzilla :: Email Notifications, defect)

2.19.2
defect
Not set
blocker

Tracking

()

RESOLVED WORKSFORME

People

(Reporter: gerv, Assigned: gerv)

Details

Attachments

(1 file)

The checkin for bug 277623 broke createaccount.cgi: "Software error: Undefined subroutine &main::PerformSubsts called at CGI.pl line 199." Patch coming up. Gerv
Attached patch Patch v.1Splinter Review
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
Attachment #177322 - Flags: review?(mkanat)
Severity: normal → blocker
Target Milestone: --- → Bugzilla 2.20
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! ;)
We should do the minimum fix to unbreak the tree, and handle any enhancements separately, IMO. Gerv
This seems to have been fixed by some other change. Gerv
Status: ASSIGNED → RESOLVED
Closed: 20 years ago
Resolution: --- → WORKSFORME
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)
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.

Attachment

General

Created:
Updated:
Size: