Last modified: 2013-09-14 16:09:17 UTC

Wikimedia Bugzilla is closed!

Wikimedia migrated from Bugzilla to Phabricator. Bug reports are handled in Wikimedia Phabricator.
This static website is read-only and for historical purposes. It is not possible to log in and except for displaying bug reports and their history, links might be broken. See T55747, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 53747 - VisualEditor breaks when Polish-language diacritics (ążśźęćńół) are input
VisualEditor breaks when Polish-language diacritics (ążśźęćńół) are input
Status: RESOLVED FIXED
Product: VisualEditor
Classification: Unclassified
Language (Other open bugs)
unspecified
All All
: Highest critical
: VE-deploy-2013-09-12
Assigned To: D Chan
https://pl.wikipedia.org/wiki/Wikiped...
: i18n
Depends on:
Blocks: ve-nonenglish
  Show dependency treegraph
 
Reported: 2013-09-04 12:09 UTC by Bartosz Dziewoński
Modified: 2013-09-14 16:09 UTC (History)
11 users (show)

See Also:
Web browser: ---
Mobile Platform: ---
Assignee Huggle Beta Tester: ---


Attachments

Description Bartosz Dziewoński 2013-09-04 12:09:32 UTC
Since approx. August 30-31 VisualEditor breaks when Polish-language diacritics (ążśźęćń󳥯ŚŹĘĆŃÓŁ) are input. The characters themselves disappear and text around them gets mangled and moved around.

They're all input using AltGr (right Alt) + diacritic-less version of the characters, with the exception of "x" mapping to "ź" (so, respectively, azsxecnol).

I have been unable to replicate the issue myself, but it's definitely happening.

pl.wp thread: https://pl.wikipedia.org/wiki/Wikipedia:Kawiarenka/Kwestie_techniczne#VisualEditor_a_polskie_diakrytyki

This is repeatedly breaking page text on the Polish Wikipedia, so I'm marking the bug "highest critical".
Comment 1 Bartosz Dziewoński 2013-09-04 12:11:08 UTC
Examples of edits apparently exhibiting this issue:

* https://pl.wikipedia.org/w/index.php?title=Rondo&curid=122034&diff=37421631&oldid=35999089 - "Największe rondo" -> "Najwi kszerondo"

* https://pl.wikipedia.org/w/index.php?title=Euronews&diff=prev&oldid=37394282 - "dostpna" should be "dostępna", "Midzyrnodowym" should be "Międzynarodowym", etc.

* https://pl.wikipedia.org/w/index.php?title=1967&diff=next&oldid=37422197 - "ki rapers" should probably be "Amerykański raper" (both occurences - this happened on two consecutive edits)

Results of a user trying to type in "Pójdź, kińże tę chmurność w głąb flaszy!" when reproducing this bug:
* https://www.mediawiki.org/w/index.php?title=User:G%C5%BCdacz&oldid=775661
* https://www.mediawiki.org/w/index.php?title=User:G%C5%BCdacz&oldid=775666
Comment 2 Bartosz Dziewoński 2013-09-04 12:23:33 UTC
Actually, I sort of have reproduced it now. I used Opera 12, but some users at pl.wp reported it happening on Chrome – it's probably not browser-dependent.

You'll probably need to set your system keyboard to "Polish (programmer's)" or similarly named one. (It's the same as standard US layout, but includes the AltGr diacritics. A "Polish (typist's)" layout exists on some systems, but nobody ever uses it, so forget it.)

1.
Try inputting some text with diacritics, such as the "Pójdź, kińże tę chmurność w głąb flaszy!" sentence above (copying and pasting text with diacritics doesn't cause issues, you need to type it). (Strings of one character such as "ąąąą" tend to work correctly for some reason.)

2.
Depending on your luck, it might look okay, or you might get parts of the sentence duplicated in the next paragraph. Regardless of that, try backspacing a little now. The cursor will likely be moved to someplace in the next paragraph and unrelated characters will be removed.

3.
Try previewing the changes. The text will differ from the one shown in editing view.
Comment 3 Oliver Keyes 2013-09-04 16:48:09 UTC
https://pl.wikipedia.org/w/index.php?title=Wikipedia:VisualEditor/Opinie&oldid=37428181#.C4.99 is a pretty good explanation, and makes clear it's not browser (or OS) dependent.
Comment 4 Chris McKenna 2013-09-04 16:54:06 UTC
Bug 53680 shows this also happens on en.wp and fr.wp and is independent of keyboard layout.
Comment 5 Moriel Schottlender 2013-09-05 17:42:51 UTC
I've tested this on master in Firefox under Ubuntu -- I managed to replicate the behavior not just in polish, but also Spanish diacritics but only when these are "added on". Trying to add diacritic to an existing character produced a pawn and then some odd duplication of characters.

That did not happen when I typed in either Hebrew or Arabic in master.

Another thing I found is that there seems to be a difference between an already-combined diacritic (like ñ which is native to the Spanish keyboard) and an "added" diacritic. When I typed the native ñ nothing broke, it added it properly and the behavior was correct. 

And finally, the typing broke and the pawn appeared only when I tried to add diacritic to a latin letter. 

When I tried to add "niqqud" to Hebrew letters, though, like pressing ש and left-Alt+a to produce שְ the behavior was as expected (no pawn, no problem)
Comment 6 Bartosz Dziewoński 2013-09-09 16:45:23 UTC
Is there any progress on this? Because today's the point when I'd suggest reverting the deployment to last known good version if nobody is going to fix this.
Comment 7 D Chan 2013-09-09 16:53:58 UTC
I'm optimistic that the following patch will fix this bug:

https://gerrit.wikimedia.org/r/#/c/82858/

If someone can test this hypothesis, great; if not, then I will hopefully be able to do so later today.
Comment 8 D Chan 2013-09-11 23:47:08 UTC
Ok, the patch is merged, and due to go live on mediawiki.org by 13 September 2013:

https://gerrit.wikimedia.org/r/#/c/82858/

Please let us know whether it fixes the bug!
Comment 9 James Forrester 2013-09-12 04:01:51 UTC
Marking this as "FIXED" on the expectation that it's fixed - please re-open if you find that it is still occurring.
Comment 10 Bartosz Dziewoński 2013-09-14 16:09:17 UTC
It seems like it does actually fix the issue (for me at least). Thanks.

So that means editing will have been completely broken for only three weeks straight by the time this gets deployed :/

Note You need to log in before you can comment on or make changes to this bug.


Navigation
Links