[Bug 14540] Names interface - Aliasing replacement with separate nickname, local alias etc.

[bugzilla-daemon at freedesktop.org]

https://bugs.freedesktop.org/show_bug.cgi?id=14540

--- Comment #39 from Simon McVittie <simon.mcvittie at collabora.co.uk> ---
(In reply to comment #34)
> This is not valid D-Bus introspection. Spell it out in full here, please.
> The ofdT shortcut is only OK in elements or attributes in the tp: namespace,
> IMO.

I've patched this, I'll attach a patch.

> ofdT, not imt1.

Likewise.

> I think InvalidHandle checking in SetAliases should be the first thing
> that's done, and the side-effects should only be allowed to happen if
> every handle is valid. 

This too.

(In reply to comment #28)
> > Shouldn't set_nickname_async() be nullable? It looks as though it'd be pretty
> > easy.
> 
> We would need a way to tell we don't support setting user's nickname, a
> property similar to AliasStorage?

I think we should have that. Perhaps Names_Flags and
Names_Flag_Can_Set_Nickname, consistent with ContactInfo?

(We could even have a Pushes_Nicknames flag, just like ContactInfo. It'd be
false on XMPP, since PEP is the exception rather than the rule.)

-- 
You are receiving this mail because:
You are the QA Contact for the bug.