Closed Bug 640212 Opened 13 years ago Closed 13 years ago

If an action fails after making a revision, the revision should be removed

Categories

(addons.mozilla.org Graveyard :: Add-on Builder, defect, P2)

defect

Tracking

(Not tracked)

VERIFIED FIXED
Builder 0.9.4

People

(Reporter: smcarthur, Assigned: smcarthur)

References

Details

Attachments

(1 file)

Since most actions create a new revision, and then save things to that revision, when there is an error, we should rollback to before that revision was saved. Example can be found at https://bugzilla.mozilla.org/show_bug.cgi?id=639925
Assignee: nobody → smcarthur
Priority: -- → P2
Target Milestone: --- → Builder 0.9.1
It would seem that the easy way of doing this, using transactions, requires our tables to use InnoDB, not MyISAM.
Depends on: 633100
Target Milestone: Builder 0.9.1 → Builder 0.9.2
Target Milestone: Builder 0.9.2 → Builder 0.9.3
Target Milestone: Builder 0.9.3 → Builder 0.9.4
The use of transactions was already in place, so with 633100 completed, this should just work.
Status: NEW → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Verified in https://builder-addons-next.allizom.org/

revision is retained when an error occurs in uploading an attachment
Status: RESOLVED → VERIFIED
Product: addons.mozilla.org → addons.mozilla.org Graveyard
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: