Closed
Bug 567281
Opened 16 years ago
Closed 16 years ago
Bugzilla::Field should use VALIDATOR_DEPENDENCIES instead of UPDATE_VALIDATORS
Categories
(Bugzilla :: Bugzilla-General, enhancement)
Tracking
()
RESOLVED
FIXED
Bugzilla 4.0
People
(Reporter: mkanat, Assigned: mkanat)
References
Details
Attachments
(1 file, 1 obsolete file)
|
6.54 KB,
patch
|
timello
:
review+
|
Details | Diff | Splinter Review |
Bugzilla::Field has quite a few UPDATE_VALIDATORS set, and should instead be using VALIDATOR_DEPENDENCIES.
|
Assignee | |
Comment 1•16 years ago
|
||
|
|
Assignee | |
Comment 2•16 years ago
|
||
Oh, oops. Forgot to add in "name" to VALIDATORS.
Attachment #446634 -
Attachment is obsolete: true
Attachment #446639 -
Flags: review?(LpSolit)
Attachment #446634 -
Flags: review?(LpSolit)
|
|
Assignee | |
Updated•16 years ago
|
Attachment #446639 -
Flags: review?(LpSolit) → review?(timello)
|
||
Comment 3•16 years ago
|
||
Comment on attachment 446639 [details] [diff] [review]
v2
This looks fine and working properly.
Attachment #446639 -
Flags: review?(timello) → review+
|
|
||
Updated•16 years ago
|
Flags: approval?
|
|
Assignee | |
Updated•16 years ago
|
Flags: approval? → approval+
|
|
Assignee | |
Comment 4•16 years ago
|
||
Thanks for the review, timello. :-)
Committing to: bzr+ssh://bzr.mozilla.org/bugzilla/trunk/
modified Bugzilla/Field.pm
Committed revision 7223.
Status: ASSIGNED → RESOLVED
Closed: 16 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•