Issue 122401 - New (.uno:AddDirect) button of standard toolbar not working well on Basic IDE
Summary: New (.uno:AddDirect) button of standard toolbar not working well on Basic IDE
Alias: None
Product: General
Classification: Code
Component: ui (show other issues)
Version: 4.0.0-dev
Hardware: All All
: P3 Normal (vote)
Target Milestone: 4.0.0
Assignee: Ariel Constenla-Haile
QA Contact:
Depends on:
Reported: 2013-05-28 20:10 UTC by hanya
Modified: 2013-07-21 10:10 UTC (History)
2 users (show)

See Also:
Issue Type: DEFECT
Latest Confirmation in: ---
Developer Difficulty: ---


Note You need to log in before you can comment on or make changes to this issue.
Description hanya 2013-05-28 20:10:08 UTC
Nothing happen when I click on the New button of standard toolbar on Basic IDE.
- Choosing an entry from the dropdown menu is working.
- To click on the New button, not working.
On other document types, it seems ok.

On Xubuntu 12.04 64bit in VMware Player, 
AOO400m2(Build:9701)  -  Rev. 1486702
2013-05-28_04:09:49 - Rev. 1486733
And also Xubuntu 32bit, Rev. 1486438 built myself.
Comment 1 Ariel Constenla-Haile 2013-05-28 20:34:45 UTC
I've got a fix for that forgot to commit (note to myself: cherry-picking from a git clone seems error prone when no using stgit, nor git-svn).

IIRC something happens, the private URL from basic is dispatched, what of course does nothing.
Comment 2 SVN Robot 2013-05-28 22:09:28 UTC
"arielch" committed SVN revision 1487142 into trunk:
i122401 - framework::NewToolbarController needs early initialization of its P...
Comment 3 Ariel Constenla-Haile 2013-05-28 22:23:18 UTC
The main reason was that the algorithms relay on the PopupMenu filled at initialization. The default behaviour of framework::PopupMenuToolbarController is the later initialization of the PopupMenuController, only when the drop-down button is pressed. If early initialization is needed, the derived class has to override initialize(). 

@hanya please check with revision 1487142

There shouldn't be any new bugs, as the code is mainly copy&paste from the old implementation.

Looking at my notes back then, the old implementation had issues with the Writer based URL containing slots in the query part, as in


it dispatched the URL without the query part. Feel free to open a new bug for this.