Tickets :: [#4285] vcard enhancement - department

6.0.0-beta1

7/7/25

Summary	vcard enhancement - department
Queue	Horde Framework Packages
Queue Version	FRAMEWORK_3
Type	Enhancement
State	Resolved
Priority	1. Low
Owners
Requester	martin (at) matuska (dot) org
Created	08/12/2006 (6904 days ago)
Due
Updated	10/29/2006 (6826 days ago)
Assigned
Resolved	10/29/2006 (6826 days ago)
Milestone
Patch	No

10/29/2006 01:55:22 PM	Jan Schneider	Comment #11 State ⇒ Resolved	Reply to this comment
Committed to HEAD and the stable versions. Thanks for the patience.

10/29/2006 01:46:20 PM	Jan Schneider	Deleted Original Message

10/23/2006 02:37:28 PM	martin (at) matuska (dot) org	Comment #10 New Attachment: vcard.cvs.v3.patch	Reply to this comment
Here is a patch without the constants (using 0 and 1 instead)

10/20/2006 02:44:22 PM	Jan Schneider	Comment #9	Reply to this comment
Unfortunately the patch breaks backward compatibility, sorry that I didn't notice that the first time. You introduce constants to the framework package that you are going to use in Turba. I don't think these constants are really necessary, simply use 0 and 1 instead.

10/20/2006 02:41:05 PM	Jan Schneider	Deleted Original Message

10/20/2006 02:38:30 PM	Jan Schneider	Deleted Original Message

10/20/2006 11:28:29 AM	martin (at) matuska (dot) org	Comment #8 New Attachment: vcard.cvs.v2.patch	Reply to this comment
Are you going to update the patch? Here is a patch to today's CVS. I forgot to include one file in it. Here is an updated one.

10/20/2006 11:24:15 AM	martin (at) matuska (dot) org	Comment #7 New Attachment: vcard.cvs.patch	Reply to this comment
Are you going to update the patch? Here is a patch to today's CVS.

10/10/2006 01:20:42 PM	Jan Schneider	Comment #6	Reply to this comment
Are you going to update the patch?

08/24/2006 02:59:10 PM	Jan Schneider	Comment #5	Reply to this comment
I had mis-understood the patch and didn't notice that the ADR and N cases are now dealt together with the ORG field. The patch looks fine, but please provide a patch against CVS HEAD.

08/22/2006 05:36:48 PM	martin (at) matuska (dot) org	Comment #4	Reply to this comment
What are these ADR and N changes about? They have nothing to do with department support. Please updload a fixed patch. There are no changes to ADR and N. The processing of the ORG attribute is modified - it is a multi-value (and not single-value, as processed without the patch) attribute separated by semicolons so it has to be processed the same way ADR and N are. ORG is a list seperated by unescaped semicolons with a specific number of slots - the VCARD 2.1 specification requires 2 slots, more slots are optional.

08/22/2006 04:01:57 PM	Jan Schneider	Comment #3 State ⇒ Feedback	Reply to this comment
What are these ADR and N changes about? They have nothing to do with department support. Please updload a fixed patch.

08/14/2006 02:54:35 AM	Chuck Hagenbuch	Deleted Original Message

08/12/2006 11:17:58 PM	martin (at) matuska (dot) org	Comment #2 New Attachment: vcard.patch.v2	Reply to this comment
Here is a updated patch, a semicolon was still recorded, if both fields were blank

08/12/2006 12:28:02 PM	martin (at) matuska (dot) org	Comment #1 Priority ⇒ 1. Low State ⇒ New New Attachment: vcard.patch Queue ⇒ Horde Framework Packages Summary ⇒ vcard enhancement - department Type ⇒ Enhancement	Reply to this comment
Attached patch adds organizational unit (department) support for vcard This is required by the vcard 2.1 specification Tested -> Vcard export, SyncML with synthesis client