Differential D20669

Normalize case of our extensions
ClosedPublic
Actions

Authored by vkrause on Apr 18 2019, 3:24 PM.

Details

Reviewers
mlaurent
Commits
R174:28da29cfff6e: Normalize case of our extensions
Summary

Some backends (like my Nextcloud server) seem to change the case, making
our code not detect those anymore after a roundtrip. So fix that during
importing a vCard.

Diff Detail

Repository
R174 KContacts
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
vkrause created this revision.Apr 18 2019, 3:24 PM
Restricted Application added a project: KDE PIM. · View Herald TranscriptApr 18 2019, 3:24 PM
Restricted Application added a subscriber: kde-pim. · View Herald Transcript
vkrause requested review of this revision.Apr 18 2019, 3:24 PM
mlaurent accepted this revision.Apr 26 2019, 3:09 PM
mlaurent added a subscriber: mlaurent.

Please add me as reviewer otherwise I can't see them I have a lot of emails :)
And I don't know why I don't see your review in my phabricator :)

This revision is now accepted and ready to land.Apr 26 2019, 3:09 PM
Closed by commit R174:28da29cfff6e: Normalize case of our extensions (authored by vkrause). · Explain WhyApr 27 2019, 8:26 AM
This revision was automatically updated to reflect the committed changes.

Showing Only Differences

This revision modifies 3 more files that are hidden because they were not modified between selected diffs and they have no inline comments.

Revision Contents
Changeset List

PathPackages
Msrc/vcardtool.cpp (18 lines)

Diff 57087

View Options

src/vcardtool.cpp

Loading...