UriHandler doesn't emit onOpened if the app needs to be started

Bug #1391689 reported by Michael Zanetti on 2014-11-11
10
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Ubuntu Notes app
High
Michael Zanetti
Ubuntu UI Toolkit
Invalid
Undecided
Unassigned
reminders-app (Ubuntu)
Undecided
Unassigned

Bug Description

Having this in the QML code:

    Connections {
        target: UriHandler
        onOpened: {
            ...
        }
    }

and registering the according click hooks, it works fine if the app is already running when the url is dispatched. However, if the app needs to be started, it will start, but the onOpened handler is never called.

Related branches

David Planella (dpm) on 2014-11-12
Changed in reminders-app:
status: New → Triaged
importance: Undecided → High
XiaoGuo, Liu (liu-xiao-guo) wrote :
XiaoGuo, Liu (liu-xiao-guo) wrote :

Compile the two apps and install them onto your phones

Test cases:

- When "myapp" is not running, using url dispatcher, in launchmyapp to invoke it. The parameters are passed in by command line
- When "myapp" is running, the above "UriHandler" will be called, and the parameters will be passed in

Michael Zanetti (mzanetti) wrote :

Turns out one needs to parse the application's command line arguments for this. This works, but would be nice if the docs would mention this. Especially since it differs how ContentHub behaves. That one launches the app and then emits the signal when the app has started on its own.

Changed in reminders-app:
status: Triaged → In Progress
assignee: nobody → Michael Zanetti (mzanetti)

Fix committed into lp:reminders-app at revision 313, scheduled for release in reminders-app, milestone 2014-11-27

Changed in reminders-app:
status: In Progress → Fix Committed
Tim Peeters (tpeeters) on 2014-11-17
Changed in ubuntu-ui-toolkit:
status: New → Invalid
Launchpad Janitor (janitor) wrote :
Download full text (5.1 KiB)

This bug was fixed in the package reminders-app - 0.4+15.04.20141118-0ubuntu1

---------------
reminders-app (0.4+15.04.20141118-0ubuntu1) vivid; urgency=low

  [ Akiva Avraham ]
  * Updated library imports to qtquick-2.3.
  * Upgraded ubuntu.components to 1.1 Upgraded ubuntu.components.xxxx to
    1.0.

  [ Paolo Rotolo ]
  * Added Tags icon.

  [ Alan Pope ]
  * Fix incorrect email address.

  [ Michael Zanetti ]
  * Add count properties to models.
  * add API to save notebooks.
  * correctly update the model when a notebook changes.
  * Some performance optimizations when having many, big notes
  * improve startup sequence
  * suppress some warnings. (LP: #1316800)
  * Add support for tags. (LP: #1379747)
  * Fixed bug 1378014 - Notes should be sorted by modification date by
    default Fixed bug 1380255 - Add day of the month to the notes
    preview. (LP: #1378014, #1380255)
  * Fixed change accounts page. (LP: #1379751)
  * Split the actual evernote code out from the QML plugin into a
    libqtevernote. (LP: #1385994)
  * Improve reminders delegate. (LP: #1318054)
  * fix pulldown positioning. (LP: #1368548)
  * split loading of notes into chunks
  * Don't reload the note's content when reloading a note for its
    resources. (LP: #1380987)
  * improve sorting by falling back on title if the first sortrole is
    equal. (LP: #1380987)
  * add gif support
  * Improve handling of connection errors.
  * include tags when cloning a note.
  * add scrollbars to listviews.
  * Don't always build translations
  * implement full note caching
  * improve note edit view. (LP: #1391410, #1316947)
  * small fixes in the new edit mode
  * Created URI handlers to load the app from the scope. (LP: #1391689,
    #1385995)

  [ Víctor R. Ruiz ]
  * "Search notes" unstranslated. Update to the code and template. (LP:
    #1356944)
  * Another batch of untranslated strings. Updated template. (LP:
    #1356944)

  [ Riccardo Padovani ]
  * Fixed #1340644 - Notes go in default notebook when edited. (LP:
    #1340644)
  * Upgraded the WebView to Oxide. (LP: #1317580)
  * Fixed #1340686 - Selecting a notebook then add a note doesn't
    default the note to the notebook. (LP: #1340686)
  * Added a couple of new i18n.tr, updated the translations template.
  * Updated the camera confirm page to have buttons again Added a check
    to be sure to delete previous temp picture. (LP: #1342249)
  * Update string for no account advice, removed reference to add
    reminder button. (LP: #1340635)
  * Fixed pull down to refresh behavior.
  * Implemented fronted to change notebooks name. (LP: #1340640)
  * Implemented bottomEdge.
  * Added a bouncing ProgressBar. (LP: #1372420)
  * Fix #1379886 - First time reminders tab is shown, the list is offset
    from the top. (LP: #1379886)
  * Fixed #1382730 - Unable to create new notes. (LP: #1382730)
  * Enabled automatic orientation. (LP: #1385429)
  * Fixed bug 1378014 - Notes should be sorted by modification date by
    default Fixed bug 1380255 - Add day of the month to the notes
    preview. (LP: #1378014, #1380255)
  * Fixed change accounts page. (LP: #1379751)
  * Created URI handlers to load the app from the scope. (LP: #...

Read more...

Changed in reminders-app (Ubuntu):
status: New → Fix Released
Changed in reminders-app:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers