Closed Bug 809617 Opened 7 years ago Closed 7 years ago

Do not update a translation with the slug of its parent

Categories

(developer.mozilla.org :: Localization, defect, P1, major)

x86
macOS

Tracking

(Not tracked)

VERIFIED FIXED

People

(Reporter: davidwalsh, Unassigned)

References

Details

(Whiteboard: c=Editing p=3)

In researching other slug bugs, I found something very bad:  the incorrect translation slug is sent to the translation edit page.  To duplicate:

1.  Create an en-US document with a slug "html"
2.  Translate to French with a slug of "somethingelse"
3.  Click the edit button for the French document.
4.  Click "save changes"
5.  You'll see that the slug says "html", which is the slug of the root/parent document, *not* the French translation

My research shows that this never quite worked correctly.  If I pass the document_slug as the revision.slug, the translate page 404s.
CC'ing other devs on this because it's a fairly large issue.
Severity: normal → major
A few other things I'll point out:

1.  Redirects are being created properly, so this isn't the end of the world
2.  This is probably causing the occasional "slug is taken" warnings.
Blocks: 782491
Blocks: 798809
Whiteboard: c=Editing
Whiteboard: c=Editing → c=Editing p=
Priority: -- → P1
QA Contact: dwalsh
Summary: Slug Translation Receives Wrong Slug → Do not update a translation with the slug of its parent
Component: Editing → Localization
Depends on: 810088
Depends on: 810089
Depends on: 810090
No longer blocks: 798809
Depends on: 798809
Whiteboard: c=Editing p= → c=Editing p=3
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
verified fixed translated the https://developer.allizom.org/en-US/docs/HTML/HTML5  to French
https://developer.allizom.org/fr/docs/HTML/HTML5testFrench and the new slug was saved
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.