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
1=== modified file 'CMakeLists.txt'
2--- CMakeLists.txt 2016-06-08 18:35:43 +0000
3+++ CMakeLists.txt 2017-01-05 16:15:26 +0000
4@@ -71,6 +71,7 @@
5
6 add_subdirectory(po)
7 add_subdirectory(src)
8+add_subdirectory(accounts)
9 if (NOT ("${CMAKE_SYSTEM_PROCESSOR}" STREQUAL "aarch64") AND (NOT ("${CMAKE_SYSTEM_PROCESSOR}" STREQUAL "ppc64el")) AND (NOT ("${CMAKE_SYSTEM_PROCESSOR}" STREQUAL "ppc64le")))
10 add_subdirectory(tests)
11 endif()
12
13=== added directory 'accounts'
14=== added file 'accounts/CMakeLists.txt'
15--- accounts/CMakeLists.txt 1970-01-01 00:00:00 +0000
16+++ accounts/CMakeLists.txt 2017-01-05 16:15:26 +0000
17@@ -0,0 +1,5 @@
18+file(GLOB APPLICATION_FILES *.application)
19+install(FILES ${APPLICATION_FILES} DESTINATION share/accounts/applications/)
20+
21+file(GLOB SERVICE_FILES *.service)
22+install(FILES ${SERVICE_FILES} DESTINATION share/accounts/services/)
23
24=== added file 'accounts/messaging-app-im-irc.service'
25--- accounts/messaging-app-im-irc.service 1970-01-01 00:00:00 +0000
26+++ accounts/messaging-app-im-irc.service 2017-01-05 16:15:26 +0000
27@@ -0,0 +1,19 @@
28+<?xml version="1.0" encoding="UTF-8"?>
29+<service id="messaging-app-im-irc">
30+ <type>messaging-app-im</type>
31+ <name>IRC</name>
32+ <icon>irc</icon>
33+ <provider>irc</provider>
34+
35+ <!-- default settings (account settings have precedence over these) -->
36+ <template>
37+ <group name="telepathy">
38+ <setting name="manager">irc</setting>
39+ <setting name="protocol">irc</setting>
40+ </group>
41+ <group name="auth">
42+ <setting name="method">password</setting>
43+ <setting name="mechanism">password</setting>
44+ </group>
45+ </template>
46+</service>
47
48=== added file 'accounts/messaging-app.application'
49--- accounts/messaging-app.application 1970-01-01 00:00:00 +0000
50+++ accounts/messaging-app.application 2017-01-05 16:15:26 +0000
51@@ -0,0 +1,12 @@
52+<?xml version="1.0" encoding="UTF-8"?>
53+<application id="messaging-app">
54+ <description>Messaging</description>
55+ <desktop-entry>messaging-app.desktop</desktop-entry>
56+ <translations>messaging</translations>
57+
58+ <services>
59+ <service id="messaging-app-im-irc">
60+ <description>Integrate your IRC accounts</description>
61+ </service>
62+ </services>
63+</application>
64
65=== modified file 'debian/messaging-app.install'
66--- debian/messaging-app.install 2016-05-20 04:32:06 +0000
67+++ debian/messaging-app.install 2017-01-05 16:15:26 +0000
68@@ -1,3 +1,5 @@
69+usr/share/accounts/applications/*
70+usr/share/accounts/services/*
71 usr/share/applications/messaging-app*.desktop
72 usr/share/url-dispatcher/urls
73 usr/share/content-hub/peers/messaging-app
74
75=== modified file 'src/messagingapplication.cpp'
76--- src/messagingapplication.cpp 2016-10-17 13:02:38 +0000
77+++ src/messagingapplication.cpp 2017-01-05 16:15:26 +0000
78@@ -258,6 +258,8 @@
79 QStringList participantIds = value.split(";");
80 properties["participantIds"] = participantIds;
81 }
82+ } else {
83+ properties["participantIds"] = QStringList() << value;
84 }
85 QUrlQuery query(url);
86 Q_FOREACH(const Pair &item, query.queryItems(QUrl::FullyDecoded)) {
87
88=== modified file 'src/qml/NewGroupPage.qml'
89--- src/qml/NewGroupPage.qml 2016-10-11 02:01:24 +0000
90+++ src/qml/NewGroupPage.qml 2017-01-05 16:15:26 +0000
91@@ -84,6 +84,9 @@
92 if (newGroupPage.creationInProgress) {
93 return false
94 }
95+ if (account.protocolInfo.joinExistingChannels && groupTitleField.text != "") {
96+ return true
97+ }
98 if (participantsModel.count == 0) {
99 return false
100 }
101@@ -96,6 +99,9 @@
102 onTriggered: {
103 Qt.inputMethod.commit()
104 newGroupPage.creationInProgress = true
105+ if (account.protocolInfo.joinExistingChannels) {
106+ chatEntry.chatId = groupTitleField.text
107+ }
108 chatEntry.startChat()
109 }
110 }
111@@ -249,6 +255,7 @@
112 ContactSearchWidget {
113 id: searchItem
114 parentPage: newGroupPage
115+ visible: !account.protocolInfo.joinExistingChannels
116 searchResultsHeight: flick.emptySpaceHeight
117 onContactPicked: addRecipientFromSearch(identifier, alias, avatar)
118 anchors {
119@@ -259,6 +266,7 @@
120 }
121 Rectangle {
122 id: separator2
123+ visible: !account.protocolInfo.joinExistingChannels
124 anchors {
125 left: parent.left
126 right: parent.right
127@@ -285,6 +293,7 @@
128 anchors.top: searchItem.bottom
129 anchors.left: parent.left
130 anchors.right: parent.right
131+ visible: !account.protocolInfo.joinExistingChannels
132 Repeater {
133 id: participantsRepeater
134 model: participantsModel

Subscribers

People subscribed via source and target branches

to all changes: