Tickets :: [#12194] ActiveSync: Children field is not synced

6.0.0-beta1

7/7/25

Summary	ActiveSync: Children field is not synced
Queue	Turba
Queue Version	Git master
Type	Bug
State	Resolved
Priority	1. Low
Owners	mrubinsk (at) horde (dot) org
Requester	thomas.jarosch (at) intra2net (dot) com
Created	04/18/2013 (4463 days ago)
Due
Updated	04/22/2013 (4459 days ago)
Assigned	04/19/2013 (4462 days ago)
Resolved	04/19/2013 (4462 days ago)
Github Issue Link
Github Pull Request
Milestone
Patch	Yes

04/22/2013 01:56:54 PM	Thomas Jarosch	Comment #10	Reply to this comment
The array fix works fine.

04/19/2013 10:47:52 PM	Git Commit	Comment #9	Reply to this comment
Changes have been made in Git (FRAMEWORK_5_0): commit 2889d60ba9ed57c55aa102f2e7eadbfb4abe816f Author: Michael J Rubinsky <mrubinsk@horde.org> Date: Fri Apr 19 18:39:49 2013 -0400 Revert "~~Request: 12194~~ Sync children field to activesync devices." This reverts commit ca62a608cff37dfbbf0509152d2e06c48c0ff845. This is a newly supported field, belongs in 4.1. turba/lib/Driver.php \| 16 ++-------------- 1 files changed, 2 insertions(+), 14 deletions(-) http://git.horde.org/horde-git/-/commit/2889d60ba9ed57c55aa102f2e7eadbfb4abe816f

04/19/2013 10:47:48 PM	Git Commit	Comment #8	Reply to this comment
Changes have been made in Git (FRAMEWORK_5_0): commit ca62a608cff37dfbbf0509152d2e06c48c0ff845 Author: Thomas Jarosch <thomas.jarosch@intra2net.com> Date: Thu Apr 18 11:31:05 2013 -0400 ~~Request: 12194~~ Sync children field to activesync devices. Signed-off-by: Michael J Rubinsky <mrubinsk@horde.org> turba/lib/Driver.php \| 16 ++++++++++++++-- 1 files changed, 14 insertions(+), 2 deletions(-) http://git.horde.org/horde-git/-/commit/ca62a608cff37dfbbf0509152d2e06c48c0ff845

04/19/2013 08:45:04 PM	Michael Rubinsky	State ⇒ Resolved

04/19/2013 04:18:37 PM	Michael Rubinsky	Comment #7	Reply to this comment
[Show Quoted Text - 27 lines][Hide Quoted Text] Hi, I've seen you have changed the support a little with this: --------------------- commit 307c2890949e2771e137b42827fe3b2a9bcd9363 Author: Michael J Rubinsky <mrubinsk@horde.org> Date: Thu Apr 18 12:39:19 2013 -0400 Use a more likely delimter for the freeform children field. We will always export this as a single field instead of splitting on the delimiter since we can't guarentee what delimiter the user will use and a comma (the most likely choice) can also be used to seperate name suffixes (at least in some languages). --------------------- I've tested that code and have the impression we generate ActiveSync XML data not according to the spec: With the new code, we send data like this: --------------------- 2013-04-19T17:34:15+02:00 DEBUG: [12947] O <POOMCONTACTS:Children> 2013-04-19T17:34:15+02:00 DEBUG: [12947] O Hans, blupper, horst, knut 2013-04-19T17:34:15+02:00 DEBUG: [12947] O <POOMCONTACTS:Children/> --------------------- Yup. My bad. We had to change splitting up the list to just sending it freeform, but neglected to wrap the string in an array to make up for the lack of explode().

04/19/2013 03:54:30 PM	Thomas Jarosch	Comment #6	Reply to this comment
-> We are lacking the "child" XML attribute. "XML node" of course, fat fingered this one...

04/19/2013 03:53:58 PM	Thomas Jarosch	Comment #5 State ⇒ Assigned	Reply to this comment
Hi, I've seen you have changed the support a little with this: --------------------- commit 307c2890949e2771e137b42827fe3b2a9bcd9363 Author: Michael J Rubinsky <mrubinsk@horde.org> Date: Thu Apr 18 12:39:19 2013 -0400 Use a more likely delimter for the freeform children field. We will always export this as a single field instead of splitting on the delimiter since we can't guarentee what delimiter the user will use and a comma (the most likely choice) can also be used to seperate name suffixes (at least in some languages). --------------------- I've tested that code and have the impression we generate ActiveSync XML data not according to the spec: With the new code, we send data like this: --------------------- 2013-04-19T17:34:15+02:00 DEBUG: [12947] O <POOMCONTACTS:Children> 2013-04-19T17:34:15+02:00 DEBUG: [12947] O Hans, blupper, horst, knut 2013-04-19T17:34:15+02:00 DEBUG: [12947] O <POOMCONTACTS:Children/> --------------------- My old code sent data like this: --------------------- 2013-04-19T17:47:01+02:00 DEBUG: [12973] O <POOMCONTACTS:Children> 2013-04-19T17:47:01+02:00 DEBUG: [12973] O <POOMCONTACTS:Child> 2013-04-19T17:47:01+02:00 DEBUG: [12973] O Hans 2013-04-19T17:47:01+02:00 DEBUG: [12973] O <POOMCONTACTS:Child/> 2013-04-19T17:47:01+02:00 DEBUG: [12973] O <POOMCONTACTS:Child> 2013-04-19T17:47:01+02:00 DEBUG: [12973] O blupper 2013-04-19T17:47:01+02:00 DEBUG: [12973] O <POOMCONTACTS:Child/> 2013-04-19T17:47:01+02:00 DEBUG: [12973] O <POOMCONTACTS:Child> 2013-04-19T17:47:01+02:00 DEBUG: [12973] O horst 2013-04-19T17:47:01+02:00 DEBUG: [12973] O <POOMCONTACTS:Child/> 2013-04-19T17:47:01+02:00 DEBUG: [12973] O <POOMCONTACTS:Child> 2013-04-19T17:47:01+02:00 DEBUG: [12973] O knut4 2013-04-19T17:47:01+02:00 DEBUG: [12973] O <POOMCONTACTS:Child/> 2013-04-19T17:47:01+02:00 DEBUG: [12973] O <POOMCONTACTS:Children/> --------------------- Or for a single entry: --------------------- 2013-04-19T17:50:28+02:00 DEBUG: [10285] O <POOMCONTACTS:Children> 2013-04-19T17:50:28+02:00 DEBUG: [10285] O <POOMCONTACTS:Child> 2013-04-19T17:50:28+02:00 DEBUG: [10285] O Hans 2013-04-19T17:50:28+02:00 DEBUG: [10285] O <POOMCONTACTS:Child/> 2013-04-19T17:50:28+02:00 DEBUG: [10285] O <POOMCONTACTS:Children/> --------------------- -> We are lacking the "child" XML attribute.

04/18/2013 03:32:27 PM	Michael Rubinsky	Comment #4 Assigned to Michael Rubinsky State ⇒ Resolved	Reply to this comment
Committed, thanks!

04/18/2013 03:32:01 PM	Git Commit	Comment #3	Reply to this comment
Changes have been made in Git (master): commit f3c57e3e89cafe58fc50caa853ac82defc4cdbdc Author: Thomas Jarosch <thomas.jarosch@intra2net.com> Date: Thu Apr 18 11:31:05 2013 -0400 ~~Request: 12194~~ Sync children field to activesync devices. Signed-off-by: Michael J Rubinsky <mrubinsk@horde.org> turba/lib/Driver.php \| 16 ++++++++++++++-- 1 files changed, 14 insertions(+), 2 deletions(-) http://git.horde.org/horde-git/-/commit/f3c57e3e89cafe58fc50caa853ac82defc4cdbdc

04/18/2013 02:06:18 PM	Thomas Jarosch	Comment #2 Patch ⇒ Yes New Attachment: 0001-ActiveSync-Add-support-for-the-children-field-bug-12.patch	Reply to this comment
Patch to support syncing of the children field.

04/18/2013 02:04:26 PM	Thomas Jarosch	Comment #1 Patch ⇒ No State ⇒ Unconfirmed Milestone ⇒ Queue ⇒ Turba Summary ⇒ ActiveSync: Children field is not synced Type ⇒ Bug Priority ⇒ 1. Low	Reply to this comment
Hi, attached patch adds support to sync the children field. Tested with a Nokia Lumia 920 (WP8), I didn't have another system at hand that supports this attribute. Cheers, Thomas