D8353: Resolve body part URL handlers based on part mimetype

[Volker Krause]

vkrause added a comment.


  In https://phabricator.kde.org/D8353#156919, @knauss wrote:
  
  > In https://phabricator.kde.org/D8353#156898, @vkrause wrote:
  >
  > > This makes sense and looks good to me. I still fail to understand why this worked before though, I just moved this code to a different location :)
  > >  Mapping this to the render plugin that was used is tricky, as we do not persist that information, nor will it always be a single plugin responsible for drawing one part, so I think the mimetype approach works well for this.
  >
  >
  > Wait we know excatly, that a plugin was used for one email, when DefaultRendererPrivate::renderWithFactory retruns true. So adding afterwarts the loading for URLHandler here would result in the correct URLHandlers loaded. This was done before too.
  
  
  It's more complicated than that I'm afraid, some parts delegate to others bypassing DRP, and it is possible to have multiple plugins render content (although we don't do that atm). The use-case I have in mind is the booking information extractor rendering a summary banner before to the message content. So I think we need to tie this to the link URL, not the part.
  
  Independent of that, where is the code that did this before? I'm still wondering how I missed that.

REPOSITORY
  R94 PIM: Message Library

REVISION DETAIL
  https://phabricator.kde.org/D8353

To: dvratil, vkrause, knauss
Cc: #kde_pim, dvasin, winterz, vkrause, mlaurent, knauss, dvratil
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-pim/attachments/20171018/d575300f/attachment.html>