[Bug 27167] Gabble MailNotification.RequestInboxURL returns empty string

[bugzilla-daemon at freedesktop.org]

http://bugs.freedesktop.org/show_bug.cgi?id=27167





--- Comment #2 from Nicolas Dufresne <nicolas.dufresne at collabora.co.uk>  2010-03-18 12:29:50 PST ---
(In reply to comment #1)
> I'd prefer it if this was what happened: I think it should be valid to call
> Subscribe and immediately follow up with RequestInboxURL.

Both solution makes it valid, that was the bug I was trying to fix ;)

> 
> (It's clearly wrong to call RequestMailURL before you have any mails, though
> :-)

Well no, it's right since you want have access to the web interface at anytime,
not only when count >= 0.

> I consider the error conditions in the spec to be advisory, rather than
> exhaustive: any D-Bus method call can fail at any time for any reason. If you
> think it's useful for Subscribe to wait for initial setup where necessary, I'd
> welcome a spec patch to say so.

It's clear that the best solution is to allow all method to be aliased (as long
as Subscribe is called first as stated by the spec). This would give more
flexibility on the API client side.

The only thing I can't stand is having to add more non-public crap into
connection.h. What I would propose is to create a precedent, where an extension
would define a private pointer to it's private data using an opaque structure.
This way, you don't have to change the connection every-time a new private
member must be added. Please let me know if you agree or not, so I can start
the rewrite of this patch.


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