Review Request: Further tidying up with the parameter model change

David Edmundson kde at davidedmundson.co.uk
Sat Jan 22 14:54:51 CET 2011


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

(Updated Jan. 22, 2011, 1:54 p.m.)


Review request for Telepathy.


Changes
-------

Updated with review comment incorporated and a fix for the "press enter bug"


Summary
-------

Add getters for set/unset parameters to the ParameterModel.
Also deprecated the class "ProtocolParameterValue" as it was basically the same as Parameter-item.
    
This removes the risk of the parameter model and the list of parameter values getting out of sync, and makes the interface to the plugins a lot clearer wrt to saving.


Diffs (updated)
-----

  src/KCMTelepathyAccounts/CMakeLists.txt a74ae27c89e1ad9e0ad6948bf959f4c93bd1ab7c 
  src/KCMTelepathyAccounts/abstract-account-parameters-widget.h 6357b4eaff3b5579b44d9040ffe33314bbf09d80 
  src/KCMTelepathyAccounts/abstract-account-parameters-widget.cpp d0ad71a3660ed2514817a67e6fd2a283481b54d9 
  src/KCMTelepathyAccounts/abstract-account-ui.h a90395ca4e429fa37ccf935f19fcd0063397d4e9 
  src/KCMTelepathyAccounts/account-edit-widget.h 4c78338603dc0b88230b52190ca9d373dff2628d 
  src/KCMTelepathyAccounts/account-edit-widget.cpp b554db71a17a4c9664d04f781561ce62cbc17562 
  src/KCMTelepathyAccounts/generic-advanced-options-widget.h PRE-CREATION 
  src/KCMTelepathyAccounts/generic-advanced-options-widget.cpp PRE-CREATION 
  src/KCMTelepathyAccounts/parameter-edit-model.h 0c762ea5a30253cd1c4b772f3859b681f4fa8dd4 
  src/KCMTelepathyAccounts/parameter-edit-model.cpp 0fe52a10798b03dff5491d78c1b16ad72e2f0533 
  src/KCMTelepathyAccounts/profile-select-widget.cpp PRE-CREATION 
  src/add-account-assistant.cpp 3933763957b1448aacf9e6c7efbc6b296a5e3664 
  src/edit-account-dialog.cpp 73462f2e2a93e22e6d5d67297c575f4e10d44c5c 

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


Testing
-------

Added and edited an account. Seems to still work.


Thanks,

David

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


More information about the KDE-Telepathy mailing list