diff options
author | Mike Kaganski <mike.kaganski@collabora.com> | 2020-12-18 13:12:50 +0300 |
---|---|---|
committer | Mike Kaganski <mike.kaganski@collabora.com> | 2020-12-23 07:41:00 +0100 |
commit | d0b5d0a7e6405742c0de746e7b740fd2547654cd (patch) | |
tree | a9478d1586eee12b389b8e847e553ab4937f933a /sw/source/filter/ww8/docxattributeoutput.cxx | |
parent | 88ac99d6afb1903c4f4a6a33b92fef461979703c (diff) |
tdf#138953: use original (cropped, but unrotated) object size in spPr
This not only fixes the regression from b226383a83e41bbced9fc2a02dc09a449401ec97,
but also makes the written size more correct than before, when it was
slightly larger compared to original object size.
Corrected unit test for tdf#116371 reflect that: the object in ODT is
241.78 mm x 240.61 mm. It previously was exported as 241.88 x 240.70;
now the exported size is closer: 241.79 x 240.63.
This backport introduces getShapes from 36e62098c8c541c4a3fb63eced591cf29ac56e4a
to make backporting easier.
Change-Id: Ibfe85c7cd98c089e58af8d7f3848990af8e1100f
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/107957
Tested-by: Jenkins
Reviewed-by: Mike Kaganski <mike.kaganski@collabora.com>
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/108227
Tested-by: Jenkins CollaboraOffice <jenkinscollaboraoffice@gmail.com>
Diffstat (limited to 'sw/source/filter/ww8/docxattributeoutput.cxx')
-rw-r--r-- | sw/source/filter/ww8/docxattributeoutput.cxx | 42 |
1 files changed, 27 insertions, 15 deletions
diff --git a/sw/source/filter/ww8/docxattributeoutput.cxx b/sw/source/filter/ww8/docxattributeoutput.cxx index 0e849c8142d7..7d614a206c7b 100644 --- a/sw/source/filter/ww8/docxattributeoutput.cxx +++ b/sw/source/filter/ww8/docxattributeoutput.cxx @@ -4764,13 +4764,12 @@ void DocxAttributeOutput::DefaultStyle() /* Writes <a:srcRect> tag back to document.xml if a file contains a cropped image. * NOTE : Tested on images of type JPEG,EMF/WMF,BMP, PNG and GIF. */ -void DocxAttributeOutput::WriteSrcRect(const SdrObject* pSdrObj, const SwFrameFormat* pFrameFormat ) +void DocxAttributeOutput::WriteSrcRect( + const css::uno::Reference<css::beans::XPropertySet>& xShapePropSet, + const SwFrameFormat* pFrameFormat) { - uno::Reference< drawing::XShape > xShape( const_cast<SdrObject*>(pSdrObj)->getUnoShape(), uno::UNO_QUERY ); - uno::Reference< beans::XPropertySet > xPropSet( xShape, uno::UNO_QUERY ); - uno::Reference<graphic::XGraphic> xGraphic; - xPropSet->getPropertyValue("Graphic") >>= xGraphic; + xShapePropSet->getPropertyValue("Graphic") >>= xGraphic; const Graphic aGraphic(xGraphic); Size aOriginalSize(aGraphic.GetPrefSize()); @@ -4783,7 +4782,7 @@ void DocxAttributeOutput::WriteSrcRect(const SdrObject* pSdrObj, const SwFrameFo } css::text::GraphicCrop aGraphicCropStruct; - xPropSet->getPropertyValue( "GraphicCrop" ) >>= aGraphicCropStruct; + xShapePropSet->getPropertyValue("GraphicCrop") >>= aGraphicCropStruct; sal_Int32 nCropL = aGraphicCropStruct.Left; sal_Int32 nCropR = aGraphicCropStruct.Right; sal_Int32 nCropT = aGraphicCropStruct.Top; @@ -4944,7 +4943,6 @@ void DocxAttributeOutput::FlyFrameGraphic( const SwGrfNode* pGrfNode, const Size rtl::Reference<sax_fastparser::FastAttributeList> xFrameAttributes( FastSerializerHelper::createAttrList()); - Size aSize = rSize; if (pGrfNode) { const SwAttrSet& rSet = pGrfNode->GetSwAttrSet(); @@ -4958,10 +4956,27 @@ void DocxAttributeOutput::FlyFrameGraphic( const SwGrfNode* pGrfNode, const Size // RES_GRFATR_ROTATION is in 10ths of degree; convert to 100ths for macro sal_uInt32 mOOXMLRot = oox::drawingml::ExportRotateClockwisify(nRot*10); xFrameAttributes->add(XML_rot, OString::number(mOOXMLRot)); - aSize = pGrfNode->GetTwipSize(); } } + css::uno::Reference<css::beans::XPropertySet> xShapePropSet; + if (pSdrObj) + { + css::uno::Reference<css::drawing::XShape> xShape( + const_cast<SdrObject*>(pSdrObj)->getUnoShape(), css::uno::UNO_QUERY); + xShapePropSet.set(xShape, css::uno::UNO_QUERY); + assert(xShapePropSet); + } + + Size aSize = rSize; + // We need the original (cropped, but unrotated) size of object. So prefer the object data, + // and only use passed frame size as fallback. + if (xShapePropSet) + { + if (css::awt::Size val; xShapePropSet->getPropertyValue("Size") >>= val) + aSize = Size(convertMm100ToTwip(val.Width), convertMm100ToTwip(val.Height)); + } + m_rExport.SdrExporter().startDMLAnchorInline(pFrameFormat, aSize); // picture description (used for pic:cNvPr later too) @@ -4975,11 +4990,9 @@ void DocxAttributeOutput::FlyFrameGraphic( const SwGrfNode* pGrfNode, const Size m_pSerializer->startElementNS( XML_wp, XML_docPr, docPrAttrListRef ); OUString sURL, sRelId; - if(pSdrObj) + if (xShapePropSet) { - uno::Reference< drawing::XShape > xShape( const_cast<SdrObject*>(pSdrObj)->getUnoShape(), uno::UNO_QUERY ); - uno::Reference< beans::XPropertySet > xPropSet( xShape, uno::UNO_QUERY ); - xPropSet->getPropertyValue("HyperLinkURL") >>= sURL; + xShapePropSet->getPropertyValue("HyperLinkURL") >>= sURL; if(!sURL.isEmpty()) { if (sURL.startsWith("#") && sURL.indexOf(' ') != -1 && !sURL.endsWith("|outline") && !sURL.endsWith("|table") && @@ -5060,9 +5073,8 @@ void DocxAttributeOutput::FlyFrameGraphic( const SwGrfNode* pGrfNode, const Size } m_pSerializer->endElementNS( XML_a, XML_blip ); - if (pSdrObj){ - WriteSrcRect(pSdrObj, pFrameFormat); - } + if (xShapePropSet) + WriteSrcRect(xShapePropSet, pFrameFormat); m_pSerializer->startElementNS(XML_a, XML_stretch); m_pSerializer->singleElementNS(XML_a, XML_fillRect); |