Special characters in Calendar subjects (or wherever) cause problems.

[GoogleCodeExporter]

What steps will reproduce the problem?
1. Create a new Calendar entry containing an ampersand character on the phone
2. Synchronize the phone
3. The entry is not properly synchronized.

What is the expected output? What do you see instead?
Special characters must be allowed in Calendar subjects (or wherever). E.g. 
"Coffee & Cake"

What version of the product are you using? On what operating system?
On Froyo with rev 145

Please provide any additional information below.
XML text nodes are not escaped during creation of the XML document. This should 
be done using any escaping method, e.g. 
"xml.createTextNode(TextUtils.htmlEncode(value));".
There might be better ways out there to escape to valid XML...

Please see attached diff file.

Original issue reported on code.google.com by must...@gmx.de on 20 Dec 2010 at 4:38

Attachments:

• ampersandFix.diff

[GoogleCodeExporter]

Hmm, if you encode the text... where do you decode it then? :)

And btw: why dont you have a committer account if you're already providing 
patches :)

We'll switch to GIT soon, so you might want to get permission to push to git 
then :)

Original comment by STrille...@googlemail.com on 20 Dec 2010 at 6:56

[GoogleCodeExporter]

Unescaping (decoding) seems to be properly done by the org.w3c.dom XML parser 
classes. I made a test with an appointment of subject 
"a@~`|•√Π÷×{}][=_^°€¢£  ™®©¶\<>„!"':;/?+-()a" and 
synchronised it twice. Once from an once to the phone and it worket without any 
problems...

Good question ;-) I already wrote an email directly to Arthur concerning the 
commiter account and I am looking forward to participate in the project.

Original comment by must...@gmx.de on 21 Dec 2010 at 8:36

[GoogleCodeExporter]

I've committed your fix to the froyo branch on gitorious.

@musterh75: Please create yourself an account at gitorious
@Arthur: We need a committer account for musterh75 on gitorious :)

Original comment by STrille...@googlemail.com on 10 Jan 2011 at 12:10

• Changed state: Fixed