Merge lp:~michael-sheldon/webbrowser-app/fix-vcard-mime-mapping into lp:webbrowser-app

Proposed by Michael Sheldon
Status: Merged
Approved by: Olivier Tilloy
Approved revision: 1188
Merged at revision: 1206
Proposed branch: lp:~michael-sheldon/webbrowser-app/fix-vcard-mime-mapping
Merge into: lp:webbrowser-app
Diff against target: 13 lines (+2/-1)
1 file modified
src/app/MimeTypeMapper.js (+2/-1)
To merge this branch: bzr merge lp:~michael-sheldon/webbrowser-app/fix-vcard-mime-mapping
Reviewer Review Type Date Requested Status
PS Jenkins bot continuous-integration Needs Fixing
Olivier Tilloy Approve
Review via email: mp+272119@code.launchpad.net

Commit message

Add support for alternative mimetype header for vcards

Description of the change

Add support for alternative mimetype header for vcards

To post a comment you must log in.
Revision history for this message
Olivier Tilloy (osomon) wrote :

Looks good to me.

review: Approve
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Needs Fixing (continuous-integration)

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'src/app/MimeTypeMapper.js'
2--- src/app/MimeTypeMapper.js 2014-11-11 15:07:46 +0000
3+++ src/app/MimeTypeMapper.js 2015-09-23 14:32:47 +0000
4@@ -24,7 +24,8 @@
5 return ContentType.Music;
6 } else if(mimeType.search("video/") === 0) {
7 return ContentType.Videos;
8- } else if(mimeType.search("text/x-vcard") === 0) {
9+ } else if(mimeType.search("text/x-vcard") === 0
10+ || mimeType.search("text/vcard") === 0) {
11 return ContentType.Contacts;
12 } else if(mimeType.search("application/epub[+]zip") === 0
13 || mimeType.search("application/vnd\.amazon\.ebook") === 0

Subscribers

People subscribed via source and target branches

to status/vote changes: