Merge lp:~phablet-team/telepathy-ofono/more_mms_group_fixes into lp:telepathy-ofono/staging

Proposed by Gustavo Pichorim Boiko
Status: Merged
Approved by: Roberto Mier Escandon
Approved revision: 180
Merged at revision: 180
Proposed branch: lp:~phablet-team/telepathy-ofono/more_mms_group_fixes
Merge into: lp:telepathy-ofono/staging
Diff against target: 41 lines (+6/-3)
3 files modified
connection.cpp (+1/-1)
ofonotextchannel.cpp (+4/-2)
ofonotextchannel.h (+1/-0)
To merge this branch: bzr merge lp:~phablet-team/telepathy-ofono/more_mms_group_fixes
Reviewer Review Type Date Requested Status
Roberto Mier Escandon (community) Approve
Review via email: mp+309525@code.launchpad.net

This proposal supersedes a proposal from 2016-10-28.

Commit message

Plug the room interface when necessary.

Description of the change

Plug the room interface when necessary.

To post a comment you must log in.
Revision history for this message
Roberto Mier Escandon (rmescandon) wrote :

lgtm

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'connection.cpp'
--- connection.cpp 2016-10-17 15:17:18 +0000
+++ connection.cpp 2016-10-28 02:48:59 +0000
@@ -747,7 +747,7 @@
747 uint targetHandleType = request.value(TP_QT_IFACE_CHANNEL + QLatin1String(".TargetHandleType")).toUInt();747 uint targetHandleType = request.value(TP_QT_IFACE_CHANNEL + QLatin1String(".TargetHandleType")).toUInt();
748 uint targetHandle = request.value(TP_QT_IFACE_CHANNEL + QLatin1String(".TargetHandle")).toUInt();748 uint targetHandle = request.value(TP_QT_IFACE_CHANNEL + QLatin1String(".TargetHandle")).toUInt();
749 QString targetId = request.value(TP_QT_IFACE_CHANNEL + QLatin1String(".TargetID")).toString();749 QString targetId = request.value(TP_QT_IFACE_CHANNEL + QLatin1String(".TargetID")).toString();
750 bool isRoom = request.contains(TP_QT_IFACE_CHANNEL_INTERFACE_ROOM + ".RoomName");750 bool isRoom = request.contains(TP_QT_IFACE_CHANNEL_INTERFACE_ROOM + QLatin1String(".RoomName"));
751751
752 if (mSelfPresence.type != Tp::ConnectionPresenceTypeAvailable) {752 if (mSelfPresence.type != Tp::ConnectionPresenceTypeAvailable) {
753 error->set(TP_QT_ERROR_NETWORK_ERROR, "No network available");753 error->set(TP_QT_ERROR_NETWORK_ERROR, "No network available");
754754
=== modified file 'ofonotextchannel.cpp'
--- ofonotextchannel.cpp 2016-10-20 23:12:23 +0000
+++ ofonotextchannel.cpp 2016-10-28 02:48:59 +0000
@@ -108,8 +108,10 @@
108 mSMSIface = Tp::BaseChannelSMSInterface::create(flash, true);108 mSMSIface = Tp::BaseChannelSMSInterface::create(flash, true);
109 baseChannel->plugInterface(Tp::AbstractChannelInterfacePtr::dynamicCast(mSMSIface));109 baseChannel->plugInterface(Tp::AbstractChannelInterfacePtr::dynamicCast(mSMSIface));
110110
111 // FIXME(MMSGroup): create and plug the Room interface, and maybe the subject or the111 if (mmsGroupChat) {
112 // roomconfig interface for the subject112 mRoomIface = Tp::BaseChannelRoomInterface::create("", "", "", 0, QDateTime());
113 baseChannel->plugInterface(Tp::AbstractChannelInterfacePtr::dynamicCast(mRoomIface));
114 }
113115
114 mBaseChannel = baseChannel;116 mBaseChannel = baseChannel;
115 mTextChannel = Tp::BaseChannelTextTypePtr::dynamicCast(mBaseChannel->interface(TP_QT_IFACE_CHANNEL_TYPE_TEXT));117 mTextChannel = Tp::BaseChannelTextTypePtr::dynamicCast(mBaseChannel->interface(TP_QT_IFACE_CHANNEL_TYPE_TEXT));
116118
=== modified file 'ofonotextchannel.h'
--- ofonotextchannel.h 2016-10-20 23:12:23 +0000
+++ ofonotextchannel.h 2016-10-28 02:48:59 +0000
@@ -66,6 +66,7 @@
66 Tp::BaseChannelMessagesInterfacePtr mMessagesIface;66 Tp::BaseChannelMessagesInterfacePtr mMessagesIface;
67 Tp::BaseChannelGroupInterfacePtr mGroupIface;67 Tp::BaseChannelGroupInterfacePtr mGroupIface;
68 Tp::BaseChannelSMSInterfacePtr mSMSIface;68 Tp::BaseChannelSMSInterfacePtr mSMSIface;
69 Tp::BaseChannelRoomInterfacePtr mRoomIface;
69 Tp::BaseChannelTextTypePtr mTextChannel;70 Tp::BaseChannelTextTypePtr mTextChannel;
70 uint mMessageCounter;71 uint mMessageCounter;
71 QMap<QString, uint> mPendingDeliveryReportTemporarilyFailed;72 QMap<QString, uint> mPendingDeliveryReportTemporarilyFailed;

Subscribers

People subscribed via source and target branches

to all changes: