On Monday, February 24, 2014 05:22:13 PM Arjen Hiemstra wrote:
> On Monday 24 February 2014 10:19:37 Allen Winter wrote:
> > On Monday, February 24, 2014 11:52:27 AM Elvis Angelaccio wrote:
> > > Hi all.
> > > 
> > > I have a problem validating a kxmlgui .rc file whit at least *two* State
> > > elements.
> > > The following minimal example should reproduce the issue:
> > > http://pastebin.com/XLjHpb4n
> > > 
> > > The xmllint output is: http://pastebin.com/NDGardNQ
> > > 
> > > The strange thing is that if I comment one of the two State elements, the
> > > .rc file validates normally.
> > > 
> > > I really don't understand where am I wrong, I think that a State element
> > > should be at the same hierarchy level than other elements such as
> > > MenuBar/ToolBar/etc.
> > 
> > I don't see anything wrong with your .rc file.
> > I'm guessing the kxmlgui.xsd needs improvement.
> > 
> > Unless/until a guru fixes the xsd, I can tell Krazy to filter away
> > xmlint messages that complain about State elements and ActionProperties.
> 
> The XSD is missing a maxOccurs="unbounded" attribute on line 381. Without it, 
> it is assumed there can be only one "State" element. I tested it here 
> locally, 
> adding that attribute makes the rc file validate.
> 

Can you commit the fix?
I suppose the fix needs to go into frameworks as well, although I don't know 
where.

If you don't have commit access, please send me a patch and I will commit for 
you.

Thanks for taking care of this.
-Allen


>> Visit http://mail.kde.org/mailman/listinfo/kde-devel#unsub to unsubscribe <<

Reply via email to