Port TNEF plugin to new rendering plugin API
ClosedPublic
Actions

Authored by vkrause on Oct 7 2017, 10:17 AM.

Details

Reviewers

knauss

Commits

Summary

By using the standard inline attachment container we also get the proper
attachment links now.

Repository

R81 KDE PIM Addons

Lint

Automatic diff as part of commit; lint not applicable.

Unit

Automatic diff as part of commit; unit tests not applicable.

Restricted Application added a subscriber: KDE PIM. · View Herald Transcript

plugins/messageviewer/bodypartformatter/autotests/data/tnef-one-file.mbox.html
16	we need the dir setting to support right-to-left languages

plugins/messageviewer/bodypartformatter/autotests/data/tnef-one-file.mbox.html
16	this is broken in the generic attachment container already then, no?

plugins/messageviewer/bodypartformatter/autotests/data/tnef-one-file.mbox.html
16	looks like this. But I never tested right-to-left language, so i don't know.

needs testing with r-t-l language, but anyways that it not part of this patch

This revision is now accepted and ready to land.Oct 10 2017, 10:18 PM

This revision was automatically updated to reflect the committed changes.

			Path	Packages
M			plugins/messageviewer/bodypartformatter/autotests/data/tnef-one-file.mbox.html (29 lines)
M			plugins/messageviewer/bodypartformatter/autotests/data/tnef-two-files.mbox.html (37 lines)
M			plugins/messageviewer/bodypartformatter/ms-tnef/CMakeLists.txt (2 lines)
M			plugins/messageviewer/bodypartformatter/ms-tnef/application_ms-tnef.cpp (171 lines)
M			plugins/messageviewer/bodypartformatter/ms-tnef/application_ms-tnef.json (6 lines)