-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/113664/
-----------------------------------------------------------
Review request for Calligra and Boudewijn Rempt.
Repository: calligra
Description
-------
Fixes crash when semantic items are inserted after the dialog incl. the rdf
page was shown & closed
Seems that KoDocumentInfoDlg is slightly flawed after the port to KPageDialog :/
For one KPageWidgetItem* are collected in d->pages ("item" vs. "page"), which
then are tried to be casted to KoPageWidgetItem*, which is bogus because
KoPageWidgetItem is not in any relationship with KPageWidgetItem anymore.
The initial plan was to make KoPageWidgetItem simply a subclass of
KPageWidgetItem. But that has the problem that KPageWidgetItem expects its
mainwidget to be passed in the constructor call, so in constructor code of
subclasses right here: KoPageWidgetItem::KoPageWidgetItem(...) :
KPageWidgetItem(widget) {...}.
Means, subclasses which would like to be complete page items setting the
widget themselves (like the KoDocumentRdfEditWidget) would have to do some
tricks to get their widget already available on call to the baseclass
constructor.
Somehow messy.
So decided to just do a quick fix for now, by adding a simple adapter/wrapper
class, which makes sure that the KoPageWidgetItem is deleted on destruction and
that any calls on the KoPageWidgetItem API after casting will happen actually
again.
Hopefully this dialog will be redone in calligra 3.0 (qt5/kf5) properly. Or
KoDocumentRdfEditWidget might get a refactoring and things could be done
differently, as that is the only calligra-internal page-extension currently.
Okay to also backport to 2.7?
Diffs
-----
Diff: http://git.reviewboard.kde.org/r/113664/diff/
Testing
-------
No more crashes happening when semantic items are inserted after the dialog
incl. the rdf page was shown & closed
Thanks,
Friedrich W. H. Kossebau
_______________________________________________
calligra-devel mailing list
[email protected]
https://mail.kde.org/mailman/listinfo/calligra-devel
