[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

Jonny Lamb <jonny.lamb at collabora.co.uk> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                URL|http://cgit.freedesktop.org |http://cgit.freedesktop.org
                   |/~smcv/telepathy-spec/log/? |/~jonny/telepathy-spec/log/
                   |h=names                     |?h=next-names

--- Comment #20 from Jonny Lamb <jonny.lamb at collabora.co.uk> 2012-04-10 15:11:56 PDT ---
I took your patch and updated it for next. I don't think there's any point in
looking at master at all for this.

(In reply to comment #16)
> In current spec technology this should be Connection.Interface.Names1

Done.

> I'm less convinced about the pretty ID, see below.

I removed pretty ID for now then.

> Not sure if there's much point in this method, we could just say "use
> GetContactAttributes". At the moment the contact attr is defined in terms of
> the method, but it needn't be.

Yeah I removed that and added a note.

(In reply to comment #18)
> @@ +199,5 @@
> > +
> > +    <tp:contact-attribute name="nickname" type="s">
> > +      <tp:docstring xmlns="http://www.w3.org/1999/xhtml">
> > +        <p>The same string that would be returned by
> > +          <tp:member-ref>GetNicknames</tp:member-ref>.
> 
> Should be updated if we drop GetNicknames()

Done.

(In reply to comment #19)
> (I'd be inclined to implement it as a TpNamesMixin which also implements
> Aliasing, then drop Aliasing in next.)

This sounds good but let's just do this straight on next.

> > I guess we can't do SetLocalAlias(self_handle) right? May be good to say it to
> > be clear.
> 
> Protocol-dependent. In XMPP, you can, but it's rarely useful (it would add you
> to your own roster with the given local alias, rather than setting your
> nickname).
> 
> It'd be worth mentioning that it's rarely useful and might not work.

I added a note about this.

Nothing else changed though; see my branch.

-- 
Configure bugmail: https://bugs.freedesktop.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA Contact for the bug.