[Approval Queue] Denying an upload does not delete the version

RESOLVED FIXED in 1.0

Status

RESOLVED FIXED
15 years ago
3 years ago

People

(Reporter: Bugzilla-alanjstrBugs, Assigned: wolf)

Tracking

Details

(Whiteboard: beta)

(Reporter)

Description

15 years ago
The entry from t_version is not removed when you deny someone's request.
(Assignee)

Comment 1

15 years ago
No it isn't.. It's marked as approved=no. :-) Are you expecting it to delete the
record entirely?
Severity: normal → enhancement
(Reporter)

Comment 2

15 years ago
Actually, yes.  Otherwise when they upload another one, they wind up with rows
of FFMMTT
(Reporter)

Updated

14 years ago
Severity: enhancement → minor
(Assignee)

Comment 3

14 years ago
If the record is removed, where's the discoverable UI on the website showing
that it was denied? ;-)
Summary: Denying an upload does not delete the version → [Approval Queue] Denying an upload does not delete the version
(Reporter)

Comment 4

14 years ago
t_approvallog, of course.
(Assignee)

Comment 5

14 years ago
Comment #2 addressed a bug in additem actually thats been fixed. There's
currently no link from the overview page to t_approvallog and not much to go on
w/o the t_version record that matched up to it (as t_al intentionally records
the id/vid of the t_v record.)

Most cases for denial will be a situation that's corrected, and the item is
resubmitted, (upon resubmittal the duplicate old record would be removed anyway).

The only thing I can think of here that's needed is to make sure any files are
deleted so that only the DB record remains.
Status: NEW → ASSIGNED
(Assignee)

Comment 6

14 years ago
Woo, set URI to null and delete the item from the approval files directory.
Whiteboard: fixed-development
(Assignee)

Updated

14 years ago
Whiteboard: fixed-development → fixed-beta
(Assignee)

Comment 7

14 years ago
Bulk Moving Developer Control Panel bugs to new component.
(Filter: massdevcpspam)
Component: Update → Developers
Product: mozilla.org → Update
Version: other → unspecified
(Reporter)

Comment 8

14 years ago
Mass-resolving bugs that have been fixed on trunk.
Status: ASSIGNED → RESOLVED
Last Resolved: 14 years ago
Resolution: --- → FIXED
(Assignee)

Comment 9

14 years ago
Sorry for the bugspam, reopening bugs wrongly marked as resolved.
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
(Assignee)

Updated

14 years ago
Status: REOPENED → ASSIGNED
(Assignee)

Updated

14 years ago
Target Milestone: --- → 1.0
(Assignee)

Updated

14 years ago
Severity: minor → normal
Status: ASSIGNED → RESOLVED
Last Resolved: 14 years ago14 years ago
Resolution: --- → FIXED
Whiteboard: fixed-beta → beta
Version: unspecified → 0.9
Product: addons.mozilla.org → addons.mozilla.org Graveyard
You need to log in before you can comment on or make changes to this bug.