Closed
Bug 1139965
Opened 9 years ago
Closed 9 years ago
Implement function to export addressbook in vCard format
Categories
(MailNews Core :: Address Book, enhancement)
MailNews Core
Address Book
Tracking
(Not tracked)
RESOLVED
FIXED
Thunderbird 39.0
People
(Reporter: neil, Assigned: neil)
Details
Attachments
(1 file)
6.14 KB,
patch
|
mkmelin
:
review+
|
Details | Diff | Splinter Review |
We can import addressbooks in LDIF, CSV, TXT and VCF format. We can export addressbooks in LDIF, CSV and TXT, but VCF is missing. This isn't quite the same as bug 924530 because that's for single cards.
Assignee | ||
Comment 1•9 years ago
|
||
Note that translateTo doesn't export all address card fields, only the ones on the main and work tabs as far as I can tell. (Getting it to export more of the card would be a separate bug, and would depend on being able to import too of course.)
Assignee | ||
Updated•9 years ago
|
Attachment #8573330 -
Flags: review?(mconley) → review?(mkmelin+mozilla)
Comment 2•9 years ago
|
||
Comment on attachment 8573330 [details] [diff] [review] Proposed patch Review of attachment 8573330 [details] [diff] [review]: ----------------------------------------------------------------- Yes this seems to work. r=mkmelin ::: mailnews/addrbook/src/nsAbManager.cpp @@ +649,5 @@ > + > + // Add the extension and build a new localFile. > + fileName.AppendLiteral(VCF_FILE_EXTENSION); > + localFile->SetLeafName(fileName); > + } this brace looks to need a lot of more indention
Attachment #8573330 -
Flags: review?(mkmelin+mozilla) → review+
Assignee | ||
Comment 3•9 years ago
|
||
Pushed comm-central changeset 0dbb2dcc87e5 with whitespace fixes.
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 39.0
You need to log in
before you can comment on or make changes to this bug.
Description
•