Review Request: Changed plugin structure and added convenience methods

Dominik Schmidt ich at dominik-schmidt.de
Thu Jan 13 19:05:34 CET 2011


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

Review request for Telepathy.


Summary
-------

This got a little huge, I'm sorry.

This patch refactors the way plugins work.
It uses (kind of) the methods from the last review request but additionally uses the QDataWidgetMapper as suggested (and partly implemented) by Thomas - please tell me your mail address!

Also it avoids passing around ParameterLists and Values and instead passes the ParameterEditModel through all levels.


Diffs
-----

  src/KCMTelepathyAccounts/CMakeLists.txt 082f3352c4e5855d87b594620e7e317160b557c1 
  src/KCMTelepathyAccounts/abstract-account-parameters-widget.h 44bba673ee0479f7253e6c2ac0860888362d0642 
  src/KCMTelepathyAccounts/abstract-account-parameters-widget.cpp d0ad71a3660ed2514817a67e6fd2a283481b54d9 
  src/KCMTelepathyAccounts/abstract-account-ui.h a90395ca4e429fa37ccf935f19fcd0063397d4e9 
  src/KCMTelepathyAccounts/abstract-account-ui.cpp a2a09c0485e2eec5e64c96a91ddaddea2747e68d 
  src/KCMTelepathyAccounts/account-edit-widget.h 9e8ac689eb15d7666a3be06f98af5a67703efd6c 
  src/KCMTelepathyAccounts/account-edit-widget.cpp d78cf93fbd9bb905ef2b382e07b0ac780f2f9e3b 
  src/KCMTelepathyAccounts/generic-advanced-options-widget.h PRE-CREATION 
  src/KCMTelepathyAccounts/generic-advanced-options-widget.cpp PRE-CREATION 
  src/KCMTelepathyAccounts/include/GenericAdvancedOptionsWidget PRE-CREATION 
  src/KCMTelepathyAccounts/include/ParameterEditModel PRE-CREATION 
  src/KCMTelepathyAccounts/parameter-edit-model.h 4eac5a7512c8e0a1f90f7d570f69d0659b61c2b6 
  src/KCMTelepathyAccounts/parameter-edit-model.cpp a30481ef12ef11533aab26b3200c94eb50139f49 
  src/KCMTelepathyAccounts/parameter-edit-widget.h 566cbe1227adb7e0bd7ee4637f1263273f83333b 
  src/KCMTelepathyAccounts/parameter-edit-widget.cpp 7dd1e530d167a860564200351dea5ae3cd00f9a8 
  src/KCMTelepathyAccounts/parameter-item.cpp 8f907f7988b10d0e58f5beddf04ebedd54fdfb00 
  src/add-account-assistant.cpp 184cf93ef8c7b1df944c3fb54cde8f5b6749cec2 
  src/edit-account-dialog.cpp 45ec8407f943da77175d6f320fe13b92b75874f1 

Diff: http://git.reviewboard.kde.org/r/100376/diff


Testing
-------

Yes, works for me for adding and editing accounts.


Thanks,

Dominik

-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://mail.kde.org/pipermail/kde-telepathy/attachments/20110113/b8307775/attachment.htm 


More information about the KDE-Telepathy mailing list