Remove usage of "ExternalPayload" switch

Review Request #101703 - Created June 20, 2011 and submitted

Information
Kevin Krammer
akonadi
master
275561
Reviewers
kdepim
It lead to payload parts not being handled correctly when configured to false. See bug 275561 for an example where changing to true solved with message copying problem and http://lists.kde.org/?t=130850247100005&r=1&w=2 for the discussion

  
Andras Mantia
Why remove and not fix the case when payload is stored in the database? Not that I'm not happy that it is not that external payload usage is broken, but the other way around. :)
  1. You mean option (2) here? http://lists.kde.org/?l=kde-pim&m=130850243326430&w=2
  2. The same behaviour can be achieved by setting the threshold to INT64_MAX, but as we saw, the actual limit imposed by the database is way lower than that anyway, so IMHO this is fine.
Volker Krause

   
Commit Hook
This review has been submitted with commit b5063615fe7d23f7cb9bea26e9feb9cc41aa0282 by Kevin Krammer to branch 1.5.
Commit Hook
This review has been submitted with commit defb52abc51ba60544257c91895fc29ff9af8dff by Kevin Krammer to branch master.
Loading...