Closed
Bug 285380
Opened 20 years ago
Closed 20 years ago
Bugzilla::DB::Schema uses ThrowCodeError incorrectly
Categories
(Bugzilla :: Bugzilla-General, defect, P1)
Tracking
()
RESOLVED
FIXED
Bugzilla 2.20
People
(Reporter: mkanat, Assigned: mkanat)
References
Details
(Keywords: regression)
Attachments
(1 file, 1 obsolete file)
(deleted),
patch
|
bugreport
:
review+
|
Details | Diff | Splinter Review |
This is causing Tinderbox to burn, but the patch was so huge I don't want to
back it out right now.
It's a very simple fix, which I will attach.
Assignee | ||
Comment 1•20 years ago
|
||
This really should be "minor," but it's making Tinderbox burn.
Assignee: general → mkanat
Priority: -- → P1
Target Milestone: --- → Bugzilla 2.20
Assignee | ||
Comment 2•20 years ago
|
||
I just changed it to a die -- this code is called mostly at the command-line
anyway.
Attachment #176815 -
Flags: review?(bugzilla)
Comment on attachment 176815 [details] [diff] [review]
Change it to a "die"
r=glob
Attachment #176815 -
Flags: review?(bugzilla) → review+
Assignee | ||
Updated•20 years ago
|
Status: NEW → ASSIGNED
Flags: approval?
Updated•20 years ago
|
Flags: approval? → approval+
Assignee | ||
Comment 4•20 years ago
|
||
Checking in Bugzilla/DB/Schema.pm;
/cvsroot/mozilla/webtools/bugzilla/Bugzilla/DB/Schema.pm,v <-- Schema.pm
new revision: 1.2; previous revision: 1.1
done
Status: ASSIGNED → RESOLVED
Closed: 20 years ago
Resolution: --- → FIXED
Assignee | ||
Comment 5•20 years ago
|
||
There are more instances of ThrowCodeError. :-(
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Assignee | ||
Comment 6•20 years ago
|
||
Attachment #176815 -
Attachment is obsolete: true
Attachment #176821 -
Flags: review?(justdave)
Updated•20 years ago
|
Attachment #176821 -
Flags: review?(justdave) → review+
Assignee | ||
Comment 7•20 years ago
|
||
Checking in Bugzilla/DB/Schema.pm;
/cvsroot/mozilla/webtools/bugzilla/Bugzilla/DB/Schema.pm,v <-- Schema.pm
new revision: 1.3; previous revision: 1.2
done
Status: REOPENED → RESOLVED
Closed: 20 years ago → 20 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•