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

Proposed by Tiago Salem Herrmann
Status: Merged
Approved by: Gustavo Pichorim Boiko
Approved revision: 642
Merged at revision: 642
Proposed branch: lp:~phablet-team/messaging-app/fix-argument-parsing
Merge into: lp:messaging-app/staging
Prerequisite: lp:~phablet-team/messaging-app/irc-service
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-parsing
Reviewer Review Type Date Requested Status
Gustavo Pichorim Boiko (community) Approve
Review via email: mp+314167@code.launchpad.net

This proposal supersedes a proposal from 2017-01-03.

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.
642. By Tiago Salem Herrmann

merge parent branch

Revision history for this message
Gustavo Pichorim Boiko (boiko) wrote :

Looks good!

review: Approve
643. By Tiago Salem Herrmann

merge parent branch

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-03-08 21:56:47 +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: