Merge lp:~phablet-team/messaging-app/fix-argument-parsing2 into lp:messaging-app/stable

Proposed by Tiago Salem Herrmann on 2017-01-04
Status: Merged
Approved by: Gustavo Pichorim Boiko on 2017-01-11
Approved revision: 616
Merged at revision: 616
Proposed branch: lp:~phablet-team/messaging-app/fix-argument-parsing2
Merge into: lp:messaging-app/stable
Diff against target: 12 lines (+2/-0)
1 file modified
src/messagingapplication.cpp (+2/-0)
To merge this branch: bzr merge lp:~phablet-team/messaging-app/fix-argument-parsing2
Reviewer Review Type Date Requested Status
Gustavo Pichorim Boiko (community) 2017-01-04 Approve on 2017-01-11
Review via email: mp+314109@code.launchpad.net

Commit message

Fix argument parsing from url-dispatcher with the "message:phoneNumber" format.

Description of the change

Fix argument parsing from url-dispatcher with the "message:phoneNumber" format.

To post a comment you must log in.
615. By Tiago Salem Herrmann on 2017-01-04

merge parent branch

616. By Tiago Salem Herrmann on 2017-01-04

merge parent branch

Gustavo Pichorim Boiko (boiko) wrote :

Looks good!

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'src/messagingapplication.cpp'
2--- src/messagingapplication.cpp 2016-10-17 13:02:38 +0000
3+++ src/messagingapplication.cpp 2017-01-04 19:03:25 +0000
4@@ -258,6 +258,8 @@
5 QStringList participantIds = value.split(";");
6 properties["participantIds"] = participantIds;
7 }
8+ } else {
9+ properties["participantIds"] = QStringList() << value;
10 }
11 QUrlQuery query(url);
12 Q_FOREACH(const Pair &item, query.queryItems(QUrl::FullyDecoded)) {

Subscribers

People subscribed via source and target branches

to all changes: