Review Request: Fix missing images in excel filter
Lukáš Tvrdý
lukast.dev at gmail.com
Fri Nov 4 09:16:40 GMT 2011
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/103044/
-----------------------------------------------------------
Review request for Calligra, Marijn Kruisselbrink and Sebastian Sauer.
Description
-------
This patch uses more of the libmso in excel filter.
So far the pictures were handled as PictureObject class. One problem was that
there was no support for groups for this object. So when you put two images into groups, they
are not displayed because the anchoring information is saved in different data-structure - in the
group structure.
Now I let the records to be handled by libmso, which provides this functionality.
It supports groups and even graphical styles like like borders etc.
This addresses bug 262865.
http://bugs.kde.org/show_bug.cgi?id=262865
Diffs
-----
filters/libmso/pictures.cpp f8bfa7c
filters/tables/excel/import/ExcelImport.cpp 1bb7f17
filters/tables/excel/import/ODrawClient.cpp 5a6507b
filters/tables/excel/import/excelimporttoods.cc 287aef7
filters/tables/excel/sidewinder/cell.h bc080f9
filters/tables/excel/sidewinder/cell.cpp 8b8c0e1
filters/tables/excel/sidewinder/excel.h 2be1f5a
filters/tables/excel/sidewinder/excel.cpp b475e5a
filters/tables/excel/sidewinder/globalssubstreamhandler.h 77c6e3a
filters/tables/excel/sidewinder/globalssubstreamhandler.cpp f9c03b1
filters/tables/excel/sidewinder/objects.h 2412f07
filters/tables/excel/sidewinder/sheet.h 9ff346d
filters/tables/excel/sidewinder/sheet.cpp 440f44b
filters/tables/excel/sidewinder/workbook.h 2e3c2fa
filters/tables/excel/sidewinder/workbook.cpp 9ad61bc
filters/tables/excel/sidewinder/worksheetsubstreamhandler.cpp 879f76f
Diff: http://git.reviewboard.kde.org/r/103044/diff/diff
Testing
-------
I did regression testing against master. My testing set consists of 976 xls files.
I found only positive changes.
Missing frames around pictures are displayed now, missing images are displayed now.
You can test the document.
Thanks,
Lukáš Tvrdý
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/calligra-devel/attachments/20111104/b17dd90f/attachment.htm>
More information about the calligra-devel
mailing list