KConfigDialogManager and QButtonGroup

[Martin Koller]

On Dienstag, 2. Mai 2017 11:30:04 CEST Albert Astals Cid wrote:
> El dilluns, 1 de maig de 2017, a les 20:31:28 CEST, Martin Koller va escriure:
> > Hi,
> > 
> > trying to fix https://bugs.kde.org/show_bug.cgi?id=378930
> > 
> > I see that the mboxresource has a configuration dialog which uses
> > KConfigDialogManager to handle the settings, but it has a QGroupBox widget
> > holding QRadioButtons, which represent one setting (e.g. one of 2 possible
> > compaction methods).
> > 
> > In the .ui file, the QGroupBox(!) has the name kcfg_CompactFrequency
> > but I assume the bug lies in the fact that this stems from Qt4 days, where
> > the groupBox did also manage the buttons, whereby now you need a
> > QButtonGroup class.
> > 
> > Am I correct and if so, what needs to be changed that KConfigDialogManager
> > works again with this radio buttons ?
> > Is KConfigDialogManager designed so that it can work with a QButtonGroup and
> > a set of radio buttons ?
> 
> Yes, see 
> https://cgit.kde.org/kconfigwidgets.git/tree/src/kconfigdialogmanager.cpp#n202

Debugging into this, I see that it does not work here since the QGroupBox
contains 2 radio buttons and one spinbox, but the kconfigdialogmanager.cpp
checks for ... wait

https://cgit.kde.org/kconfigwidgets.git/commit/src/kconfigdialogmanager.cpp?id=7db9744511373f74c28c0f40545677c9bd1b3b07

seems to solve the problem already


-- 
Best regards/Schöne Grüße

Martin
A: Because it breaks the logical sequence of discussion
Q: Why is top posting bad?

()  ascii ribbon campaign - against html e-mail 
/\                        - against proprietary attachments

Geschenkideen, Accessoires, Seifen, Kulinarisches: www.lillehus.at