Tickets :: Comment on [#14301] CarDav Sync problem

* Your Email Address
* Spam protection	Enter the letters below: . ..__..__.. .._. \| \|[__]\| \|\| \| \| \|/\\|\| \|\|__\\|__\|_\|_
Comment	>> This is a follow-up of bug 12955 and bug 14046. _ensureEmail() exits >> early (even after the recent logic fix), because it doesn't do it's >> job, as soon as an email field exists. For one it doesn't check if >> that field actually has data. > > In my testing, $this->attributes[somefield] is only set if it > contains data. The method's job was only to ensure that there was at > least one email field populated if we have the data - which is why > it exits early. > >> And even if it would, that won't work >> with editing existing contacts that may already have email fields >> set, if they are updated with email fields that have TYPE set. > > I will look at this case, though this sounds messy. Especially if the > TYPE doesn't match any of the configured email fields.
Attachment
Watch this ticket