Skip to content

Revert to decoding of XML data using Erlang unicode module#115

Closed
trbngr wants to merge 1 commit intoelixir-ecto:masterfrom
oforce:raw-xml-decode
Closed

Revert to decoding of XML data using Erlang unicode module#115
trbngr wants to merge 1 commit intoelixir-ecto:masterfrom
oforce:raw-xml-decode

Conversation

@trbngr
Copy link
Copy Markdown

@trbngr trbngr commented Nov 7, 2020

No description provided.

@coveralls
Copy link
Copy Markdown

coveralls commented Nov 7, 2020

Coverage Status

Coverage increased (+0.07%) to 73.204% when pulling 4c57f86 on oforce:raw-xml-decode into 11e2f91 on livehelpnow:master.

@mjaric
Copy link
Copy Markdown
Member

mjaric commented Dec 3, 2020

Thanks for PR! Since xml transferred as regular ucs2 string, instead of merging this PR I changed global ucs2_to_utf8 function, so it affects all strings, also I made unit test that cheks all ucs2 chars decoding, so it should be covered completely from now on when erlang :unicode module is used.

@mjaric mjaric closed this Dec 3, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants