1418661 - Sending a translation change result in an Internal server error page

Status: RESOLVED FIXED

(developer.mozilla.org Graveyard :: General, enhancement, P1)

Description

[Clément Lefèvre]

What did you do?
================
1. I updated an article with a translation update

What happened?
==============
It resulted in an Internal Server Error webpage. Translation got sent and updated though.
It's reproduced on a 100% basis, by other people either.

What should have happened?
==========================
It shouldn't show an Internal server error page

Is there anything else we should know?
======================================

Comment 1

[Clément Lefèvre]

This part is not sure: I might have get this error too when going back in my history.

Comment 2

[John Whitlock [:jwhitlock]]

This may be due to an issue introduced when fixing bug 1413289, which is fixed in https://github.com/mozilla/kuma/pull/4532 and should be merged Monday.

Clément, do you remember the URL of the page with the issue?

Comment 3

[Clément Lefèvre]

(In reply to John Whitlock [:jwhitlock] from comment #2)
> This may be due to an issue introduced when fixing bug 1413289, which is
> fixed in https://github.com/mozilla/kuma/pull/4532 and should be merged
> Monday.
> 
> Clément, do you remember the URL of the page with the issue?

I got it for example on https://developer.mozilla.org/fr/Add-ons/WebExtensions/manifest.json/arriere-plan or https://developer.mozilla.org/fr/Add-ons/WebExtensions/manifest.json/auteur
But it seems like every single page is affected anyway.

Comment 4

[John Whitlock [:jwhitlock]]

I've confirmed that the bug related to bug 1413289 is the cause.  The {{AddonSidebar}} adds a link to:

https://developer.mozilla.org/fr/Add-ons/Themes

but the page URL is:

https://developer.mozilla.org/fr/Add-ons/Thèmes

(see the accented è in Thèmes). Our database treats them as equal strings; the new code does not. PR 4532 should fix this tomorrow. Until then, there will be issues rendering French pages using the {{AddonSidebar}} macro.

Comment 7

[John Whitlock [:jwhitlock]]

The fix is pushed to stage and production. I was able to force-refresh the two pages:

https://developer.mozilla.org/fr/Add-ons/WebExtensions/manifest.json/arriere-plan
https://developer.mozilla.org/fr/Add-ons/WebExtensions/manifest.json/auteur

Thanks for the bug report!