[Kde-pim] Review Request: Move name member from Akonadi::ResourceBase up to Akonadi::Agentbase

Sascha Peilicke sasch.pe at gmx.de
Thu Jun 18 12:47:38 BST 2009


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

Review request for KDE PIM.


Summary
-------

Currently only resources can have a name but there's no reason for agents to not have one too. This patch moves the corresponding member/methods/signals up the Akonadi::AgentBase class.

The name is mainly useful to distinguish multiple agents of the same type. Altough this would be possible also via their identifier, the latter lacks some sort of expressiveness (from an user's perspective) and thus does not look too nice.


Diffs
-----

  trunk/KDE/kdepimlibs/akonadi/agentbase.h 983389 
  trunk/KDE/kdepimlibs/akonadi/agentbase.cpp 983389 
  trunk/KDE/kdepimlibs/akonadi/agentbase_p.h 983389 
  trunk/KDE/kdepimlibs/akonadi/agentinstance.cpp 983389 
  trunk/KDE/kdepimlibs/akonadi/resourcebase.h 983389 
  trunk/KDE/kdepimlibs/akonadi/resourcebase.cpp 983389 

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


Testing
-------

successfully compiled kdepimlibs and kdepim, restarted akonadi server, added/removed agents and resources, set the name of an agent.

no problems so far


Thanks,

Sascha

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



More information about the kde-pim mailing list