1390111 - mobile/android/base unescaping should handle \' and \u0027

Status: RESOLVED FIXED

(Webtools Graveyard :: Pontoon, enhancement, P3)

Description

[Axel Hecht [:Pike]]

Pootle escapes ' as \u0027 while pontoon escapes it as \' for Android DTD files. Bah, bad Android, no cookie.

On unescaping, pontoon should take both encodings and map them to '. Probably same for ", too.

Comment 1

[[github robot]]

Commit pushed to master at https://github.com/mozilla/pontoon

https://github.com/mozilla/pontoon/commit/eb5f8d553f7efc9ebe30e063921cce353c8b6f5d
Fix bug 1390111: Unescape \u0027 & \u0022 in Android on sync

Comment 2

[Jordi Serratosa]

The \u0027 & \u0022  characters now look good in Pontoon, but they appear in the downloadable TMX translation memory. 
For example, in: https://pontoon.mozilla.org/ca/firefox-for-android/ca.firefox-for-android.tmx

The characters \u0027 & \u0022 should be also unescaped in the TMX.

Comment 3

[GitHub Bugzilla PR Linker]

Attachment: Link to GitHub pull-request: https://github.com/mozilla/pontoon/pull/738

Comment 4

[[github robot]]

Commit pushed to master at https://github.com/mozilla/pontoon

https://github.com/mozilla/pontoon/commit/682cec26cd02c05dd0604e3c6542b6a5c16a2ddf
Fix bug 1390111: Unescape Android quotes in TM (#738)

Handling of different escape sequences for mobile Android strings was
fixed in eb5f8d5. However, data migration in 4537eac only covered the
Translation model, leaving out TranslationMemoryEntry. See bugs 1390111
and 1390113 for more details.