Bug 33157

Summary:	Haze should implement the Protocol interface
Product:	Telepathy	Reporter:	Thomas Richard <thomas9999>
Component:	haze	Assignee:	Telepathy bugs list <telepathy-bugs>
Status:	RESOLVED FIXED	QA Contact:	Telepathy bugs list <telepathy-bugs>
Severity:	normal
Priority:	medium
Version:	unspecified
Hardware:	Other
OS:	All
Whiteboard:
i915 platform:		i915 features:

Description Thomas Richard 2011-01-15 08:21:46 UTC

When i call ProfileManager::create, i get this in my debug console:

tp-qt4 0.5.2.1 WARNING: Error parsing config file for connection manager "haze" - introspecting 
QDBusConnection: error: could not send message to service "org.freedesktop.Telepathy.ConnectionManager.haze" path "/org/freedesktop/Telepathy/ConnectionManager/haze/local-xmpp" interface "org.freedesktop.DBus.Properties" member "GetAll": Invalid object path: /org/freedesktop/Telepathy/ConnectionManager/haze/local-xmpp
tp-qt4 0.5.2.1 WARNING: Properties.GetAll(Protocol) failed: "org.freedesktop.DBus.Error.UnknownMethod": "Method "GetAll" with signature "s" on interface "org.freedesktop.DBus.Properties" doesn't exist
"
tp-qt4 0.5.2.1 WARNING:   Full functionality requires CM support for the Protocol interface 
tp-qt4 0.5.2.1 WARNING: Properties.GetAll(Protocol) failed: "org.freedesktop.DBus.Error.UnknownMethod": "Method "GetAll" with signature "s" on interface "org.freedesktop.DBus.Properties" doesn't exist
"
tp-qt4 0.5.2.1 WARNING:   Full functionality requires CM support for the Protocol interface 
tp-qt4 0.5.2.1 WARNING: Properties.GetAll(Protocol) failed: "org.freedesktop.DBus.Error.UnknownMethod": "Method "GetAll" with signature "s" on interface "org.freedesktop.DBus.Properties" doesn't exist
"
tp-qt4 0.5.2.1 WARNING:   Full functionality requires CM support for the Protocol interface 
tp-qt4 0.5.2.1 WARNING: Properties.GetAll(Protocol) failed: "org.freedesktop.DBus.Error.UnknownMethod": "Method "GetAll" with signature "s" on interface "org.freedesktop.DBus.Properties" doesn't exist
"
tp-qt4 0.5.2.1 WARNING:   Full functionality requires CM support for the Protocol interface 
tp-qt4 0.5.2.1 WARNING: Properties.GetAll(Protocol) failed: "org.freedesktop.DBus.Error.UnknownMethod": "Method "GetAll" with signature "s" on interface "org.freedesktop.DBus.Properties" doesn't exist
"
tp-qt4 0.5.2.1 WARNING:   Full functionality requires CM support for the Protocol interface 
tp-qt4 0.5.2.1 WARNING: Properties.GetAll(Protocol) failed: "org.freedesktop.DBus.Error.UnknownMethod": "Method "GetAll" with signature "s" on interface "org.freedesktop.DBus.Properties" doesn't exist
"
tp-qt4 0.5.2.1 WARNING:   Full functionality requires CM support for the Protocol interface 
tp-qt4 0.5.2.1 WARNING: Properties.GetAll(Protocol) failed: "org.freedesktop.DBus.Error.UnknownMethod": "Method "GetAll" with signature "s" on interface "org.freedesktop.DBus.Properties" doesn't exist
"
tp-qt4 0.5.2.1 WARNING:   Full functionality requires CM support for the Protocol interface 
tp-qt4 0.5.2.1 WARNING: Properties.GetAll(Protocol) failed: "org.freedesktop.DBus.Error.UnknownMethod": "Method "GetAll" with signature "s" on interface "org.freedesktop.DBus.Properties" doesn't exist
"
tp-qt4 0.5.2.1 WARNING:   Full functionality requires CM support for the Protocol interface 
tp-qt4 0.5.2.1 WARNING: Properties.GetAll(Protocol) failed: "org.freedesktop.DBus.Error.UnknownMethod": "Method "GetAll" with signature "s" on interface "org.freedesktop.DBus.Properties" doesn't exist
"
tp-qt4 0.5.2.1 WARNING:   Full functionality requires CM support for the Protocol interface 
tp-qt4 0.5.2.1 WARNING: Properties.GetAll(Protocol) failed: "org.freedesktop.DBus.Error.UnknownMethod": "Method "GetAll" with signature "s" on interface "org.freedesktop.DBus.Properties" doesn't exist
"
tp-qt4 0.5.2.1 WARNING:   Full functionality requires CM support for the Protocol interface 
tp-qt4 0.5.2.1 WARNING: Properties.GetAll(Protocol) failed: "org.freedesktop.DBus.Error.UnknownMethod": "Method "GetAll" with signature "s" on interface "org.freedesktop.DBus.Properties" doesn't exist
"
tp-qt4 0.5.2.1 WARNING:   Full functionality requires CM support for the Protocol interface 
tp-qt4 0.5.2.1 WARNING: Properties.GetAll(Protocol) failed: "org.freedesktop.DBus.Error.UnknownMethod": "Method "GetAll" with signature "s" on interface "org.freedesktop.DBus.Properties" doesn't exist
"
tp-qt4 0.5.2.1 WARNING:   Full functionality requires CM support for the Protocol interface 
tp-qt4 0.5.2.1 WARNING: Properties.GetAll(Protocol) failed: "org.freedesktop.DBus.Error.UnknownMethod": "Method "GetAll" with signature "s" on interface "org.freedesktop.DBus.Properties" doesn't exist
"
tp-qt4 0.5.2.1 WARNING:   Full functionality requires CM support for the Protocol interface 
tp-qt4 0.5.2.1 WARNING: Properties.GetAll(Protocol) failed: "org.freedesktop.DBus.Error.UnknownMethod": "Method "GetAll" with signature "s" on interface "org.freedesktop.DBus.Properties" doesn't exist
"
tp-qt4 0.5.2.1 WARNING:   Full functionality requires CM support for the Protocol interface 
tp-qt4 0.5.2.1 WARNING: Properties.GetAll(Protocol) failed: "com.trolltech.QtDBus.Error.InvalidObjectPath": "Invalid object path: /org/freedesktop/Telepathy/ConnectionManager/haze/local-xmpp"
tp-qt4 0.5.2.1 WARNING:   Full functionality requires CM support for the Protocol interface 
tp-qt4 0.5.2.1 WARNING: Properties.GetAll(Protocol) failed: "org.freedesktop.DBus.Error.UnknownMethod": "Method "GetAll" with signature "s" on interface "org.freedesktop.DBus.Properties" doesn't exist
"
tp-qt4 0.5.2.1 WARNING:   Full functionality requires CM support for the Protocol interface 
tp-qt4 0.5.2.1 WARNING: Properties.GetAll(Protocol) failed: "org.freedesktop.DBus.Error.UnknownMethod": "Method "GetAll" with signature "s" on interface "org.freedesktop.DBus.Properties" doesn't exist
"
tp-qt4 0.5.2.1 WARNING:   Full functionality requires CM support for the Protocol interface

ConnectionManager::interfaces() returns an empty list for Haze (but also does for other cm's)

kDebug() << "Interfaces for" << d->connectionManager->name();
kDebug() << d->connectionManager->interfaces();

returns:
kcmshell(27840) AddAccountAssistant::pageTwo: Interfaces for "haze"
kcmshell(27840) AddAccountAssistant::pageTwo: ()

kcmshell(27840) AddAccountAssistant::pageTwo: Interfaces for "gabble"
kcmshell(27840) AddAccountAssistant::pageTwo: ()

Comment 1 Olli Salli 2011-04-04 02:09:37 UTC

The bug about incorrectly building the object path for protocols with the '-' character was fixed in tp-qt4 by this commit:

commit 62558b649efc3f2ae504ceda6edc98bb67953446
Author: Andre Moreira Magalhaes (andrunko) <andre.magalhaes@collabora.co.uk>
Date:   Wed Jan 26 12:32:11 2011 -0200

    ConnectionManager: Properly escape protocol name with "-".

The above fix is present in tp-qt4 >= 0.5.6.

ConnectionManager::interfaces() being empty is fine. The only CM interface currently in existence is the AccountStorage draft interface, which is, as far as I know not implemented by any public CM.

As for the warning about missing CM support, the warning in tp-qt4 is descriptive, and stems from the fact that haze doesn't implement the Protocol D-Bus interface (or didn't by the time you reported). We could of course not print anything, but then you'd be left in the dark as to why a lot of the functionality offered by Tp::ConnectionManager, and therefore indirectly by Tp::ProfileManager and Tp::Account seems to work incorrectly.

Reassigning to haze. If it does, in fact, already support the Protocol interface, feel free to ALREADYFIXED this. In this case, reporter: just upgrading your haze should cut it.

Comment 2 Simon McVittie 2013-09-16 18:13:17 UTC

> Reassigning to haze. If it does, in fact, already support the Protocol
> interface, feel free to ALREADYFIXED this.

It does, since 0.5.0.

Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.