Tickets :: Comment on [#14944] CardDAV: Bad comma separation for CATEGORIES property, aka tags

* Your Email Address
* Spam protection	Enter the letters below: . .. .. .. ,.__ \| \|\|\ \|\|\/\| \./ \| \ \|__\|\| \\|\| \| \| \|__/
Comment	> Thanks for digging into this. And for pointing out where I was wrong. > > The trimming for CardDAV export is done in function tovCard(...) , > case '__tags': > https://github.com/horde/turba/blob/master/lib/Driver.php#L1818 > leading to > https://github.com/horde/Core/blob/master/lib/Horde/Core/Tagger.php#L89 > > I was seeing the misbehavior with a Debian Jessie installation (which > is oldoldstable now), where > case '__tags': is not calling Turba_Tagger::split() but doing a > simple explode(',', $val) instead. And unfortunately I thought the > error is caused by synchronizeTags(), which hasn't changed much since > then. > > So this was fixed ages ago, for Debian since Stretch/9 , as I see now. > Sorry for stealing your time, Michael. > > >> ...and I can't reproduce this using the 'Card Book' add on to >> Thunderbird as described in the ticket. My tags are being correctly >> trimmed as they are synchronized. >> >> Please provide a step-by-step way of reproducing this. >
Attachment
Watch this ticket