Bug in kcfg, KColorCombo not handled correct

[Christoph Feck]

On Thursday 09 February 2012 01:44:07 Ben Cooksley wrote:
> On Fri, Nov 4, 2011 at 9:50 AM, Christoph Feck <christoph at maxiom.de> 
wrote:
> > On Thursday 03 November 2011 13:14:31 Dag Andersen wrote:
> >> Torsdag 03 november 2011 10:32:43 skrev Christoph Feck:
> >> > On Thursday 03 November 2011 09:47:45 Dag Andersen wrote:
> >> > > AFAICS this commit:
> >> > > 8e95387f0d96ceb8439339643d1b9286a9204662
> >> > > broke kcfg handling of KColorCombo.
> >> > > 
> >> > > The attached patch fixes it for me.
> >> > 
> >> > Hm, KColorCombo has a user property, so it should work. The
> >> > patch is not good, because no other subclasses of QComboBox
> >> > are handled. Can you find out why the user property of
> >> > KColorCombo is not respected?
> >> 
> >> I added property
> >> kcfg_property=color
> >> to my KColorCombos and now it works, but this seems like an
> >> unnecessary complication to me.
> >> 
> >> > Christoph Feck (kdepepo)
> > 
> > I think I see what's going wrong. We decided to ignore the user
> > property for QComboBox once Qt added it, because it was different
> > from what KDE expected (KDE needs the text, Qt uses the index).
> > 
> > We should continue to ignore this user property from QComboBox
> > and all its subclasses, _unless_ the subclass has its own user
> > property, like KColorCombo does.
> > 
> > I have to check if Qt's meta system can supply this
> > information...
> 
> What is the status of this?
> Has the issue been fixed?

I filed bug 293702 so it will not be forgotten again. I currently have 
no easy way to build KDE stuff, so someone else could have a look at 
it. Sorry.

> 
> > Christoph Feck (kdepepo)
> 
> Regards,
> Ben