<table><tr><td style="">vkrause added inline comments.
</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/D8175" rel="noreferrer">View Revision</a></tr></table><br /><div><strong>INLINE COMMENTS</strong><div><div style="margin: 6px 0 12px 0;"><div style="border: 1px solid #C7CCD9; border-radius: 3px;"><div style="padding: 0; background: #F7F7F7; border-color: #e3e4e8; border-style: solid; border-width: 0 0 1px 0; margin: 0;"><div style="color: #74777d; background: #eff2f4; padding: 6px 8px; overflow: hidden;"><a style="float: right; text-decoration: none;" href="https://phabricator.kde.org/D8175#inline-34424" rel="noreferrer">View Inline</a><span style="color: #4b4d51; font-weight: bold;">knauss</span> wrote in <span style="color: #4b4d51; font-weight: bold;">bodypartformatterfactory.cpp:156</span></div>
<div style="margin: 8px 0; padding: 0 12px; color: #74777D;"><p style="padding: 0; margin: 8px;">no the idea is that in mimetreeparser there is no plugin loader, this should be in messageviewer. Because the plugins  may be specific for a specific messageviewer, that's why the messageviewer needs to control this. For Kube as example the plugins would need different plugins, becuase tithey support a different set. That was the base idea of this BaseFactory to give full control ovr the plugins to the messageviewer.</p></div></div>
<div style="margin: 8px 0; padding: 0 12px;"><p style="padding: 0; margin: 8px;">The solution for that is adding a way to change the plugin search path (or add some other way of filtering the plugins by additional metadata), not to duplicate the entire plugin loading code. This becomes even more relevant once we add on-demand plugin loading, when this wont be a single function anymore.<br />
Your approach would also mean we would never be able to use MTP stand-alone without MV, but I think e.g. our composer would actually only need MTP, not the full MV.</p></div></div></div></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/D8175" rel="noreferrer">https://phabricator.kde.org/D8175</a></div></div><br /><div><strong>To: </strong>vkrause, knauss<br /><strong>Cc: </strong>KDE PIM, dvasin, winterz, vkrause, mlaurent, knauss, dvratil<br /></div>