two "block" menu items on gadu-gadu contacts

[liviopl]

it seems that pidgin supports gadu-gadu blocking ability and also his own, which doesn't notify the contact (ignores his messages).

this is confusing. pidgin's included blocking functionality should be unavailable for gadu-gadu contacts.

[liviopl]

screenshot

[deryni]

I don't think I understand the difference between the two block methods, can you explain the difference? Does Gadu Gadu provide two different types of blocking or just one? If it only provides one then the gg plugin shouldn't provide its own block menu item and should just use the normal privacy system in pidgin.

[liviopl]

Protocol provides blocking method, and Pidgin provides own.

If it only provides one then the gg plugin shouldn't provide its own block menu item and should just use the normal privacy system in pidgin.

No, it shouldn't use Pidgin's method.

Protocol notifies blocked contact that it is blocked, while Pidgin's method only ignores the messages.

Pidgin's method shouldn't be available for Gadu-Gadu contacts.

[deryni]

pidgin only provides internal privacy support when the protocol doesn't implement its own (a design decision I fought when it was made, I don't think pidgin should do that) the reason the pidgin method is available for gg is because the gg plugin doesn't tell libpurple that it supports blocking buddies, if it did that then pidgin would use that method and not the internal one. So this is a gg bug and as such I am assigning it to Bartosz.

[bernmeister]

Is this still happening in Pidgin 2.5.8?

[deryni]

Yes, this should still be the case in 2.5.8 and I still believe it to be an issue with the gg prpl. There is actually a comment in the gg code indicating that the code which provides the GG specific block/unblock menu items will need to go at some point. It doesn't look like making the change would be all that difficult, but I don't know enough about the features that the GG protocol provides to know what exact changes need to be made. Also, given the privacy rewrite SoC project I'm not sure how much this wants attention at the moment.

It would be handy if someone who uses the official GG client and/or knows the GG protocol could explain the full feature set of GG blocking though.

[rekkanoryo]

kkszysiu, any chance you might be interested in taking this on?

[deryni]

Like I said originally, it looked to me like this should be a trivial change to make to the code. I just don't know GG at all so wasn't comfortable doing it myself. The main question is how to map the GG block stuff to the current privacy api (and whether there is privacy status stuff that needs handling, that is the "allow all", "allow below", etc. choices pidgin shows). Assuming the simple case I would think it should be enough to simply add them to the deny list, which would require exposing ggp_menu_block (modified appropriately) as the add_deny and rem_deny PrplInfo? callbacks.

[tomkiewicz@…]

(In [d225b7ae359aa9448f679e2386a1f75326487b2e]):
Merged two buddy blocking methods in Gadu-Gadu protocol. Fixes #5303

[kkszysiu]

I see that you're the one of Pidgin developers. Congratz! :)

[tomkiewicz]

Just CPW with commit and trac permissions ;).