[ISE] When doing a revert

RESOLVED FIXED

Status

Mozilla Developer Network
General
RESOLVED FIXED
3 years ago
3 years ago

People

(Reporter: teoli, Unassigned)

Tracking

({in-triage})

Details

(Whiteboard: [specification][type:bug])

(Reporter)

Description

3 years ago
What did you do?
================
1. Go to https://developer.mozilla.org/en-US/docs/Web/API/URLUtils/hash$revert/471955
2. Added a comment
3. Click the ok button

What happened?
==============
ISE

What should have happened?
==========================
Revert or explicit error message

Is there anything else we should know?
======================================
There was a "move" between current revision and the one I want to revert to.

(I'm manually reverting now as this is needed)
Blocks: 1174209
Keywords: in-triage
We reverted a test page without trouble so it is probably in combination with the move that is the problem.
Hm, the error log (https://errormill.mozilla.org/mdn/mdn/group/398408/) indicates a race condition during saving the revision in question, when it tries to make the old revision the current of the revisions's document. This may be a bigger issue with the autocommit behavior or an issue with the MySQL backend. Needs more eyes on this.
Sent https://github.com/mozilla/kuma/pull/3483 - it doesn't prevent this issue, but it brings the proper error message up to the UI. I.e., you need to delete the redirect document before you can do the revert.

Comment 4

3 years ago
Commits pushed to master at https://github.com/mozilla/kuma

https://github.com/mozilla/kuma/commit/b32a29f4fc7bf3fff97d56e951c17879e68d9a8b
fix bug 1174498 - error message for revert dupes

When a user reverts a document to an old revision, we want to catch the
IntegrityError if it collides with the document's redirect, so the user knows
they can fix it by deleting the redirect.

https://github.com/mozilla/kuma/commit/1691b674170a8c1e7eb72c5f97e5ef712d1a7fcb
Merge pull request #3483 from mozilla/revert-dupe-ise-1174498

fix bug 1174498 - error message for revert dupes

Updated

3 years ago
Status: NEW → RESOLVED
Last Resolved: 3 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.