----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/103628/ -----------------------------------------------------------
(Updated Feb. 26, 2013, 9:05 a.m.) Review request for Telepathy. Changes ------- Finally update this review request! The default display name is now generated on demand only when needed, and not automatically when the fields are updated. This simplifies a lot the code, and avoid several slot calls. Description ------- This patch lets each plugin generate a "default" display name, based on the parameters set. - Default name is used if display name is empty or if it is the previous default name. - If the current display name _contains_ the old default display name, only the default part is replaced. For example, if old default name is f...@bar.com and the display name is Foo <f...@bar.com> when the user changes something and the new default display name is f...@buz.com, the new display name becomes Foo <f...@buz.com> (This happens realtime while the user is typing) - Otherwise the user set display name is just left unchanged. Also all the plugins were updated. For accounts without a specific plugin, the display name is set automatically using the "account" parameter if it exists. If it does not exist the user will have to add it manually, but this doesn't seem a big issue to me... Branch here: http://quickgit.kde.org/?p=clones%2Ftelepathy-accounts-kcm%2Fddomenichelli%2Ftelepathy-accounts-kcm.git&a=shortlog&h=refs/heads/displayname This addresses bug 284930. http://bugs.kde.org/show_bug.cgi?id=284930 Diffs (updated) ----- plugins/salut/salut-main-options-widget.h ab51e1901340ab8630ba19e50b49c77eeb5ecd2d plugins/rakia/rakia-main-options-widget.cpp ecb4ee4ef4b23869b3f73077e49a5bd3ed7f035e plugins/rakia/rakia-main-options-widget.h bd1291b8318dbc468b005a54315a9365aaf35b0c plugins/idle/main-options-widget.cpp f8aeb4b7bb777cc9b5973271344d1ebaac78aa52 plugins/idle/main-options-widget.h bf50f3059ca4f9fb267079d71b777b666be32a40 plugins/haze/yahoo-main-options-widget.cpp d8bf7e132081ae1debad319a0cee6d3e1a218db0 plugins/haze/yahoo-main-options-widget.h 38ddb1a738bed52ba5b8b4f1c06de7924aef78d5 plugins/haze/skype-main-options-widget.cpp 87a3448601b2f338ca9d7361424b7a926c2b14ee plugins/haze/skype-main-options-widget.h e081243138497181336c5a9faa133973a3054490 plugins/haze/myspaceim-main-options-widget.cpp 94e475de6f89d5095bbcc9798b144600042e87cb plugins/haze/myspaceim-main-options-widget.h 6c6de7f29795ee688666c7e47b2d1154f1df167a plugins/haze/msn-main-options-widget.cpp 90d719e9c19f514040a01a311420e80eb17a926f plugins/haze/msn-main-options-widget.h 8dea406a74ae64286824986f03b50e789e49f0af plugins/haze/icq-main-options-widget.cpp b6dcbae0d419c732d9803b02410890774a683708 plugins/haze/icq-main-options-widget.h bef946cec890b649ee96e544293e30e85b2197a6 plugins/haze/aim-main-options-widget.cpp 99068e21e0ebb0832d8d790bd2a1a1eca3addca3 plugins/haze/aim-main-options-widget.h b966fb6ea06378a3e01e82ab4bd56f645036e116 plugins/gabble/main-options-widget.cpp 927bb32f1e29a7e4e7dbed64ed4eb7df56e96014 plugins/gabble/main-options-widget.h 46c071862a7a17b784cb17e453f023802f2c0da5 plugins/gabble/main-options-widget-msn.cpp 913a808810b2368850657bc4f76edfa4d7d1f681 plugins/gabble/main-options-widget-msn.h 6d70cbeb171c7c10161e54a932bc1c499ea8d345 plugins/gabble/main-options-widget-kde-talk.cpp bd60004436699cb88b8766a8ad53521598545601 plugins/gabble/main-options-widget-kde-talk.h 58673ccbd51791195e125b4e1da5ab5c0f28bd5a plugins/gabble/main-options-widget-googletalk.cpp ed97bffb0dab256af5ab2a17ad1279094e49092b plugins/gabble/main-options-widget-googletalk.h b73df1129648dedb084af2d1f8ee056d06fa6bd4 plugins/gabble/main-options-widget-facebook.cpp 7e800bef0c6a1bf99bbfea84d3cb411c0ebac280 plugins/gabble/main-options-widget-facebook.h abeacc5c5b9a387d1e68151ea835bff306e5bc96 plugins/butterfly/main-options-widget.cpp 7e4800717673b6ee7d549bae7afbf3126e0a28d4 plugins/butterfly/main-options-widget.h 19a44c818544aa747718505b4e3db158cda79f4b plugins/sunshine/sunshine-main-options-widget.cpp f6beaec2f60535abf00f3c86b562a547173477ab plugins/sunshine/sunshine-main-options-widget.h fbb6dca508d9bf7b313c4b14dcd1c2097cc772d5 plugins/salut/salut-main-options-widget.cpp ff0339a29a5d2d79fd967a3eddbd9b5fd6fb97d3 src/add-account-assistant.cpp 07eae584091e7e1a47ca7c63b1652d5ba545ae94 src/edit-account-dialog.h afa13b804850bb5ab30603ade542566d3455e428 src/KCMTelepathyAccounts/parameter-edit-widget.cpp fc666c5c760ae31672fbefc332217ff68a7cd633 src/KCMTelepathyAccounts/parameter-edit-widget.h 0e92bb7df35870778b339fb83e2ddb232e9abb6f src/KCMTelepathyAccounts/account-edit-widget.cpp 312a4c913394c39e1b131994fbc0a8d02b84d5df src/KCMTelepathyAccounts/account-edit-widget.h 5db8119dfae2a31e51fca60daf40b6b083359f6f src/KCMTelepathyAccounts/abstract-account-parameters-widget.h 0c6cd95db610b7cebebe3e06ca15b446fd7d024d src/KCMTelepathyAccounts/abstract-account-parameters-widget.cpp c2b9a6788eed7a1f5bea9b55e51d735eaf8e8d94 src/salut-details-dialog.cpp f95826dcaa7b1fda96a695bd5f23ba9a3d4ffa63 src/salut-details-dialog.h 804d61b903bdf8c0946b1960aedac5b63b5b1b97 src/edit-account-dialog.cpp 8b9ae32b5844668455a44d8e04c01940e7f8fb41 src/salut-enabler.h 7d4d640d19509024b9da269cb061ba43d2896dd9 src/salut-enabler.cpp b16e57f27a736404b7f9afa11439f01e36eab484 Diff: http://git.reviewboard.kde.org/r/103628/diff/ Testing ------- Created an account and modified the display name. Edited an account and modified the display name. Created salut account and edited using the dialog. Edited salut account. Tested most of the plugins. More random tests. Thanks, Daniele E. Domenichelli
_______________________________________________ KDE-Telepathy mailing list KDE-Telepathy@kde.org https://mail.kde.org/mailman/listinfo/kde-telepathy
