jpilot inserts bogus text in vCard export

Bug #366543 reported by David Gibson
2
Affects Status Importance Assigned to Milestone
jpilot (Ubuntu)
Fix Released
Undecided
Unassigned

Bug Description

Binary package hint: jpilot

The version of jpilot in Ubuntu 9.04 (that is jpilot 1.6.0-1) incorrectly exports to vcard format.

When exporting addresses to vcard format, jpilot it inserts an extra copy of the category information in front of
the FN: tag, meaning the vcf output is invalid.

i.e. it produces:
        BEGIN:VCARD
        VERSION:3.0
        PRODID:-//Judd Montgomery//NONSGML J-Pilot 1.6.0//EN
        UID:palm-addressbook-003430dd-000003e8-David Gibson@mulberryst
        CATEGORIES:Business
--> "Business",FN:ABN Amro Morgans
        N:ABN Amro Morgans
        ORG:ABN Amro Morgans
        TEL;TYPE=work,pref:6232 4999
        ADR:;;Level 1\, 25 Napier Close\, Deakin;;;;

        END:VCARD

When it should produce this:
        BEGIN:VCARD
        VERSION:3.0
        PRODID:-//Judd Montgomery//NONSGML J-Pilot 1.6.0//EN
        UID:palm-addressbook-003430dd-000003e8-David Gibson@mulberryst
        CATEGORIES:Business
--> FN:ABN Amro Morgans
        N:ABN Amro Morgans
        ORG:ABN Amro Morgans
        TEL;TYPE=work,pref:6232 4999
        ADR:;;Level 1\, 25 Napier Close\, Deakin;;;;

        END:VCARD

I've located the source of the problem, it's in address_gui.c:
         utf = charset_p2newj(contact_app_info.category.name[mcont->attrib & 0x0F], 16, char_set);
         str_to_vcard_str(csv_text, sizeof(csv_text), utf);
         fprintf(out, "CATEGORIES:%s%s", csv_text, CRLF);
         fprintf(out, "\"%s\",", utf);
         g_free(utf);

The last fprintf() line is bogus and should be removed. This problem is also in jpilot's upstream CVS.

Revision history for this message
Ludovic Rousseau (ludovic-rousseau-gmail) wrote :

Now fixed upstream in CVS version 1.229
http://lists.jpilot.org/pipermail/jpilot-commits/2009-April/000000.html

Will be fixed in Debian unstable soon.

Revision history for this message
Ludovic Rousseau (ludovic-rousseau-gmail) wrote :

Fixed in Debian package version 1.6.2.3-1.

I don't know when the Ubuntu package will be rsynchronized with the Debian one.

Revision history for this message
Ludovic Rousseau (ludovic-rousseau-gmail) wrote :

natty now has version 1.8.0.2.
Closing the bug report.

Changed in jpilot (Ubuntu):
status: New → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.