[SCM] ktp-contact-list packaging branch, master, updated. debian/15.12.1-2-1070-g6c56f91

Maximiliano Curia maxy at moszumanska.debian.org
Sat May 28 00:13:45 UTC 2016


Gitweb-URL: http://git.debian.org/?p=pkg-kde/applications/ktp-contact-list.git;a=commitdiff;h=427cb59

The following commit has been merged in the master branch:
commit 427cb59dbd34df616c7ebfa9e1b233206d484acb
Author: David Edmundson <kde at davidedmundson.co.uk>
Date:   Fri Aug 9 14:14:34 2013 +0100

    Remove hack to work around connections with no RosterGroups
    
    This is now handled inside GlobalContactManager which upgrades
    connections to have roster/rostergroups() on demand
    
    This also theoretically makes AccountManager loading faster so the
    current presence will be updated quicker
    
    REVIEW: 111966
---
 main-widget.cpp | 50 +-------------------------------------------------
 1 file changed, 1 insertion(+), 49 deletions(-)

diff --git a/main-widget.cpp b/main-widget.cpp
index b9129fa..88cc868 100644
--- a/main-widget.cpp
+++ b/main-widget.cpp
@@ -66,52 +66,6 @@
 #include "tooltips/tooltipmanager.h"
 #include "context-menu.h"
 
-/** Start of bodge to work around https://bugs.freedesktop.org/show_bug.cgi?id=57739
-    Due to a bug in TelepathyQt if we request a feature that the connection does not support, it simply fails.
-
-    Local-XMPP does not support grouping, so salut the local-xmpp spec does not either.
-    When we request the ContactRosterGroups feature on all connections, salut fails and therefore doesn't show any contacts.
-
-    Fetching ContactRosterGroups seperately is not a viable option as contacts would move after loading
-
-    In this hack we make a new ConnectionFactory that fetches ContactRosterGroups on all connections _Except_ salut
-    by overriding the featuresFor method which determines which features should be added to a given DBus proxy, in this case a connection.
-
-    When https://bugs.freedesktop.org/show_bug.cgi?id=57739 is fixes all this code should be removed and we should create a standard Tp::ConnectionFactory
- */
-namespace KTp {
-    class ConnectionFactory : Tp::ConnectionFactory {
-    public:
-        static Tp::ConnectionFactoryPtr create(const QDBusConnection &bus, const Tp::Features &features=Tp::Features());
-    protected:
-        ConnectionFactory(const QDBusConnection &bus, const Tp::Features& features);
-        virtual Tp::Features featuresFor(const Tp::DBusProxyPtr &proxy) const;
-    };
-}
-
-Tp::ConnectionFactoryPtr KTp::ConnectionFactory::create(const QDBusConnection &bus, const Tp::Features &features)
-{
-    return Tp::ConnectionFactoryPtr(new KTp::ConnectionFactory(bus, features));
-}
-
-KTp::ConnectionFactory::ConnectionFactory(const QDBusConnection &bus, const Tp::Features &features): Tp::ConnectionFactory(bus, features)
-{
-}
-
-Tp::Features KTp::ConnectionFactory::featuresFor(const Tp::DBusProxyPtr &proxy) const
-{
-    Tp::Features features = Tp::FixedFeatureFactory::featuresFor(proxy);
-
-    Tp::ConnectionPtr cm = Tp::ConnectionPtr::qObjectCast<>(proxy);
-    if (cm && cm->cmName() == QLatin1String("salut")) {
-        features.remove(Tp::Connection::FeatureRosterGroups);
-    }
-    return features;
-}
-/** End of bodge*/
-
-
-
 bool kde_tp_filter_contacts_by_publication_status(const Tp::ContactPtr &contact)
 {
     return contact->publishState() == Tp::Contact::PresenceStateAsk;
@@ -515,10 +469,8 @@ void MainWidget::setupTelepathy()
                                                                        << Tp::Account::FeatureProtocolInfo
                                                                        << Tp::Account::FeatureProfile);
 
-    Tp::ConnectionFactoryPtr connectionFactory = KTp::ConnectionFactory::create(QDBusConnection::sessionBus(),
+    Tp::ConnectionFactoryPtr connectionFactory = Tp::ConnectionFactory::create(QDBusConnection::sessionBus(),
                                                                                Tp::Features() << Tp::Connection::FeatureCore
-                                                                               << Tp::Connection::FeatureRosterGroups
-                                                                               << Tp::Connection::FeatureRoster
                                                                                << Tp::Connection::FeatureSelfContact);
 
     Tp::ContactFactoryPtr contactFactory = KTp::ContactFactory::create(Tp::Features()  << Tp::Contact::FeatureAlias

-- 
ktp-contact-list packaging



More information about the pkg-kde-commits mailing list