Comment 1 for bug 1581495

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