815564 - B2G SMS: 7bits encoding is not working fine when mixing replaceable characters like á with non replaceable characters like ä

Status: RESOLVED WONTFIX

(Core :: DOM: Device Interfaces, defect)

Description

[Carlos Martínez Toral [:carlosmartinez]]

Tested in unagi device with:

gaia 56e45e8
gecko 4eaacd3

STR:
1-Open SMS app with a SIM with enableStrict7BitEncodingForSms="true"
2-Write a message containing:
   Ááàâãäåæ Ééèêëęē€ɛ Ííïìîīį Óóöòôōœøϴ Úúüùûū
3-Type destination number
4-Tap in send button

Expected result --> Destination device should receive: Aaàâãäåæ Eeèêëęē€ɛ Iiïìîīį Ooöòôōœøϴ Uuüùûū 
Actual result --> Destination device receives: Ááàâãäåæ Ééèêëęē€ɛ Ííïìîīį Óóöòôōœøϴ Úúüùûū

Comment 1

[José Antonio Olivera Ortega [:jaoo]]

Nom'ing for basecamp since it's a blocker for v1.

Comment 2

[Carlos Martínez Toral [:carlosmartinez]]

Making some tests with a VIVO SIM card, I´ve seen that üÜ characters are not visible in destination device, although something is sent because if I wrote a SMS with only üÜ I receive an empty message.

Comment 3

[Vicamo Yang [:vicamo][:vyang]]

The input string "Ááàâãäåæ Ééèêëęē€ɛ Ííïìîīį Óóöòôōœøϴ Úúüùûū" has following code values:
U+c1 U+e1 U+e0 U+e2 U+e3 U+e4 U+e5 U+e6 U+20 U+c9 U+e9 U+e8 U+ea U+eb U+119 U+113 U+20ac U+25b U+20 U+cd U+ed U+ef U+ec U+ee U+12b U+12f U+20 U+d3 U+f3 U+f6 U+f2 U+f4 U+14d U+153 U+f8 U+3f4 U+20 U+da U+fa U+fc U+f9 U+fb U+16b

But characters "â"(U+e2), "ã"(U+e3), ... are neither part of SMS default 7-Bit Alphabet nor characters listed in bug 790192. The message will then be encoded in UCS2, and that's a CORRECT behaviour. You should either send me a full list of characters that should follow the transformation, or accept it.

PS. You can this utility web tool to convert unicode strings: http://timc.idv.tw/encodetest/

Comment 4

[Vicamo Yang [:vicamo][:vyang]]

Attachment: add test script: WIP

Depends on bug 759529.

Comment 5

[Andrew Overholt [:overholt]]

Please re-nom for blocking-basecamp if you re-open.

Comment 6

[Beatriz Rodríguez [:brg]]

We agreed to not-reopen this issue and follow next actions in https://bugzilla.mozilla.org/show_bug.cgi?id=816082