content-hub

Merge lp:~ken-vandine/content-hub/invoke_destination into lp:~content-hub-team/content-hub/trunk

  1. invoke_destination
  2. Merge into trunk
Proposed by Ken VanDine
Status: Merged
Approved by: Ken VanDine
Approved revision: 57
Merged at revision: 58
Proposed branch: lp:~ken-vandine/content-hub/invoke_destination
Merge into: lp:~content-hub-team/content-hub/trunk
Diff against target: 10 lines (+1/-0)
1 file modified
src/com/ubuntu/content/detail/service.cpp (+1/-0)
To merge this branch: bzr merge lp:~ken-vandine/content-hub/invoke_destination
Reviewer Review Type Date Requested Status
PS Jenkins bot continuous-integration Approve
Sebastien Bacher (community) Approve
Ubuntu Phablet Team Pending
Review via email: mp+190217@code.launchpad.net

Commit message

invoke the destination when the transfer is charged

Description of the change

invoke the destination when the transfer is charged

To post a comment you must log in.
Revision history for this message
Sebastien Bacher (seb128) wrote :

looks fine, I guess you tested it, right? ;-)

Revision history for this message
Sebastien Bacher (seb128) :
review: Approve
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :

PASSED: Continuous integration, rev:57
http://jenkins.qa.ubuntu.com/job/content-hub-ci/94/
Executed test runs:
    SUCCESS: http://jenkins.qa.ubuntu.com/job/content-hub-saucy-amd64-ci/93
    SUCCESS: http://jenkins.qa.ubuntu.com/job/content-hub-saucy-armhf-ci/93
        deb: http://jenkins.qa.ubuntu.com/job/content-hub-saucy-armhf-ci/93/artifact/work/output/*zip*/output.zip
    SUCCESS: http://jenkins.qa.ubuntu.com/job/content-hub-saucy-i386-ci/93

Click here to trigger a rebuild:
http://10.97.0.26:8080/job/content-hub-ci/94/rebuild

review: Approve (continuous-integration)

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'src/com/ubuntu/content/detail/service.cpp'
2--- src/com/ubuntu/content/detail/service.cpp 2013-10-08 16:41:48 +0000
3+++ src/com/ubuntu/content/detail/service.cpp 2013-10-09 18:49:29 +0000
4@@ -222,6 +222,7 @@
5 if (state == cuc::Transfer::charged)
6 {
7 qDebug() << Q_FUNC_INFO << "Charged";
8+ d->app_manager->invoke_application(transfer->destination().toStdString());
9 this->connect_import_handler(transfer->destination(), transfer->import_path());
10 }
11

Subscribers

People subscribed via source and target branches