[#8654] trimming composite fields
Summary trimming composite fields
Queue Turba
Queue Version 2.3.2
Type Enhancement
State Resolved
Priority 1. Low
Owners Jan Schneider <jan (at) horde (dot) org>
Requester xk3 (at) mompl (dot) org
Created 10/20/09 (149 days ago)
Due
Updated 01/12/10 (65 days ago)
Assigned
Resolved 10/20/09 (149 days ago)
Attachments
Milestone 2.3.3
Patch No

History
01/12/10 CVS Commit Comment #4 Reply to this comment
Changes have been made in Git for this ticket:

Condense whitespace in composite fields (Request #8654).

http://git.horde.org/diff.php/turba/docs/CHANGES?rt=horde-git&r1=0b2556e1780336d4bbb3e0e4874acf008754a7f5&r2=5e439288c37893b800669b3d521dd84d59d0d41f
http://git.horde.org/diff.php/turba/lib/Driver.php?rt=horde-git&r1=82bb2e1d13c4b192fa4d6156e0aa1ac5597190fa&r2=5e439288c37893b800669b3d521dd84d59d0d41f
10/20/09 Jan Schneider Milestone ⇒ 2.3.3
State ⇒ Resolved
Summary ⇒ trimming composite fields
Assigned to Jan Schneider
 
10/20/09 CVS Commit Comment #3 Reply to this comment
Changes have been made in CVS for this ticket:

MFH: Condense whitespace in composite fields (Request #8654).
http://cvs.horde.org/diff.php/turba/docs/CHANGES?rt=horde&r1=1.181.2.240&r2=1.181.2.241&ty=u
http://cvs.horde.org/diff.php/turba/lib/Driver.php?rt=horde&r1=1.57.2.91&r2=1.57.2.92&ty=u
10/20/09 CVS Commit Comment #2 Reply to this comment
Changes have been made in Git for this ticket:

Condense whitespace in composite fields (Request #8654).

http://git.horde.org/diff.php/turba/docs/CHANGES?rt=horde-hatchery&r1=0b2556e1780336d4bbb3e0e4874acf008754a7f5&r2=5e439288c37893b800669b3d521dd84d59d0d41f
http://git.horde.org/diff.php/turba/lib/Driver.php?rt=horde-hatchery&r1=82bb2e1d13c4b192fa4d6156e0aa1ac5597190fa&r2=5e439288c37893b800669b3d521dd84d59d0d41f
10/20/09 xk3 (at) mompl (dot) org Comment #1
State ⇒ New
Patch ⇒
Milestone ⇒
Queue ⇒ Turba
Summary ⇒ trimming comopsite fields
Type ⇒ Enhancement
Priority ⇒ 1. Low		 Reply to this comment
a trim() is often not enough for a composite field, if some of the 
fields it is composed from are empty.



I always patch in toDriverKeys() the trim() with a 
preg_replace('/\s\s+/', ' ', ...). This avoids names such as 
"Firstname__Lastname", if the middle name is empty.