Bug 194732 - 'plugActionList' does not work in combination with 'KParts::MainWindow' and manually inserted KXMLGUIClients
Summary: 'plugActionList' does not work in combination with 'KParts::MainWindow' and m...
Status: RESOLVED FIXED
Alias: None
Product: kdelibs
Classification: Frameworks and Libraries
Component: kdeui (show other bugs)
Version: SVN
Platform: Compiled Sources Linux
: NOR normal
Target Milestone: ---
Assignee: kdelibs bugs
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2009-05-31 11:18 UTC by Michel Ludwig
Modified: 2013-11-30 01:48 UTC (History)
4 users (show)

See Also:
Latest Commit:
Version Fixed In:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Michel Ludwig 2009-05-31 11:18:08 UTC
Version:            (using Devel)
OS:                Linux
Installed from:    Compiled sources

In Kile we use a 'KParts::MainWindow' for the GUI, which is set up with 'KParts::MainWindow::createGUI'. After that we populate dynamic menus using 'plugActionList'.

This works fine the first time we create the GUI, but then we merge in a KatePart view using 'guiFactory()->addClient'. After this is done, calling 'unplugActionList' or 'plugActionList' does not have any effect anymore.

Initial investigations have shown that there seems to be a problem with the merging of the XML GUI files. Before we merge the KatePart the client name reported in 'KXMLGUIFactory::plugActionList' is set to 'standard_containers'. But after that the name reported there changes to 'kile'. In consequence 'ContainerNode::plugActionList' does not find the right merging place anymore as the cached name 'standard_containers' does not match 'kile'.

My guess would be that even after the first merging the reported client name should not be 'standard_containers' but 'kile' instead.
Comment 1 Thomas Braun 2009-06-03 19:42:52 UTC
The problem is also in KDE 4.3 beta1. Any chance for a fix till 4.3 release?
Comment 2 Michel Ludwig 2009-06-08 23:13:54 UTC
Change the component to 'shortcuts' where it seems to fit better!
Comment 3 Michel Ludwig 2009-06-09 08:58:45 UTC
Sorry, I was a little confused, changing it back to 'kdeui'...
Comment 4 Joris Guisson 2010-07-22 11:49:33 UTC
Anybody interested in fixing this ? This is a rather serious issue and makes using multiple parts in an application impossible, if you want to use an ActionList to switch between the parts.
Comment 5 David Faure 2010-10-19 13:52:01 UTC
I think I fixed this already?

------------------------------------------------------------------------
r1026838 | dfaure | 2009-09-22 20:05:06 +0200 (Tue, 22 Sep 2009)

Merge attributes of the toplevel element too, so that we get <gui name="myapp"> and not <gui name="standard_containers">
 in the guiclient's domDocument(). Seen when testing a kedittoolbar change that would make it use domDocument().


Can you tell me if you're still having the problem?
Comment 6 Christoph Feck 2013-07-08 03:29:46 UTC
If you can provide the information requested in comment 5, please add it.
Comment 7 Christoph Feck 2013-11-19 02:09:19 UTC
To further investigate this issue, KDE developers need the information requested in comment #5. If you can provide it, or need help with finding that information, please add a comment.
Comment 8 Christoph Feck 2013-11-30 01:48:39 UTC
No response, changing status. Please reopen, if you have more information.