913208 - Allow deleting of tracking flags that do not have bug values set but have activity entries in bugs_activity

Status: RESOLVED FIXED

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

Description

[David Lawrence [:dkl]]

Currently you cannot delete a tracking flag if the flag has either 1) currently set on one or more bugs and/or 2) has activity entries in the bugs_activity table from being previously set on bugs.

We should allow deletion as long as 1) is not true. We will just delete the bugs_activity entries at time of field removal.

dkl

Comment 1

[David Lawrence [:dkl]]

Attachment: 913208_1.patch

I am pretty sure this patch is OK and will work fine but due to the possibility of unintended data loss, I just want you to take a quick look at it before commit :)

Thanks
dkl

Comment 2

[:glob ✱]

Comment on attachment 800409 [details] [diff] [review]
913208_1.patch

r=glob

this looks fine.

rather than having the warning always visible (meaning we'll probably block it out), how about instead checking if the field has activity, and changing the text of the delete button:  "Delete Flag" / "Delete Flag and Activity".

Comment 3

[David Lawrence [:dkl]]

Committing to: bzr+ssh://[e-mail]/bmo/4.2
modified extensions/TrackingFlags/lib/Admin.pm
modified extensions/TrackingFlags/lib/Flag.pm                
modified extensions/TrackingFlags/template/en/default/hook/global/user-error-errors.html.tmpl
modified extensions/TrackingFlags/template/en/default/pages/tracking_flags_admin_edit.html.tmpl
Committed revision 8986.