Tickets :: [#9964] SyncML module produces wrong WBXML encoding

Summary	SyncML module produces wrong WBXML encoding - syncing with some phones may fail
Queue	Synchronization
Queue Version	Git master
Type	Bug
State	Resolved
Priority	2. Medium
Owners	jan (at) horde (dot) org
Requester	kaibin.bao (at) baihu (dot) de
Created	04/25/2011 (5156 days ago)
Due
Updated	04/27/2011 (5154 days ago)
Assigned
Resolved	04/27/2011 (5154 days ago)
Milestone
Patch	No

04/27/2011 01:12:12 PM	Jan Schneider	Comment #3 Assigned to Jan Schneider State ⇒ Resolved Patch ⇒ No	Reply to this comment
Your patch was not quite correct, because it moved the the line to a wrong place, but the general solution was correct.

04/27/2011 01:11:33 PM	Git Commit	Comment #2	Reply to this comment
Changes have been made in Git for this ticket: [jan] Don't set codepage multiple times (~~Bug #9964~~). 2 files changed, 2 insertions(+), 4 deletions(-) http://git.horde.org/horde-git/-/commit/93a44ef2f196213216ff895de4e0aab6a589d66f

04/25/2011 05:27:33 AM	kaibin (dot) bao (at) baihu (dot) de	Comment #1 Priority ⇒ 2. Medium New Attachment: wbxmlencoder.patch Patch ⇒ Yes Milestone ⇒ Queue ⇒ Synchronization Summary ⇒ SyncML module produces wrong WBXML encoding - syncing with some phones may fail Type ⇒ Bug State ⇒ Unconfirmed	Reply to this comment
Bug in the WbXml-Encoder prevents syncing with my cell phone (Samsung GT-5230): In horde/framework/Xml_Wbxml/lib/Horde/Xml/Wbxml/Encoder.php, Horde_Xml_Wbxml_Encoder::startElement() calls changecodepage() serveral times because changecodepage() sets $this->_currentURI to a value other than $uri when called (see line 400). The patch moves the code line "$this->_currentURI = $uri;" from changecodepage() to startElement() to improve locality of variables.