Crash when accessing a preview in the uapp explorer scope

Bug #1581495 reported by Albert Astals Cid
10
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Canonical System Image
Fix Released
Undecided
Unassigned
unity-scopes-shell (Ubuntu)
Fix Released
High
Paweł Stołowski

Bug Description

When trying to access a preview in the uapp explorer scope it, unity8-dash crashes

Backtrace: http://paste.ubuntu.com/16387928/

Related branches

description: updated
Changed in unity-scopes-shell (Ubuntu):
assignee: nobody → Pawel Stolowski (stolowski)
Revision history for this message
Paweł Stołowski (stolowski) wrote :

Ok, the scope has a bug as it pushed two widgets with same ID on preview. Obviously we shouldn't crash, so that needs to be handled. Respective snipper of unity8-dash.log when this happens:

--- Widget lookups dump "addReplaceWidget" ---
Widget "header" at position 0 , lookup index 0
Widget "art" at position 1 , lookup index 1
Widget "types" at position 2 , lookup index 2
Widget "author" at position 3 , lookup index 3
Widget "hearts" at position 4 , lookup index 4
Widget "stars" at position 5 , lookup index 5
Widget "actions" at position 6 , lookup index 6
PreviewModel::addWidgetToColumnModel(): processing widget "author"
PreviewModel::addWidgetToColumnModel(): destination for widget "author" is row 7 , column 0
PreviewWidgetModel::moveWidget(): invalid destRow 7

The offending widgets are authorWidget and moreWidget in this code, both with "author" id: https://github.com/bhdouglass/uappexplorer-scope/blob/master/src/uappexplorer-scope.go

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

Status changed to 'Confirmed' because the bug affects multiple users.

Changed in unity-scopes-shell (Ubuntu):
status: New → Confirmed
Changed in canonical-devices-system-image:
status: New → Confirmed
Revision history for this message
Paweł Stołowski (stolowski) wrote :
Changed in unity-scopes-shell (Ubuntu):
status: Confirmed → In Progress
Changed in unity-scopes-shell (Ubuntu):
importance: Undecided → High
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package unity-scopes-shell - 0.5.7+16.10.20160525-0ubuntu1

---------------
unity-scopes-shell (0.5.7+16.10.20160525-0ubuntu1) yakkety; urgency=medium

  [ Gary.Wzl ]
  * Stop typing timer when reset navigation tag(tapping cancel). Also
    make sure relevant signal(searchQueryChanged) is triggered to keep
    query string updated as query string is empty after tapping cancel
    (LP: #1576311)

  [ Marcus Tomlinson ]
  * Only create an empty settings file when attempting to write to one
    (LP: #1583055)

  [ Pawel Stolowski ]
  * Don't depend on qt5-default.
  * Reject preview widgets with duplicated IDs. (LP: #1581495)
  * Set internet connectivity status in ActionMetadata (previews and
    preview actions). (LP: #1502282)
  * Workaround dependency issue on yakkety.

 -- Pawel Stolowski <email address hidden> Wed, 25 May 2016 08:16:27 +0000

Changed in unity-scopes-shell (Ubuntu):
status: In Progress → Fix Released
Changed in canonical-devices-system-image:
status: Confirmed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

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