Closed Bug 286008 Opened 19 years ago Closed 19 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: 19 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: