Merge lp:~phablet-team/messaging-app/fix-argument-parsing into lp:messaging-app/staging

Proposed by Tiago Salem Herrmann
Status: Superseded
Proposed branch: lp:~phablet-team/messaging-app/fix-argument-parsing
Merge into: lp:messaging-app/staging
Diff against target: 134 lines (+50/-0)
7 files modified
CMakeLists.txt (+1/-0)
accounts/CMakeLists.txt (+5/-0)
accounts/messaging-app-im-irc.service (+19/-0)
accounts/messaging-app.application (+12/-0)
debian/messaging-app.install (+2/-0)
src/messagingapplication.cpp (+2/-0)
src/qml/NewGroupPage.qml (+9/-0)
To merge this branch: bzr merge lp:~phablet-team/messaging-app/fix-argument-parsing
Reviewer Review Type Date Requested Status
Ubuntu Phablet Team Pending
Review via email: mp+314043@code.launchpad.net

This proposal has been superseded by a proposal from 2017-01-05.

Commit message

Fix argument parsing with the messaging:number pattern.

Description of the change

Fix argument parsing with the messaging:number pattern.

To post a comment you must log in.
641. By Tiago Salem Herrmann

merge parent branch

642. By Tiago Salem Herrmann

merge parent branch

643. By Tiago Salem Herrmann

merge parent branch

Unmerged revisions

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'CMakeLists.txt'
--- CMakeLists.txt 2016-06-08 18:35:43 +0000
+++ CMakeLists.txt 2017-01-05 16:15:26 +0000
@@ -71,6 +71,7 @@
7171
72add_subdirectory(po)72add_subdirectory(po)
73add_subdirectory(src)73add_subdirectory(src)
74add_subdirectory(accounts)
74if (NOT ("${CMAKE_SYSTEM_PROCESSOR}" STREQUAL "aarch64") AND (NOT ("${CMAKE_SYSTEM_PROCESSOR}" STREQUAL "ppc64el")) AND (NOT ("${CMAKE_SYSTEM_PROCESSOR}" STREQUAL "ppc64le")))75if (NOT ("${CMAKE_SYSTEM_PROCESSOR}" STREQUAL "aarch64") AND (NOT ("${CMAKE_SYSTEM_PROCESSOR}" STREQUAL "ppc64el")) AND (NOT ("${CMAKE_SYSTEM_PROCESSOR}" STREQUAL "ppc64le")))
75 add_subdirectory(tests)76 add_subdirectory(tests)
76endif()77endif()
7778
=== added directory 'accounts'
=== added file 'accounts/CMakeLists.txt'
--- accounts/CMakeLists.txt 1970-01-01 00:00:00 +0000
+++ accounts/CMakeLists.txt 2017-01-05 16:15:26 +0000
@@ -0,0 +1,5 @@
1file(GLOB APPLICATION_FILES *.application)
2install(FILES ${APPLICATION_FILES} DESTINATION share/accounts/applications/)
3
4file(GLOB SERVICE_FILES *.service)
5install(FILES ${SERVICE_FILES} DESTINATION share/accounts/services/)
06
=== added file 'accounts/messaging-app-im-irc.service'
--- accounts/messaging-app-im-irc.service 1970-01-01 00:00:00 +0000
+++ accounts/messaging-app-im-irc.service 2017-01-05 16:15:26 +0000
@@ -0,0 +1,19 @@
1<?xml version="1.0" encoding="UTF-8"?>
2<service id="messaging-app-im-irc">
3 <type>messaging-app-im</type>
4 <name>IRC</name>
5 <icon>irc</icon>
6 <provider>irc</provider>
7
8 <!-- default settings (account settings have precedence over these) -->
9 <template>
10 <group name="telepathy">
11 <setting name="manager">irc</setting>
12 <setting name="protocol">irc</setting>
13 </group>
14 <group name="auth">
15 <setting name="method">password</setting>
16 <setting name="mechanism">password</setting>
17 </group>
18 </template>
19</service>
020
=== added file 'accounts/messaging-app.application'
--- accounts/messaging-app.application 1970-01-01 00:00:00 +0000
+++ accounts/messaging-app.application 2017-01-05 16:15:26 +0000
@@ -0,0 +1,12 @@
1<?xml version="1.0" encoding="UTF-8"?>
2<application id="messaging-app">
3 <description>Messaging</description>
4 <desktop-entry>messaging-app.desktop</desktop-entry>
5 <translations>messaging</translations>
6
7 <services>
8 <service id="messaging-app-im-irc">
9 <description>Integrate your IRC accounts</description>
10 </service>
11 </services>
12</application>
013
=== modified file 'debian/messaging-app.install'
--- debian/messaging-app.install 2016-05-20 04:32:06 +0000
+++ debian/messaging-app.install 2017-01-05 16:15:26 +0000
@@ -1,3 +1,5 @@
1usr/share/accounts/applications/*
2usr/share/accounts/services/*
1usr/share/applications/messaging-app*.desktop3usr/share/applications/messaging-app*.desktop
2usr/share/url-dispatcher/urls4usr/share/url-dispatcher/urls
3usr/share/content-hub/peers/messaging-app5usr/share/content-hub/peers/messaging-app
46
=== modified file 'src/messagingapplication.cpp'
--- src/messagingapplication.cpp 2016-10-17 13:02:38 +0000
+++ src/messagingapplication.cpp 2017-01-05 16:15:26 +0000
@@ -258,6 +258,8 @@
258 QStringList participantIds = value.split(";");258 QStringList participantIds = value.split(";");
259 properties["participantIds"] = participantIds;259 properties["participantIds"] = participantIds;
260 }260 }
261 } else {
262 properties["participantIds"] = QStringList() << value;
261 }263 }
262 QUrlQuery query(url);264 QUrlQuery query(url);
263 Q_FOREACH(const Pair &item, query.queryItems(QUrl::FullyDecoded)) {265 Q_FOREACH(const Pair &item, query.queryItems(QUrl::FullyDecoded)) {
264266
=== modified file 'src/qml/NewGroupPage.qml'
--- src/qml/NewGroupPage.qml 2016-10-11 02:01:24 +0000
+++ src/qml/NewGroupPage.qml 2017-01-05 16:15:26 +0000
@@ -84,6 +84,9 @@
84 if (newGroupPage.creationInProgress) {84 if (newGroupPage.creationInProgress) {
85 return false85 return false
86 }86 }
87 if (account.protocolInfo.joinExistingChannels && groupTitleField.text != "") {
88 return true
89 }
87 if (participantsModel.count == 0) {90 if (participantsModel.count == 0) {
88 return false91 return false
89 }92 }
@@ -96,6 +99,9 @@
96 onTriggered: {99 onTriggered: {
97 Qt.inputMethod.commit()100 Qt.inputMethod.commit()
98 newGroupPage.creationInProgress = true101 newGroupPage.creationInProgress = true
102 if (account.protocolInfo.joinExistingChannels) {
103 chatEntry.chatId = groupTitleField.text
104 }
99 chatEntry.startChat()105 chatEntry.startChat()
100 }106 }
101 }107 }
@@ -249,6 +255,7 @@
249 ContactSearchWidget {255 ContactSearchWidget {
250 id: searchItem256 id: searchItem
251 parentPage: newGroupPage257 parentPage: newGroupPage
258 visible: !account.protocolInfo.joinExistingChannels
252 searchResultsHeight: flick.emptySpaceHeight259 searchResultsHeight: flick.emptySpaceHeight
253 onContactPicked: addRecipientFromSearch(identifier, alias, avatar)260 onContactPicked: addRecipientFromSearch(identifier, alias, avatar)
254 anchors {261 anchors {
@@ -259,6 +266,7 @@
259 }266 }
260 Rectangle {267 Rectangle {
261 id: separator2268 id: separator2
269 visible: !account.protocolInfo.joinExistingChannels
262 anchors {270 anchors {
263 left: parent.left271 left: parent.left
264 right: parent.right272 right: parent.right
@@ -285,6 +293,7 @@
285 anchors.top: searchItem.bottom293 anchors.top: searchItem.bottom
286 anchors.left: parent.left294 anchors.left: parent.left
287 anchors.right: parent.right295 anchors.right: parent.right
296 visible: !account.protocolInfo.joinExistingChannels
288 Repeater {297 Repeater {
289 id: participantsRepeater298 id: participantsRepeater
290 model: participantsModel299 model: participantsModel

Subscribers

People subscribed via source and target branches

to all changes: