[Kde-pim] Review Request: Convert IMAP flags to internal Akonadi flags in the IMAP resource.

[Leo Franchi]

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

(Updated 2010-07-27 14:01:21.844837)


Review request for KDE PIM and Kevin Ottens.


Changes
-------

OK, changes made as a result of Kevin's comments. Thanks!


Summary
-------

As a result of the recent Akonadi server changes that have made the akonadi server case-sensitive for flags, the decision was made to store flags as all-uppercase. However, the flags from imap servers are not upper-cased, they are simply capitalized. This results in collection statistics always being incorrect (listing all items as unread).

After discussion with Tobias, a solution was proposed to do a mapping of KImap flags <--> Akonadi flags in the imap resource, as that is the interface between KImap and Akonadi. For now this just includes 4 standard imap server flags. I don't know if this is the best way to do this, but figured i'd throw it out there and start a discussion.

 Comments are appreciated :)


Diffs (updated)
-----

  /branches/KDE/4.5/kdepim/runtime/resources/imap/CMakeLists.txt 1155485 
  /branches/KDE/4.5/kdepim/runtime/resources/imap/imapflags.h PRE-CREATION 
  /branches/KDE/4.5/kdepim/runtime/resources/imap/imapflags.cpp PRE-CREATION 
  /branches/KDE/4.5/kdepim/runtime/resources/imap/imapresource.h 1155485 
  /branches/KDE/4.5/kdepim/runtime/resources/imap/imapresource.cpp 1155485 
  /branches/KDE/4.5/kdepimlibs/akonadi/kmime/CMakeLists.txt 1155485 
  /branches/KDE/4.5/kdepimlibs/akonadi/kmime/messageflags.h PRE-CREATION 
  /branches/KDE/4.5/kdepimlibs/akonadi/kmime/messageflags.cpp PRE-CREATION 

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


Testing
-------


Thanks,

Leo

_______________________________________________
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/