Closed Bug 917203 Opened 11 years ago Closed 11 years ago

tracking flags with flag_type of "status" is unclear

Categories

(bugzilla.mozilla.org :: Extensions, defect)

Production
x86
macOS
defect
Not set
normal

Tracking

()

RESOLVED FIXED

People

(Reporter: glob, Assigned: glob)

References

Details

Attachments

(1 file)

in TrackingFlags::Constants there's a flag_type of "status".  the migration code in bug 880829 doesn't migrate any existing fields into this flag_type.

it looks like it's only used in one place - the release tracking report only shows fields with this type.

however if a field's flag_type is changed from 'tracking' to 'status', it is displayed on show_bug in its own group, which isn't correct.


we should remove the 'status' flag_type, and change the report's code to do a regex on the flag's name.
Attached patch 917203_1.patchSplinter Review
Attachment #805872 - Flags: review?(dkl)
Comment on attachment 805872 [details] [diff] [review]
917203_1.patch

Review of attachment 805872 [details] [diff] [review]:
-----------------------------------------------------------------

r=dkl
Attachment #805872 - Flags: review?(dkl) → review+
Committing to: bzr+ssh://bjones%40mozilla.com@bzr.mozilla.org/bmo/4.2/
modified extensions/BMO/lib/Util.pm
modified extensions/TrackingFlags/lib/Constants.pm
Committed revision 9017.
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Component: Extensions: TrackingFlags → Extensions
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: