sw/inc/unodraw.hxx | 9 +--------
sw/source/core/unocore/unodraw.cxx | 28 +++++++++-------------------
2 files changed, 10 insertions(+), 27 deletions(-)
New commits:
commit 40babcfa637957bf7b59caa3cd12a630189e3e63
Author: Noel Grandin <noel.gran...@collabora.co.uk>
AuthorDate: Thu Apr 20 13:49:44 2023 +0200
Commit: Noel Grandin <noel.gran...@collabora.co.uk>
CommitDate: Thu Apr 20 15:05:02 2023 +0200
tdf#154827 and tdf#154428 graphics anchored in Writer as character..
..become anchored at paragraph when reopening the document
regression from
commit 8611f6e259b807b4f19c8dc0eab86ca648891ce3
ref-count SdrObject
This used to work by accident, previously, the SwXGroupShape that we
need would get created and destroyed and handful of times, and then
properly created.
Now, however, the first time we create it it sticks around, which means
it does not get the m_pFormat field configured correctly.
Fix this by removing the caching of m_pFormat from SwXShape, and just
fetching it from the model when we need it.
Change-Id: I10fe2ad0b877b3d928d5288e73cfed373b27a506
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/150687
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.gran...@collabora.co.uk>
diff --git a/sw/inc/unodraw.hxx b/sw/inc/unodraw.hxx
index 1f9efea10eac..cc3daaf49d69 100644
--- a/sw/inc/unodraw.hxx
+++ b/sw/inc/unodraw.hxx
@@ -132,7 +132,6 @@ class SwXShape : public SwXShapeBaseClass, public
SvtListener
friend class SwXGroupShape;
friend class SwFmDrawPage;
const SwFmDrawPage* m_pPage;
- SwFrameFormat* m_pFormat;
css::uno::Reference< css::uno::XAggregation > m_xShapeAgg;
// reference to <XShape>, determined in the
@@ -195,12 +194,6 @@ class SwXShape : public SwXShapeBaseClass, public
SvtListener
@throws css::uno::RuntimeException
*/
css::uno::Any _getPropAtAggrObj( const OUString& _rPropertyName );
- void SetFrameFormat(SwFrameFormat* pFormat)
- {
- EndListeningAll();
- StartListening(pFormat->GetNotifier());
- m_pFormat = pFormat;
- }
protected:
virtual ~SwXShape() override;
@@ -252,7 +245,7 @@ public:
virtual OUString SAL_CALL getShapeType( ) override;
SwShapeDescriptor_Impl* GetDescImpl() {return m_pImpl.get();}
- SwFrameFormat* GetFrameFormat() const { return m_pFormat; }
+ SwFrameFormat* GetFrameFormat() const;
const css::uno::Reference< css::uno::XAggregation >&
GetAggregationInterface() const {return m_xShapeAgg;}
// helper
diff --git a/sw/source/core/unocore/unodraw.cxx
b/sw/source/core/unocore/unodraw.cxx
index 7337ce297895..61e2649f212b 100644
--- a/sw/source/core/unocore/unodraw.cxx
+++ b/sw/source/core/unocore/unodraw.cxx
@@ -639,7 +639,7 @@ void SwFmDrawPage::add(const uno::Reference<
drawing::XShape > & xShape)
static_cast< cppu::OWeakObject * > ( this
) );
// we're already registered in the model / SwXDrawPage::add() already
called
- if(pShape->m_pPage || pShape->m_pFormat || !pShape->m_bDescriptor )
+ if(pShape->m_pPage || !pShape->m_bDescriptor )
return;
// we're inserted elsewhere already
@@ -787,7 +787,6 @@ void SwFmDrawPage::add(const uno::Reference<
drawing::XShape > & xShape)
{
pFormat->SetFormatName(pSvxShape->GetSdrObject()->GetName(),
false);
}
- pShape->SetFrameFormat(pFormat);
}
pShape->m_bDescriptor = false;
@@ -917,7 +916,6 @@ SwXShape::SwXShape(
uno::Reference<uno::XInterface> & xShape,
SwDoc const*const pDoc)
: m_pPage(nullptr)
- , m_pFormat(nullptr)
, m_pPropSet(aSwMapProvider.GetPropertySet(PROPERTY_MAP_TEXT_SHAPE))
,
m_pPropertyMapEntries(aSwMapProvider.GetPropertyMapEntries(PROPERTY_MAP_TEXT_SHAPE))
, m_pImpl(new SwShapeDescriptor_Impl(pDoc))
@@ -948,16 +946,20 @@ SwXShape::SwXShape(
SdrObject* pObj = SdrObject::getSdrObjectFromXShape(m_xShapeAgg);
if(pObj)
{
- auto pFormat = ::FindFrameFormat( pObj );
- if(pFormat)
- SetFrameFormat(pFormat);
-
lcl_addShapePropertyEventFactories( *pObj, *this );
m_pImpl->m_bInitializedPropertyNotifier = true;
}
}
+SwFrameFormat* SwXShape::GetFrameFormat() const
+{
+ SdrObject* pObj = SdrObject::getSdrObjectFromXShape(m_xShapeAgg);
+ if(pObj)
+ return ::FindFrameFormat( pObj );
+ return nullptr;
+}
+
void SwXShape::AddExistingShapeToFormat( SdrObject const & _rObj )
{
SdrObjListIter aIter( _rObj, SdrIterMode::DeepNoGroups );
@@ -972,12 +974,7 @@ void SwXShape::AddExistingShapeToFormat( SdrObject const &
_rObj )
if ( pSwShape )
{
if ( pSwShape->m_bDescriptor )
- {
- auto pFormat = ::FindFrameFormat( pCurrent );
- if ( pFormat )
- pSwShape->SetFrameFormat(pFormat);
pSwShape->m_bDescriptor = false;
- }
if ( !pSwShape->m_pImpl->m_bInitializedPropertyNotifier )
{
@@ -2091,7 +2088,6 @@ void SwXShape::Notify(const SfxHint& rHint)
{
if(rHint.GetId() == SfxHintId::Dying)
{
- m_pFormat = nullptr;
EndListeningAll();
}
}
@@ -2835,12 +2831,6 @@ void SwXGroupShape::add( const uno::Reference< XShape >&
xShape )
}
}
pSwShape->m_bDescriptor = false;
- //add the group member to the format of the group
- SwFrameFormat* pShapeFormat = ::FindFrameFormat( pSvxShape->GetSdrObject()
);
-
- if(pShapeFormat)
- pSwShape->SetFrameFormat(pShapeFormat);
-
}
void SwXGroupShape::remove( const uno::Reference< XShape >& xShape )
