content-hub

Import broken for ContentTypes without default apps

Bug #1429695 reported by Robert Schroll
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
content-hub
Fix Released
High
Ken VanDine
content-hub (Ubuntu)
Fix Released
Undecided
Unassigned

Bug Description

The content hub is broken when trying to import a content type for which there is no default handler (for example, ContentType.EBooks). When you try to import from the default provider, the ContentTransferHint will open and start a spinner, but nothing else happens. If you try to choose a provider, you at least get an error message stating that there are no providers.

This is seen on a device running vivid/devel r1.

Related branches

lp:~ken-vandine/content-hub/lp1429695
Michael Sheldon (community): Approve
PS Jenkins bot: Approve (continuous-integration)
Diff: 29 lines (+9/-3)
1 file modified
import/Ubuntu/Content/contentpeer.cpp (+9/-3)
lp:ubuntu/vivid-proposed/content-hub
Ken VanDine (ken-vandine)
Changed in content-hub:
status: New → Confirmed
importance: Undecided → High
assignee: nobody → Ken VanDine (ken-vandine)
Revision history for this message
Ken VanDine (ken-vandine) wrote :

We actually don't have an API to return feedback for errors to the app that requested the transfer. We could set the state to aborted immediately if requesting default peer when there is no defined default peer, the requester could then handle the aborted transfer. However, we should really add API to the ContentTransfer object to return errors to the requestor as well as an additional state for ContentTransfer.Error.

Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package content-hub - 0.0+15.04.20150331-0ubuntu1

---------------
content-hub (0.0+15.04.20150331-0ubuntu1) vivid; urgency=medium

  [ Ken VanDine ]
  * When requesting an import from an unknown peer, abort the transfer.
    This is most likely to occur when requesting the default peer for a
    ContentType that has no default registered. (LP: #1429695)
 -- CI Train Bot <email address hidden> Tue, 31 Mar 2015 14:10:11 +0000

Changed in content-hub (Ubuntu):
status: New → Fix Released
Ken VanDine (ken-vandine)
Changed in content-hub:
status: Confirmed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.
Subscribing...

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.