[Kde-pim] Review Request: Apply template when switching identity in message composer

[Jonathan Armond]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://reviewboard.kde.org/r/833/
-----------------------------------------------------------

(Updated 2009-06-15 09:19:20.116278)


Review request for KDE PIM.


Changes
-------

Reply to list should use same template as Reply to all (according to GUI).
Revert back to not using template for Forward as Attachment (separate patch if required).


Summary
-------

Apply new template when switching identity in the message composer, as long the message has not been modified. NB: changing identity no longer marks message as modified.

Apply template to Forward As Attachment messages.

First attempt at fixing bug 143199.


This addresses bugs 143199, 179754 and 186843.
    https://bugs.kde.org/show_bug.cgi?id=143199
    https://bugs.kde.org/show_bug.cgi?id=179754
    https://bugs.kde.org/show_bug.cgi?id=186843


Diffs (updated)
-----

  /trunk/KDE/kdepim/kmail/composer.h 982275 
  /trunk/KDE/kdepim/kmail/kmcommands.cpp 982275 
  /trunk/KDE/kdepim/kmail/kmcomposewin.h 982275 
  /trunk/KDE/kdepim/kmail/kmcomposewin.cpp 982275 
  /trunk/KDE/kdepim/kmail/kmkernel.cpp 982275 
  /trunk/KDE/kdepim/kmail/kmmainwidget.cpp 982275 
  /trunk/KDE/kdepim/kmail/templateparser.h 982275 
  /trunk/KDE/kdepim/kmail/templateparser.cpp 982275 

Diff: http://reviewboard.kde.org/r/833/diff


Testing
-------

Template properly parsed for new, reply, reply all and forward. Switches between custom, global and no templates properly. Doesn't switch after modifying message.


Thanks,

Jonathan

_______________________________________________
KDE PIM mailing list kde-pim at kde.org
https://mail.kde.org/mailman/listinfo/kde-pim
KDE PIM home page at http://pim.kde.org/