After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 710178 - Skip whitespace between oFono vCards
Skip whitespace between oFono vCards
Status: RESOLVED FIXED
Product: folks
Classification: Platform
Component: oFono backend
git master
Other All
: Normal minor
: Unset
Assigned To: folks-maint
folks-maint
Depends on:
Blocks:
 
 
Reported: 2013-10-15 11:26 UTC by Philip Withnall
Modified: 2013-10-22 06:50 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
ofono: Skip whitespace between vCards when splitting them up (1.56 KB, patch)
2013-10-15 11:28 UTC, Philip Withnall
committed Details | Review

Description Philip Withnall 2013-10-15 11:26:22 UTC
The vCards we get from oFono are typically separated by a blank line, and the code currently doesn’t strip that line, resulting in lots of these warnings:

    libebook-contacts-WARNING **: vcard began without a BEGIN:VCARD

libebook then skips the blank line and correctly parses the vCard.

Patch coming to skip the line ourselves and eliminate the warnings.
Comment 1 Philip Withnall 2013-10-15 11:28:16 UTC
Created attachment 257344 [details] [review]
ofono: Skip whitespace between vCards when splitting them up

This eliminates the following libebook warning which we were triggering.
libebook itself skips the whitespace, so this introduces no functional
changes.

    libebook-contacts-WARNING **: vcard began without a BEGIN:VCARD
Comment 2 Philip Withnall 2013-10-22 06:50:39 UTC
Comment on attachment 257344 [details] [review]
ofono: Skip whitespace between vCards when splitting them up

Reviewed on IRC by Xavier.

commit f9907524e16a121b7b88ca369b34a7719c510072
Author: Philip Withnall <philip.withnall@collabora.co.uk>
Date:   Tue Oct 15 12:27:06 2013 +0100

    ofono: Skip whitespace between vCards when splitting them up
    
    This eliminates the following libebook warning which we were triggering.
    libebook itself skips the whitespace, so this introduces no functional
    changes.
    
        libebook-contacts-WARNING **: vcard began without a BEGIN:VCARD
    
    https://bugzilla.gnome.org/show_bug.cgi?id=710178

 NEWS                                    | 1 +
 backends/ofono/ofono-persona-store.vala | 4 ++++
 2 files changed, 5 insertions(+)