<table><tr><td style="">knauss added a comment.
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D8071" rel="noreferrer">View Revision</a></tr></table><br /><div><div><p>The main idea is that the rendering process  should not get access to KMime::Content and all calls to this are going via the MessagePart. That is leaks into the renderer is because of getting a fast working solution.<br />
So the right way IMO is to move the getter into proctected and than every MP that needs to have it public mark the getter public or even provide these information the Render needs getFileName, contentDescription ...<br />
But maybe this makes it more completcated than easier.<br />
I know, that my aproch treats plugins autors to writer a propper MessagePart...</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R94 PIM: Message Library</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D8071" rel="noreferrer">https://phabricator.kde.org/D8071</a></div></div><br /><div><strong>To: </strong>vkrause, knauss<br /><strong>Cc: </strong>KDE PIM, dvasin, winterz, vkrause, mlaurent, knauss, dvratil<br /></div>