Bug 194732

Summary:	'plugActionList' does not work in combination with 'KParts::MainWindow' and manually inserted KXMLGUIClients
Product:	[Unmaintained] kdelibs	Reporter:	Michel Ludwig <michel.ludwig>
Component:	kdeui	Assignee:	kdelibs bugs <kdelibs-bugs-null>
Status:	RESOLVED FIXED
Severity:	normal	CC:	cfeck, faure, joris.guisson, thomas.braun
Priority:	NOR
Version First Reported In:	SVN
Target Milestone:	---
Platform:	Compiled Sources
OS:	Linux
Latest Commit:		Version Fixed/Implemented In:
Sentry Crash Report:

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.