Mercurial > pidgin
changeset 12641:e0e8b0735c5a
[gaim-migrate @ 14977]
A change from Bartosz Oler. I think this is intended to hide blocked as a status that one can set on themself. Anyway, this is all wrong, so I added some comments to that effect.
committer: Tailor Script <tailor@pidgin.im>
| author | Richard Laager <rlaager@wiktel.com> |
|---|---|
| date | Fri, 23 Dec 2005 12:02:01 +0000 |
| parents | de306144dc8a |
| children | 022bf81f4087 |
| files | src/protocols/gg/gg.c |
| diffstat | 1 files changed, 6 insertions(+), 2 deletions(-) [+] |
line wrap: on
line diff
--- a/src/protocols/gg/gg.c Fri Dec 23 06:48:27 2005 +0000 +++ b/src/protocols/gg/gg.c Fri Dec 23 12:02:01 2005 +0000 @@ -1452,9 +1452,10 @@ NULL); types = g_list_append(types, type); - /* Why is blocked a status??? -- rlaager */ + /* This status is wrong. It shouldn't exist. This prpl must not be + * using the privacy stuff correctly. -- rlaager */ type = gaim_status_type_new_with_attrs( - GAIM_STATUS_INVISIBLE, "blocked", _("Blocked"), TRUE, TRUE, FALSE, + GAIM_STATUS_INVISIBLE, "blocked", _("Blocked"), TRUE, FALSE, FALSE, "message", _("Message"), gaim_value_new(GAIM_TYPE_STRING), NULL); types = g_list_append(types, type); @@ -1475,6 +1476,9 @@ ggp_bmenu_add_to_chat, NULL, NULL); m = g_list_append(m, act); + /* Using a blist node boolean here is also wrong. + * Once the Block and Unblock actions are added to the core, + * this will have to go. -- rlaager */ if (gaim_blist_node_get_bool(node, "blocked")) { act = gaim_blist_node_action_new(_("Unblock"), ggp_bmenu_block, NULL, NULL);