Review Request: [CRASH FIX] fix Konquerer crashing with Calligra parts

Commit Hook null at kde.org
Tue Jul 3 20:13:58 BST 2012


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/105425/#review15347
-----------------------------------------------------------


This review has been submitted with commit 92e6bb6583b43eb4d3d552f497931f6303323faf by Friedrich W. H. Kossebau to branch master.

- Commit Hook


On July 3, 2012, 7:05 p.m., Friedrich W. H. Kossebau wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/105425/
> -----------------------------------------------------------
> 
> (Updated July 3, 2012, 7:05 p.m.)
> 
> 
> Review request for Calligra.
> 
> 
> Description
> -------
> 
> None of the Calligra parts uses Part::setWidget(...) or reimplements Part::widget().
> So all KPart supporting programs like Konqueror will crash when they try to get the widget from the part (and expect to get one).
> Just try yourself and do a Preview with... for some Calligra supported file.
> 
> Attached patch fixes that by simply removing the claim that CalligraParts are real KParts/ReadOnlyPart,KParts/ReadWritePart. By-effect is that sadly support for loading in Konqueror or KDevelop is gone, but well, effectively it already is. Hopefully the Parts->Doc/View refactoring from Boudewijn/Boemann can get us that back in the future?
> 
> Did not check, but guess this patch needs backporting to 2.5 as well.
> 
> 
> Diffs
> -----
> 
>   flow/part/flowpart.desktop fb17a3b 
>   karbon/data/karbonpart.desktop c5e0b0b 
>   krita/kritapart.desktop 00fe3bf 
>   plan/planpart.desktop dfd3d0f 
>   plugins/chartshape/kchartpart.desktop 027f628 
>   plugins/formulashape/kformulapart.desktop 1a91584 
>   servicetypes/calligrapart.desktop ee551d8 
>   sheets/sheetspart.desktop 5264778 
>   stage/part/stagepart.desktop e1f5ce6 
>   words/part/wordspart.desktop f0ca727 
> 
> Diff: http://git.reviewboard.kde.org/r/105425/diff/
> 
> 
> Testing
> -------
> 
> Tried to preview files with Konqueror & to load them with Calligra programs.
> 
> 
> Thanks,
> 
> Friedrich W. H. Kossebau
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/calligra-devel/attachments/20120703/f79530a8/attachment.htm>


More information about the calligra-devel mailing list