Merge lp:~tiagosh/address-book-app/fix-1234234 into lp:address-book-app

Proposed by Tiago Salem Herrmann
Status: Merged
Approved by: Renato Araujo Oliveira Filho
Approved revision: 95
Merged at revision: 97
Proposed branch: lp:~tiagosh/address-book-app/fix-1234234
Merge into: lp:address-book-app
Diff against target: 18 lines (+3/-3)
1 file modified
src/app/addressbookapp.cpp (+3/-3)
To merge this branch: bzr merge lp:~tiagosh/address-book-app/fix-1234234
Reviewer Review Type Date Requested Status
Renato Araujo Oliveira Filho (community) Approve
PS Jenkins bot continuous-integration Approve
Review via email: mp+189972@code.launchpad.net

Commit message

Use url decoding before using values.

Description of the change

Use url decoding before using values.

To post a comment you must log in.
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Approve (continuous-integration)
Revision history for this message
Renato Araujo Oliveira Filho (renatofilho) wrote :

looks good and works as expected.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'src/app/addressbookapp.cpp'
--- src/app/addressbookapp.cpp 2013-10-04 00:13:49 +0000
+++ src/app/addressbookapp.cpp 2013-10-08 20:55:15 +0000
@@ -324,11 +324,11 @@
324 method.invoke(mainView);324 method.invoke(mainView);
325 break;325 break;
326 case 1:326 case 1:
327 method.invoke(mainView, Q_ARG(QVariant, QVariant(args[0])));327 method.invoke(mainView, Q_ARG(QVariant, QVariant(QUrl::fromPercentEncoding(args[0].toUtf8()))));
328 break;328 break;
329 case 2:329 case 2:
330 method.invoke(mainView, Q_ARG(QVariant, QVariant(args[0])),330 method.invoke(mainView, Q_ARG(QVariant, QVariant(QUrl::fromPercentEncoding(args[0].toUtf8()))),
331 Q_ARG(QVariant, QVariant(args[1])));331 Q_ARG(QVariant, QVariant(QUrl::fromPercentEncoding(args[1].toUtf8()))));
332 break;332 break;
333 default:333 default:
334 qWarning() << "Invalid arguments";334 qWarning() << "Invalid arguments";

Subscribers

People subscribed via source and target branches