Ubuntu RTM
ubuntu-ui-toolkit package

Page pushed on a stack gets instantiated twice, and PageStack.push() returns null

Bug #1495451 reported by Olivier Tilloy
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
ubuntu-ui-toolkit (Ubuntu RTM)
Fix Committed
Critical
Zsombor Egri

Bug Description

It looks to me like a regression introduced by the latest UITK landing (1.3.1627+15.04.20150908-0ubuntu1).
See the attached standalone reproducer (run it with qmlscene).
In a MainView, pushing a page component on a page stack instantiates the page twice.
Additionally, the result of pageStack.push(pageComponent) is null, and it seems the 'currentPage' property of the stack is not updated.

Related branches

lp://qastaging/~aacid/ubuntu-ui-toolkit/setIncubator
PS Jenkins bot: Approve (continuous-integration)
Zsombor Egri: Approve
Diff: 11 lines (+1/-0)
1 file modified
src/Ubuntu/Components/1.3/PageWrapperUtils.js (+1/-0)
Revision history for this message
Olivier Tilloy (osomon) wrote :
Revision history for this message
Olivier Tilloy (osomon) wrote :

I’m also seeing the following error printed twice on the console, I guess it’s related:

     file:///usr/lib/arm-linux-gnueabihf/qt5/qml/Ubuntu/Components/1.3/PageWrapperUtils.js:45: TypeError: Type error

Revision history for this message
Olivier Tilloy (osomon) wrote :

Is my example code doing something obviously wrong that would trigger this behaviour, or is this a valid bug?

Tim Peeters (tpeeters)
Changed in ubuntu-ui-toolkit (Ubuntu):
status: New → Confirmed
importance: Undecided → Critical
Zsombor Egri (zsombi)
Changed in ubuntu-ui-toolkit (Ubuntu):
assignee: nobody → Zsombor Egri (zsombi)
Zsombor Egri (zsombi)
Changed in ubuntu-ui-toolkit (Ubuntu):
status: Confirmed → In Progress
Revision history for this message
Zsombor Egri (zsombi) wrote :

Olivier, please confirm that https://code.launchpad.net/~aacid/ubuntu-ui-toolkit/setIncubator/+merge/271119 branch fixes the bug, at least I cannot reproduce it anymore.

Revision history for this message
Olivier Tilloy (osomon) wrote :

I confirm that Albert’s branch fixes the issue.

Zsombor Egri (zsombi)
Changed in ubuntu-ui-toolkit (Ubuntu):
status: In Progress → Fix Committed
Zoltan Balogh (bzoltan)
affects: ubuntu-ui-toolkit (Ubuntu) → ubuntu-ui-toolkit (Ubuntu RTM)
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.
Subscribing...

Other bug subscribers

Bug attachments

Remote bug watches

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