6.0.0-alpha10
▾
Tasks
New Task
Search
Photos
Wiki
▾
Tickets
New Ticket
Search
dev.horde.org
Toggle Alerts Log
Help
5/15/25
H
istory
A
ttachments
C
omment
W
atch
Download
Comment on [#9554] store multiple email addresses as separate 'mail' attributes in LDAP
*
Your Email Address
*
Spam protection
Enter the letters below:
.___._..__. __.\ / [__ | | |(__ >< | _|_|__\.__)/ \
Comment
>> 1) The email form field always returns a single value, though we >> could of course split this up again, or add an option to return an >> array instead. >> 2) But more important, there isn't any way in Turba to store one >> field into several attributes. We do have composite fields, but these >> store in a *fixed* number of *different* attributes, not in a >> *variable* number of the *same* attribute. There is no infrastructure >> for that at all, so this is a larger undertaking. > > Is this necessary at all? > As far as I can say, the emails (multi_value) field works fine with > IMP and ActiveSync. > > It works with ActiveSync in the expected way, the first three email > addresses from emails are provided as email1, email2 and email3 to an > activesync device and read this way from the device. (three email > address are the limit of activesync) > > this happens ins turba/lib/Driver.php at line 2690: > case 'emails': > $address = 1; > foreach (explode(',', $value) as $email) { > while ($address <= 3 && > $message->{'email' . $address . 'address'}) { > $address++; > } > if ($address > 3) { > break; > } > $message->{'email' . $address . 'address'} = $email; > $address++; > } > break; > > A smiliar code snippet is there for vcards. > > For LDAP I find the "array_merge" inside the _read function off > Ldap.php. In funtion _write, there is nothing. > > My suggestions: > > 1) generic version > at line 304: > if source is comma seperated array and target mutli value > for i in array ... > this one might be dangerous with other fields als children/child > > 2) > Or a hardcoded version just for "emails" > somehow ugly but works and no side effects > > 3) > in backends.local.php the opportunity/example for 2) > > My php skills are just not good enough at the moment :(
Attachment
Watch this ticket
N
ew Ticket
M
y Tickets
S
earch
Q
uery Builder
R
eports
Saved Queries
Open Bugs
Bugs waiting for Feedback
Open Bugs in Releases
Open Enhancements
Enhancements waiting for Feedback
Bugs with Patches
Enhancements with Patches
Release Showstoppers
Stalled Tickets
New Tickets
Horde 5 Showstoppers