swunolocking1: #i105557#: fix locking for SwXFootnote:

add Impl struct, and use UnoImplPtr to lock destructor.
move all implementation details out of header, into Impl struct.
disable broken SwClient iteration.
make constructor protected, and add factory function CreateXFootnote.
use new XUnoTunnel and XServiceInfo helpers.
clean up the implementation a little.

5 files changed, 449 insertions, 333 deletions

diff --git a/sw/source/core/unocore/unoclbck.cxx b/sw/source/core/unocore/unoclbck.cxx
index 1d2185b381fb..6cc9cc0ee910 100644
--- a/sw/source/core/unocore/unoclbck.cxx
+++ b/sw/source/core/unocore/unoclbck.cxx
@@ -72,20 +72,7 @@ SwXReferenceMark* SwUnoCallBack::GetRefMark(const SwFmtRefMark& rMark)
  ---------------------------------------------------------------------------*/
 SwXFootnote* SwUnoCallBack::GetFootnote(const SwFmtFtn& rMark)
 {
-    SwClientIter aIter( *this );
-    SwXFootnote* pxFootnote = (SwXFootnote*)aIter.First( TYPE( SwXFootnote ));
-    while(pxFootnote)
-    {
-        SwDoc* pDoc = pxFootnote->GetDoc();
-        if(pDoc)
-        {
-            const SwFmtFtn* pFtn = pxFootnote->FindFmt();
-            if(pFtn == &rMark)
-                return pxFootnote;
-        }
-        pxFootnote = (SwXFootnote*)aIter.Next( );
-    }
-    return 0;
+    return SwXFootnote::GetXFootnote(*this, rMark);
 }
 
 /* -----------------------------27.11.00 17:15--------------------------------
diff --git a/sw/source/core/unocore/unocoll.cxx b/sw/source/core/unocore/unocoll.cxx
index 079ba1fa0b95..57f9b05a9c01 100644
--- a/sw/source/core/unocore/unocoll.cxx
+++ b/sw/source/core/unocore/unocoll.cxx
@@ -1727,8 +1727,8 @@ uno::Any SwXFootnotes::getByIndex(sal_Int32 nIndex)
 
             if(nCount == nIndex)
             {
-                xRef = new SwXFootnote(GetDoc(), rFtn);
-                aRet.setValue(&xRef, ::getCppuType((uno::Reference<XFootnote>*)0));
+                xRef = SwXFootnote::CreateXFootnote(*GetDoc(), rFtn);
+                aRet <<= xRef;
                 break;
             }
             nCount++;
@@ -1762,12 +1762,7 @@ sal_Bool SwXFootnotes::hasElements(void) throw( uno::RuntimeException )
  ---------------------------------------------------------------------------*/
 Reference<XFootnote>    SwXFootnotes::GetObject( SwDoc& rDoc, const SwFmtFtn& rFmt )
 {
-    Reference<XTextContent> xContent = ((SwUnoCallBack*)rDoc.GetUnoCallBack())->
-                                                            GetFootnote(rFmt);
-    if(!xContent.is())
-        xContent = new SwXFootnote(&rDoc, rFmt);
-    Reference<XFootnote> xRet(xContent, UNO_QUERY);
-    return xRet;
+    return SwXFootnote::CreateXFootnote(rDoc, rFmt);
 }
 
 /******************************************************************
diff --git a/sw/source/core/unocore/unocrsrhelper.cxx b/sw/source/core/unocore/unocrsrhelper.cxx
index 4d179f133446..29318ce49986 100644
--- a/sw/source/core/unocore/unocrsrhelper.cxx
+++ b/sw/source/core/unocore/unocrsrhelper.cxx
@@ -445,8 +445,9 @@ sal_Bool getCrsrPropertyValue(const SfxItemPropertySimpleEntry& rEntry
                 {
                     if( pAny )
                     {
-                        uno::Reference< XFootnote >  xFoot = new SwXFootnote(rPam.GetDoc(), rFtn);
-                        pAny->setValue(&xFoot, ::getCppuType((uno::Reference<XFootnote>*)0));
+                        const uno::Reference< text::XFootnote > xFootnote =
+                            SwXFootnote::CreateXFootnote(*rPam.GetDoc(), rFtn);
+                        *pAny <<= xFootnote;
                     }
                 }
                 else
diff --git a/sw/source/core/unocore/unoftn.cxx b/sw/source/core/unocore/unoftn.cxx
index 3d3937f56524..777ebb3e114f 100644
--- a/sw/source/core/unocore/unoftn.cxx
+++ b/sw/source/core/unocore/unoftn.cxx
@@ -31,9 +31,11 @@
 // MARKER(update_precomp.py): autogen include statement, do not remove
 #include "precompiled_sw.hxx"
 
+#include <rtl/uuid.h>
+
 #include <vos/mutex.hxx>
 #include <vcl/svapp.hxx>
-#include <rtl/uuid.h>
+#include <comphelper/sequence.hxx>
 
 #include <unomid.h>
 #include <unofootnote.hxx>
@@ -42,6 +44,7 @@
 #include <unoparagraph.hxx>
 #include <unomap.hxx>
 #include <unoprnms.hxx>
+#include <unoevtlstnr.hxx>
 #include <doc.hxx>
 #include <ftnidx.hxx>
 #include <fmtftn.hxx>
@@ -55,118 +58,233 @@ using namespace ::com::sun::star;
 using ::rtl::OUString;
 
 /******************************************************************
- *
+ * SwXFootnote
  ******************************************************************/
-/* -----------------------------13.03.00 12:15--------------------------------
 
- ---------------------------------------------------------------------------*/
-const uno::Sequence< sal_Int8 > & SwXFootnote::getUnoTunnelId()
+class SwXFootnote::Impl
+    : public SwClient
 {
-    static uno::Sequence< sal_Int8 > aSeq = ::CreateUnoTunnelId();
-    return aSeq;
-}
-/* -----------------------------10.03.00 18:04--------------------------------
 
- ---------------------------------------------------------------------------*/
-sal_Int64 SAL_CALL SwXFootnote::getSomething( const uno::Sequence< sal_Int8 >& rId )
-    throw(uno::RuntimeException)
-{
-    if( rId.getLength() == 16
-        && 0 == rtl_compareMemory( getUnoTunnelId().getConstArray(),
-                                        rId.getConstArray(), 16 ) )
+public:
+
+    SwXFootnote &               m_rThis;
+    const bool                  m_bIsEndnote;
+    SwEventListenerContainer    m_ListenerContainer;
+    bool                        m_bIsDescriptor;
+    const SwFmtFtn *            m_pFmtFtn;
+    ::rtl::OUString             m_sLabel;
+
+    Impl(   SwXFootnote & rThis,
+            SwDoc *const pDoc, SwFmtFtn const*const pFootnote,
+            const bool bIsEndnote)
+        : SwClient((pDoc) ? pDoc->GetUnoCallBack() : 0)
+        , m_rThis(rThis)
+        , m_bIsEndnote(bIsEndnote)
+        , m_ListenerContainer(static_cast< ::cppu::OWeakObject* >(&rThis))
+        , m_bIsDescriptor(0 == pFootnote)
+        , m_pFmtFtn(pFootnote)
     {
-        return sal::static_int_cast< sal_Int64 >( reinterpret_cast< sal_IntPtr >(this) );
     }
-    return SwXText::getSomething( rId );
-}
-/* -----------------------------06.04.00 16:36--------------------------------
+
+    const SwFmtFtn* GetFootnoteFormat() const {
+        return m_rThis.GetDoc() ? m_pFmtFtn : 0;
+    }
+
+    SwFmtFtn const& GetFootnoteFormatOrThrow() {
+        SwFmtFtn const*const pFootnote( GetFootnoteFormat() );
+        if (!pFootnote) {
+            throw uno::RuntimeException(OUString(RTL_CONSTASCII_USTRINGPARAM(
+                        "SwXFootnote: disposed or invalid")), 0);
+        }
+        return *pFootnote;
+    }
+
+    void    Invalidate();
+
+    // SwClient
+    virtual void    Modify(SfxPoolItem *pOld, SfxPoolItem *pNew);
+
+};
+
+/* -----------------------------07.01.00 12:39--------------------------------
 
  ---------------------------------------------------------------------------*/
-OUString SwXFootnote::getImplementationName(void) throw( uno::RuntimeException )
+void SwXFootnote::Impl::Invalidate()
 {
-    return C2U("SwXFootnote");
+    if (GetRegisteredIn())
+    {
+        const_cast<SwModify*>(GetRegisteredIn())->Remove(this);
+    }
+    m_ListenerContainer.Disposing();
+    m_pFmtFtn = 0;
+    m_rThis.SetDoc(0);
 }
-/* -----------------------------06.04.00 16:36--------------------------------
 
- ---------------------------------------------------------------------------*/
-BOOL SwXFootnote::supportsService(const OUString& rServiceName) throw( uno::RuntimeException )
+/* -----------------18.01.99 09:12-------------------
+ *
+ * --------------------------------------------------*/
+void SwXFootnote::Impl::Modify(SfxPoolItem *pOld, SfxPoolItem *pNew)
 {
-    return  !rServiceName.compareToAscii("com.sun.star.text.Footnote") ||
-            !rServiceName.compareToAscii("com.sun.star.text.TextContent") ||
-            !rServiceName.compareToAscii("com.sun.star.text.Text") ||
-            (m_bIsEndnote && !rServiceName.compareToAscii("com.sun.star.text.Endnote"));
-;
-}
-/* -----------------------------06.04.00 16:36--------------------------------
+    ClientModify(this, pOld, pNew);
 
- ---------------------------------------------------------------------------*/
-uno::Sequence< OUString > SwXFootnote::getSupportedServiceNames(void) throw( uno::RuntimeException )
-{
-    uno::Sequence< OUString > aRet(m_bIsEndnote ? 4 : 3);
-    OUString* pArray = aRet.getArray();
-    pArray[0] = C2U("com.sun.star.text.Footnote");
-    pArray[1] = C2U("com.sun.star.text.TextContent");
-    pArray[2] = C2U("com.sun.star.text.Text");
-    if(m_bIsEndnote)
-        pArray[3] = C2U("com.sun.star.text.Endnote");
-    return aRet;
+    if (!GetRegisteredIn()) // removed => dispose
+    {
+        Invalidate();
+    }
+    else if (pOld)
+    {
+        switch (pOld->Which())
+        {
+            case RES_FOOTNOTE_DELETED:
+                if (static_cast<const void*>(m_pFmtFtn) ==
+                        static_cast<SwPtrMsgPoolItem *>(pOld)->pObject)
+                {
+                    Invalidate();
+                }
+                break;
+        }
+    }
 }
+
 /*-- 10.12.98 15:31:44---------------------------------------------------
 
   -----------------------------------------------------------------------*/
-TYPEINIT1(SwXFootnote, SwClient);
-
-SwXFootnote::SwXFootnote(sal_Bool bEndnote) :
-    SwXText(0, CURSOR_FOOTNOTE),
-    aLstnrCntnr( (text::XTextContent*)this),
-    pFmtFtn(0),
-    m_bIsDescriptor(sal_True),
-    m_bIsEndnote(bEndnote)
+SwXFootnote::SwXFootnote(const bool bEndnote)
+    : SwXText(0, CURSOR_FOOTNOTE)
+    , m_pImpl( new SwXFootnote::Impl(*this, 0, 0, bEndnote) )
 {
-
 }
 /*-- 10.12.98 15:31:45---------------------------------------------------
 
   -----------------------------------------------------------------------*/
-SwXFootnote::SwXFootnote(SwDoc* _pDoc, const SwFmtFtn& rFmt) :
-    SwXText(_pDoc, CURSOR_FOOTNOTE),
-    aLstnrCntnr( (text::XTextContent*)this),
-    pFmtFtn(&rFmt),
-    m_bIsDescriptor(sal_False),
-    m_bIsEndnote(rFmt.IsEndNote())
+SwXFootnote::SwXFootnote(SwDoc & rDoc, const SwFmtFtn& rFmt)
+    : SwXText(& rDoc, CURSOR_FOOTNOTE)
+    , m_pImpl( new SwXFootnote::Impl(*this, &rDoc, &rFmt, rFmt.IsEndNote()) )
 {
-    GetDoc()->GetUnoCallBack()->Add(this);
 }
 /*-- 10.12.98 15:31:45---------------------------------------------------
 
   -----------------------------------------------------------------------*/
 SwXFootnote::~SwXFootnote()
 {
+}
+
+SwXFootnote *
+SwXFootnote::GetXFootnote(
+        SwModify const& /*rUnoCB*/, SwFmtFtn const& /*rFootnoteFmt*/)
+{
+    // re-use existing SwXFootnote
+    // #i105557#: do not iterate over the registered clients: race condition
+    // to do this properly requires the SwXFootnote to register at the
+    // SwFmtFtn directly, not at the unocallback
+    // also this function must return a uno Reference!
+#if 0
+    SwClientIter aIter( rUnoCB );
+    SwXFootnote::Impl * pXFootnote = static_cast<SwXFootnote::Impl*>(
+            aIter.First( TYPE( SwXFootnote::Impl )));
+    while (pXFootnote)
+    {
+        SwDoc *const pDoc = pXFootnote->m_rThis.GetDoc();
+        if (pDoc)
+        {
+            SwFmtFtn const*const pFtn = pXFootnote->GetFootnoteFormat();
+            if (pFtn == &rFootnoteFmt)
+            {
+                return & pXFootnote->m_rThis;
+            }
+        }
+        pXFootnote = static_cast<SwXFootnote::Impl*>(aIter.Next());
+    }
+#endif
+    return 0;
+}
+
+SwXFootnote *
+SwXFootnote::CreateXFootnote(SwDoc & rDoc, SwFmtFtn const& rFootnoteFmt)
+{
+    SwXFootnote *const pXFootnote(
+        GetXFootnote(*rDoc.GetUnoCallBack(), rFootnoteFmt));
+    return (pXFootnote)
+        ?   pXFootnote
+        :   new SwXFootnote(rDoc, rFootnoteFmt);
+}
 
+/* -----------------------------13.03.00 12:15--------------------------------
+
+ ---------------------------------------------------------------------------*/
+const uno::Sequence< sal_Int8 > & SwXFootnote::getUnoTunnelId()
+{
+    static uno::Sequence< sal_Int8 > aSeq = ::CreateUnoTunnelId();
+    return aSeq;
 }
-/* -----------------------------21.03.00 15:39--------------------------------
+/* -----------------------------10.03.00 18:04--------------------------------
+
+ ---------------------------------------------------------------------------*/
+sal_Int64 SAL_CALL
+SwXFootnote::getSomething(const uno::Sequence< sal_Int8 >& rId)
+throw (uno::RuntimeException)
+{
+    const sal_Int64 nRet( ::sw::UnoTunnelImpl<SwXFootnote>(rId, this) );
+    return (nRet) ? nRet : SwXText::getSomething(rId);
+}
+
+/* -----------------------------06.04.00 16:36--------------------------------
+
+ ---------------------------------------------------------------------------*/
+OUString SAL_CALL
+SwXFootnote::getImplementationName() throw (uno::RuntimeException)
+{
+    return C2U("SwXFootnote");
+}
+
+/* -----------------------------06.04.00 16:36--------------------------------
 
  ---------------------------------------------------------------------------*/
-uno::Sequence< uno::Type > SAL_CALL SwXFootnote::getTypes(  ) throw(uno::RuntimeException)
+static char const*const g_ServicesFootnote[] =
+{
+    "com.sun.star.text.TextContent",
+    "com.sun.star.text.Footnote",
+    "com.sun.star.text.Text",
+    "com.sun.star.text.Endnote", // NB: only supported for endnotes!
+};
+static const size_t g_nServicesEndnote(
+    sizeof(g_ServicesFootnote)/sizeof(g_ServicesFootnote[0]));
+static const size_t g_nServicesFootnote( g_nServicesEndnote - 1 ); // NB: omit!
+
+sal_Bool SAL_CALL SwXFootnote::supportsService(const OUString& rServiceName)
+throw (uno::RuntimeException)
 {
-    uno::Sequence< uno::Type > aFtnTypes = SwXFootnote_Base::getTypes();
-    uno::Sequence< uno::Type > aTextTypes = SwXText::getTypes();
+    vos::OGuard g(Application::GetSolarMutex());
+    return ::sw::SupportsServiceImpl(
+            (m_pImpl->m_bIsEndnote) ? g_nServicesEndnote : g_nServicesFootnote,
+            g_ServicesFootnote, rServiceName);
+}
 
-    long nIndex = aFtnTypes.getLength();
-    aFtnTypes.realloc(  aFtnTypes.getLength() + aTextTypes.getLength());
+uno::Sequence< OUString > SAL_CALL
+SwXFootnote::getSupportedServiceNames() throw (uno::RuntimeException)
+{
+    vos::OGuard g(Application::GetSolarMutex());
+    return ::sw::GetSupportedServiceNamesImpl(
+            (m_pImpl->m_bIsEndnote) ? g_nServicesEndnote : g_nServicesFootnote,
+            g_ServicesFootnote);
+}
 
-    uno::Type* pFtnTypes = aFtnTypes.getArray();
-    const uno::Type* pTextTypes = aTextTypes.getConstArray();
-    for(long nPos = 0; nPos < aTextTypes.getLength(); nPos++)
-        pFtnTypes[nIndex++] = pTextTypes[nPos];
+/* -----------------------------21.03.00 15:39--------------------------------
 
-    return aFtnTypes;
+ ---------------------------------------------------------------------------*/
+uno::Sequence< uno::Type > SAL_CALL
+SwXFootnote::getTypes() throw (uno::RuntimeException)
+{
+    const uno::Sequence< uno::Type > aTypes = SwXFootnote_Base::getTypes();
+    const uno::Sequence< uno::Type > aTextTypes = SwXText::getTypes();
+    return ::comphelper::concatSequences(aTypes, aTextTypes);
 }
 
 /* -----------------------------21.03.00 15:39--------------------------------
 
  ---------------------------------------------------------------------------*/
-uno::Sequence< sal_Int8 > SAL_CALL SwXFootnote::getImplementationId(  ) throw(uno::RuntimeException)
+uno::Sequence< sal_Int8 > SAL_CALL
+SwXFootnote::getImplementationId() throw (uno::RuntimeException)
 {
     vos::OGuard aGuard(Application::GetSolarMutex());
     static uno::Sequence< sal_Int8 > aId( 16 );
@@ -181,7 +299,8 @@ uno::Sequence< sal_Int8 > SAL_CALL SwXFootnote::getImplementationId(  ) throw(un
 /* -----------------------------21.03.00 15:46--------------------------------
 
  ---------------------------------------------------------------------------*/
-uno::Any SAL_CALL SwXFootnote::queryInterface(const uno::Type& rType)
+uno::Any SAL_CALL
+SwXFootnote::queryInterface(const uno::Type& rType)
 throw (uno::RuntimeException)
 {
     const uno::Any ret = SwXFootnote_Base::queryInterface(rType);
@@ -193,26 +312,36 @@ throw (uno::RuntimeException)
 /*-- 10.12.98 15:31:47---------------------------------------------------
 
   -----------------------------------------------------------------------*/
-OUString SwXFootnote::getLabel(void) throw( uno::RuntimeException )
+OUString SAL_CALL SwXFootnote::getLabel() throw (uno::RuntimeException)
 {
     vos::OGuard aGuard(Application::GetSolarMutex());
-    String sRet;
-    const SwFmtFtn*  pFmt = SwXFootnote::FindFmt();
+
+    ::rtl::OUString sRet;
+    SwFmtFtn const*const pFmt = m_pImpl->GetFootnoteFormat();
     if(pFmt)
+    {
         sRet = pFmt->GetNumStr();
-    else if(m_bIsDescriptor)
-        return m_sLabel;
+    }
+    else if (m_pImpl->m_bIsDescriptor)
+    {
+        sRet = m_pImpl->m_sLabel;
+    }
     else
+    {
         throw uno::RuntimeException();
+    }
     return sRet;
 }
+
 /*-- 10.12.98 15:31:48---------------------------------------------------
 
   -----------------------------------------------------------------------*/
-void SwXFootnote::setLabel(const OUString& aLabel) throw( uno::RuntimeException )
+void SAL_CALL
+SwXFootnote::setLabel(const OUString& aLabel) throw (uno::RuntimeException)
 {
     vos::OGuard aGuard(Application::GetSolarMutex());
-    const SwFmtFtn*  pFmt = FindFmt();
+
+    SwFmtFtn const*const pFmt = m_pImpl->GetFootnoteFormat();
     if(pFmt)
     {
         const SwTxtFtn* pTxtFtn = pFmt->GetTxtFtn();
@@ -222,157 +351,172 @@ void SwXFootnote::setLabel(const OUString& aLabel) throw( uno::RuntimeException
         SwPaM aPam(rTxtNode, *pTxtFtn->GetStart());
         GetDoc()->SetCurFtn(aPam, aLabel, pFmt->GetNumber(), pFmt->IsEndNote());
     }
-    else if(m_bIsDescriptor)
-        m_sLabel = String(aLabel);
+    else if (m_pImpl->m_bIsDescriptor)
+    {
+        m_pImpl->m_sLabel = String(aLabel);
+    }
     else
+    {
         throw uno::RuntimeException();
-
+    }
 }
+
 /* -----------------18.02.99 13:32-------------------
  *
  * --------------------------------------------------*/
-void SwXFootnote::attachToRange(const uno::Reference< text::XTextRange > & xTextRange)
-            throw( lang::IllegalArgumentException, uno::RuntimeException )
+void SAL_CALL
+SwXFootnote::attach(const uno::Reference< text::XTextRange > & xTextRange)
+throw (lang::IllegalArgumentException, uno::RuntimeException)
 {
-    if(!m_bIsDescriptor)
+    vos::OGuard aGuard(Application::GetSolarMutex());
+
+    if (!m_pImpl->m_bIsDescriptor)
+    {
         throw uno::RuntimeException();
-    uno::Reference<lang::XUnoTunnel> xRangeTunnel( xTextRange, uno::UNO_QUERY);
-    SwXTextRange* pRange = 0;
-    OTextCursorHelper* pCursor = 0;
-    if(xRangeTunnel.is())
+    }
+    const uno::Reference<lang::XUnoTunnel> xRangeTunnel(
+            xTextRange, uno::UNO_QUERY);
+    SwXTextRange *const pRange =
+        ::sw::UnoTunnelGetImplementation<SwXTextRange>(xRangeTunnel);
+    OTextCursorHelper *const pCursor =
+        ::sw::UnoTunnelGetImplementation<OTextCursorHelper>(xRangeTunnel);
+    SwDoc *const pNewDoc =
+        (pRange) ? pRange->GetDoc() : ((pCursor) ? pCursor->GetDoc() : 0);
+    if (!pNewDoc)
+    {
+        throw lang::IllegalArgumentException();
+    }
+
+    SwUnoInternalPaM aPam(*pNewDoc);
+    //das muss jetzt sal_True liefern
+    ::sw::XTextRangeToSwPaM(aPam, xTextRange);
+
+    UnoActionContext aCont(pNewDoc);
+    pNewDoc->DeleteAndJoin(aPam);
+    aPam.DeleteMark();
+    SwFmtFtn aFootNote(m_pImpl->m_bIsEndnote);
+    if (m_pImpl->m_sLabel.getLength())
     {
-        pRange  = reinterpret_cast< SwXTextRange * >(
-                sal::static_int_cast< sal_IntPtr >( xRangeTunnel->getSomething( SwXTextRange::getUnoTunnelId()) ));
-        pCursor = reinterpret_cast< OTextCursorHelper * >(
-                sal::static_int_cast< sal_IntPtr >( xRangeTunnel->getSomething( OTextCursorHelper::getUnoTunnelId()) ));
+        aFootNote.SetNumStr(m_pImpl->m_sLabel);
     }
-    SwDoc* pNewDoc = pRange ? (SwDoc*)pRange->GetDoc() : pCursor ? (SwDoc*)pCursor->GetDoc() : 0;
-    if(pNewDoc)
+
+    SwXTextCursor const*const pTextCursor(
+            dynamic_cast<SwXTextCursor*>(pCursor));
+    const bool bForceExpandHints( (pTextCursor)
+            ? pTextCursor->IsAtEndOfMeta() : false );
+    const SetAttrMode nInsertFlags = (bForceExpandHints)
+        ? nsSetAttrMode::SETATTR_FORCEHINTEXPAND
+        : nsSetAttrMode::SETATTR_DEFAULT;
+
+    pNewDoc->InsertPoolItem(aPam, aFootNote, nInsertFlags);
+
+    SwTxtFtn *const pTxtAttr = static_cast<SwTxtFtn*>(
+        aPam.GetNode()->GetTxtNode()->GetTxtAttrForCharAt(
+                aPam.GetPoint()->nContent.GetIndex()-1, RES_TXTATR_FTN ));
+
+    if (pTxtAttr)
     {
-        SwUnoInternalPaM aPam(*pNewDoc);
-        //das muss jetzt sal_True liefern
-        ::sw::XTextRangeToSwPaM(aPam, xTextRange);
-
-        UnoActionContext aCont(pNewDoc);
-        SwTxtAttr* pTxtAttr = 0;
-        pNewDoc->DeleteAndJoin(aPam);
-        aPam.DeleteMark();
-        SwFmtFtn aFootNote(m_bIsEndnote);
-        if(m_sLabel.Len())
-            aFootNote.SetNumStr(m_sLabel);
-
-        SwXTextCursor const*const pTextCursor(
-                dynamic_cast<SwXTextCursor*>(pCursor));
-        const bool bForceExpandHints( (pTextCursor)
-                ? pTextCursor->IsAtEndOfMeta() : false );
-        const SetAttrMode nInsertFlags = (bForceExpandHints)
-            ? nsSetAttrMode::SETATTR_FORCEHINTEXPAND
-            : nsSetAttrMode::SETATTR_DEFAULT;
-
-        pNewDoc->InsertPoolItem(aPam, aFootNote, nInsertFlags);
-
-        pTxtAttr = aPam.GetNode()->GetTxtNode()->GetTxtAttrForCharAt(
-                    aPam.GetPoint()->nContent.GetIndex()-1, RES_TXTATR_FTN );
-
-        if(pTxtAttr)
+        const SwFmtFtn& rFtn = pTxtAttr->GetFtn();
+        m_pImpl->m_pFmtFtn = &rFtn;
+        pNewDoc->GetUnoCallBack()->Add(m_pImpl.get());
+        // force creation of sequence id - is used for references
+        if (pNewDoc->IsInReading())
+        {
+            pTxtAttr->SetSeqNo(pNewDoc->GetFtnIdxs().Count());
+        }
+        else
         {
-            const SwFmtFtn& rFtn = pTxtAttr->GetFtn();
-            pFmtFtn = &rFtn;
-            pNewDoc->GetUnoCallBack()->Add(this);
-            //force creation of sequence id - is used for references
-            if(pNewDoc->IsInReading())
-                ((SwTxtFtn*)pTxtAttr)->SetSeqNo(pNewDoc->GetFtnIdxs().Count());
-            else
-                ((SwTxtFtn*)pTxtAttr)->SetSeqRefNo();
+            pTxtAttr->SetSeqRefNo();
         }
-        m_bIsDescriptor = sal_False;
-        SetDoc(pNewDoc);
     }
-    else
-        throw lang::IllegalArgumentException();
+    m_pImpl->m_bIsDescriptor = sal_False;
+    SetDoc(pNewDoc);
 }
-/*-- 10.12.98 15:31:48---------------------------------------------------
 
-  -----------------------------------------------------------------------*/
-void SwXFootnote::attach(const uno::Reference< text::XTextRange > & xTextRange)
-            throw( lang::IllegalArgumentException, uno::RuntimeException )
-{
-    vos::OGuard aGuard(Application::GetSolarMutex());
-    attachToRange( xTextRange );
-}
 /*-- 10.12.98 15:31:48---------------------------------------------------
 
   -----------------------------------------------------------------------*/
-uno::Reference< text::XTextRange >  SwXFootnote::getAnchor(void) throw( uno::RuntimeException )
+uno::Reference< text::XTextRange > SAL_CALL
+SwXFootnote::getAnchor() throw (uno::RuntimeException)
 {
     vos::OGuard aGuard(Application::GetSolarMutex());
-    uno::Reference< text::XTextRange >  aRef;
-    const SwFmtFtn*  pFmt = FindFmt();
-    if(pFmt)
-    {
-        const SwTxtFtn* pTxtFtn = pFmt->GetTxtFtn();
-        SwPaM aPam( pTxtFtn->GetTxtNode(), *pTxtFtn->GetStart() );
-        SwPosition aMark( *aPam.Start() );
-        aPam.SetMark();
-        aPam.GetMark()->nContent++;
-        aRef = SwXTextRange::CreateXTextRange(
-                *GetDoc(), *aPam.Start(), aPam.End());
-    }
-    else
-        throw uno::RuntimeException();
-    return aRef;
+
+    SwFmtFtn const& rFmt( m_pImpl->GetFootnoteFormatOrThrow() );
+
+    SwTxtFtn const*const pTxtFtn = rFmt.GetTxtFtn();
+    SwPaM aPam( pTxtFtn->GetTxtNode(), *pTxtFtn->GetStart() );
+    SwPosition aMark( *aPam.Start() );
+    aPam.SetMark();
+    aPam.GetMark()->nContent++;
+    const uno::Reference< text::XTextRange > xRet =
+        SwXTextRange::CreateXTextRange(*GetDoc(), *aPam.Start(), aPam.End());
+    return xRet;
 }
 /*-- 10.12.98 15:31:49---------------------------------------------------
 
   -----------------------------------------------------------------------*/
-void SwXFootnote::dispose(void) throw( uno::RuntimeException )
+void SAL_CALL SwXFootnote::dispose() throw (uno::RuntimeException)
 {
     vos::OGuard aGuard(Application::GetSolarMutex());
-    const SwFmtFtn*  pFmt = FindFmt();
-    if(pFmt)
-    {
-        const SwTxtFtn* pTxtFtn = pFmt->GetTxtFtn();
-        DBG_ASSERT(pTxtFtn, "kein TextNode?");
-        SwTxtNode& rTxtNode = (SwTxtNode&)pTxtFtn->GetTxtNode();
-        xub_StrLen nPos = *pTxtFtn->GetStart();
-        SwPaM aPam(rTxtNode, nPos, rTxtNode, nPos+1 );
-        GetDoc()->DeleteAndJoin( aPam );
-    }
-    else
-        throw uno::RuntimeException();
 
+    SwFmtFtn const& rFmt( m_pImpl->GetFootnoteFormatOrThrow() );
+
+    SwTxtFtn const*const pTxtFtn = rFmt.GetTxtFtn();
+    DBG_ASSERT(pTxtFtn, "no TextNode?");
+    SwTxtNode& rTxtNode = const_cast<SwTxtNode&>(pTxtFtn->GetTxtNode());
+    const xub_StrLen nPos = *pTxtFtn->GetStart();
+    SwPaM aPam(rTxtNode, nPos, rTxtNode, nPos+1);
+    GetDoc()->DeleteAndJoin( aPam );
 }
+
 /*-- 10.12.98 15:31:49---------------------------------------------------
 
   -----------------------------------------------------------------------*/
-void SwXFootnote::addEventListener(const uno::Reference< lang::XEventListener > & aListener) throw( uno::RuntimeException )
+void SAL_CALL
+SwXFootnote::addEventListener(
+    const uno::Reference< lang::XEventListener > & xListener)
+throw (uno::RuntimeException)
 {
-    if(!GetRegisteredIn())
+    vos::OGuard g(Application::GetSolarMutex());
+
+    if (!m_pImpl->GetFootnoteFormat())
+    {
         throw uno::RuntimeException();
-    aLstnrCntnr.AddListener(aListener);
+    }
+    m_pImpl->m_ListenerContainer.AddListener(xListener);
 }
 /*-- 10.12.98 15:31:50---------------------------------------------------
 
   -----------------------------------------------------------------------*/
-void SwXFootnote::removeEventListener(const uno::Reference< lang::XEventListener > & aListener) throw( uno::RuntimeException )
+void SAL_CALL
+SwXFootnote::removeEventListener(
+    const uno::Reference< lang::XEventListener > & xListener)
+throw (uno::RuntimeException)
 {
-    if(!GetRegisteredIn() || !aLstnrCntnr.RemoveListener(aListener))
+    vos::OGuard g(Application::GetSolarMutex());
+
+    if (!m_pImpl->GetFootnoteFormat() ||
+        !m_pImpl->m_ListenerContainer.RemoveListener(xListener))
+    {
         throw uno::RuntimeException();
+    }
 }
+
 /* -----------------06.05.99 15:31-------------------
  *
  * --------------------------------------------------*/
 const SwStartNode *SwXFootnote::GetStartNode() const
 {
-    const SwStartNode *pSttNd = 0;
-    const SwFmtFtn*  pFmt = FindFmt();
+    SwFmtFtn const*const   pFmt = m_pImpl->GetFootnoteFormat();
     if(pFmt)
     {
         const SwTxtFtn* pTxtFtn = pFmt->GetTxtFtn();
         if( pTxtFtn )
-            pSttNd = pTxtFtn->GetStartNode()->GetNode().GetStartNode();
+        {
+            return pTxtFtn->GetStartNode()->GetNode().GetStartNode();
+        }
     }
-    return pSttNd;
+    return 0;
 }
 
 uno::Reference< text::XTextCursor >
@@ -380,144 +524,119 @@ SwXFootnote::CreateCursor() throw (uno::RuntimeException)
 {
     return createTextCursor();
 }
+
 /*-- 10.12.98 15:31:50---------------------------------------------------
 
   -----------------------------------------------------------------------*/
-uno::Reference< text::XTextCursor >  SwXFootnote::createTextCursor(void) throw( uno::RuntimeException )
+uno::Reference< text::XTextCursor > SAL_CALL
+SwXFootnote::createTextCursor() throw (uno::RuntimeException)
 {
     vos::OGuard aGuard(Application::GetSolarMutex());
-    uno::Reference< text::XTextCursor >  aRef;
-    const SwFmtFtn*  pFmt = FindFmt();
-    if(pFmt)
-    {
-        const SwTxtFtn* pTxtFtn = pFmt->GetTxtFtn();
-        SwPosition aPos( *pTxtFtn->GetStartNode() );
-        SwXTextCursor *const pXCursor =
-            new SwXTextCursor(*GetDoc(), this, CURSOR_FOOTNOTE, aPos);
-        aRef = static_cast<text::XWordCursor*>(pXCursor);
-        SwUnoCrsr *const pUnoCrsr = pXCursor->GetCursor();
-        pUnoCrsr->Move(fnMoveForward, fnGoNode);
-    }
-    else
-        throw uno::RuntimeException();
-    return aRef;
+
+    SwFmtFtn const& rFmt( m_pImpl->GetFootnoteFormatOrThrow() );
+
+    SwTxtFtn const*const pTxtFtn = rFmt.GetTxtFtn();
+    SwPosition aPos( *pTxtFtn->GetStartNode() );
+    SwXTextCursor *const pXCursor =
+        new SwXTextCursor(*GetDoc(), this, CURSOR_FOOTNOTE, aPos);
+    SwUnoCrsr *const pUnoCrsr = pXCursor->GetCursor();
+    pUnoCrsr->Move(fnMoveForward, fnGoNode);
+    const uno::Reference< text::XTextCursor > xRet =
+        static_cast<text::XWordCursor*>(pXCursor);
+    return xRet;
 }
+
 /*-- 10.12.98 15:31:51---------------------------------------------------
 
   -----------------------------------------------------------------------*/
-uno::Reference< text::XTextCursor >  SwXFootnote::createTextCursorByRange(
-    const uno::Reference< text::XTextRange > & aTextPosition)
-            throw( uno::RuntimeException )
+uno::Reference< text::XTextCursor > SAL_CALL
+SwXFootnote::createTextCursorByRange(
+    const uno::Reference< text::XTextRange > & xTextPosition)
+throw (uno::RuntimeException)
 {
     vos::OGuard aGuard(Application::GetSolarMutex());
-    const SwFmtFtn*  pFmt = FindFmt();
-    if( !pFmt )
-        throw uno::RuntimeException();
-    uno::Reference< text::XTextCursor >  aRef;
+
+    SwFmtFtn const& rFmt( m_pImpl->GetFootnoteFormatOrThrow() );
+
     SwUnoInternalPaM aPam(*GetDoc());
-    if (::sw::XTextRangeToSwPaM(aPam, aTextPosition))
+    if (!::sw::XTextRangeToSwPaM(aPam, xTextPosition))
     {
-        const SwTxtFtn* pTxtFtn = pFmt->GetTxtFtn();
-        const SwNode* pFtnStartNode = &pTxtFtn->GetStartNode()->GetNode();
+        throw uno::RuntimeException();
+    }
 
-        const SwNode* pStart = aPam.GetNode()->FindFootnoteStartNode();
-        if( pStart != pFtnStartNode )
-            throw uno::RuntimeException();
+    SwTxtFtn const*const pTxtFtn = rFmt.GetTxtFtn();
+    SwNode const*const pFtnStartNode = &pTxtFtn->GetStartNode()->GetNode();
 
-        aRef = static_cast<text::XWordCursor*>(
+    const SwNode* pStart = aPam.GetNode()->FindFootnoteStartNode();
+    if (pStart != pFtnStartNode)
+    {
+        throw uno::RuntimeException();
+    }
+
+    const uno::Reference< text::XTextCursor > xRet =
+        static_cast<text::XWordCursor*>(
                 new SwXTextCursor(*GetDoc(), this, CURSOR_FOOTNOTE,
                     *aPam.GetPoint(), aPam.GetMark()));
-    }
-    else
-        throw uno::RuntimeException();
-    return aRef;
+    return xRet;
 }
+
 /*-- 13.06.00 14:28:23---------------------------------------------------
 
   -----------------------------------------------------------------------*/
-uno::Reference< container::XEnumeration >  SwXFootnote::createEnumeration() throw( uno::RuntimeException )
+uno::Reference< container::XEnumeration > SAL_CALL
+SwXFootnote::createEnumeration() throw (uno::RuntimeException)
 {
     vos::OGuard aGuard(Application::GetSolarMutex());
-    uno::Reference< container::XEnumeration >  aRef;
-    const SwFmtFtn*  pFmt = FindFmt();
-    if(pFmt)
-    {
-        const SwTxtFtn* pTxtFtn = pFmt->GetTxtFtn();
-        SwPosition aPos( *pTxtFtn->GetStartNode() );
-        ::std::auto_ptr<SwUnoCrsr> pUnoCursor(
-            GetDoc()->CreateUnoCrsr(aPos, sal_False));
-        pUnoCursor->Move(fnMoveForward, fnGoNode);
-        aRef = new SwXParagraphEnumeration(this, pUnoCursor, CURSOR_FOOTNOTE);
-    }
-    return aRef;
+
+    SwFmtFtn const& rFmt( m_pImpl->GetFootnoteFormatOrThrow() );
+
+    SwTxtFtn const*const pTxtFtn = rFmt.GetTxtFtn();
+    SwPosition aPos( *pTxtFtn->GetStartNode() );
+    ::std::auto_ptr<SwUnoCrsr> pUnoCursor(
+        GetDoc()->CreateUnoCrsr(aPos, sal_False));
+    pUnoCursor->Move(fnMoveForward, fnGoNode);
+    const uno::Reference< container::XEnumeration >  xRet =
+        new SwXParagraphEnumeration(this, pUnoCursor, CURSOR_FOOTNOTE);
+    return xRet;
 }
+
 /*-- 13.06.00 14:28:24---------------------------------------------------
 
   -----------------------------------------------------------------------*/
-uno::Type SwXFootnote::getElementType(  ) throw(uno::RuntimeException)
+uno::Type SAL_CALL SwXFootnote::getElementType() throw (uno::RuntimeException)
 {
-    return ::getCppuType(static_cast<uno::Reference<XTextRange>*>(0));
+    return text::XTextRange::static_type();
 }
 /*-- 13.06.00 14:28:24---------------------------------------------------
 
   -----------------------------------------------------------------------*/
-sal_Bool SwXFootnote::hasElements(  ) throw(uno::RuntimeException)
+sal_Bool SAL_CALL SwXFootnote::hasElements() throw (uno::RuntimeException)
 {
     return sal_True;
 }
-/* -----------------------------07.01.00 12:39--------------------------------
 
- ---------------------------------------------------------------------------*/
-void SwXFootnote::Invalidate()
-{
-    if(GetRegisteredIn())
-    {
-        ((SwModify*)GetRegisteredIn())->Remove(this);
-        pFmtFtn = 0;
-        aLstnrCntnr.Disposing();
-        SetDoc(0);
-    }
-}
-/* -----------------18.01.99 09:12-------------------
- *
- * --------------------------------------------------*/
-void SwXFootnote::Modify( SfxPoolItem *pOld, SfxPoolItem *pNew)
-{
-    switch( pOld ? pOld->Which() : 0 )
-    {
-    case RES_REMOVE_UNO_OBJECT:
-    case RES_OBJECTDYING:
-        if( (void*)GetRegisteredIn() == ((SwPtrMsgPoolItem *)pOld)->pObject )
-            Invalidate();
-        break;
-    case RES_FMT_CHG:
-        // wurden wir an das neue umgehaengt und wird das alte geloscht?
-        if( ((SwFmtChg*)pNew)->pChangedFmt == GetRegisteredIn() &&
-            ((SwFmtChg*)pOld)->pChangedFmt->IsFmtInDTOR() )
-            Invalidate();
-        break;
-    case RES_FOOTNOTE_DELETED:
-        if( (void*)pFmtFtn == ((SwPtrMsgPoolItem *)pOld)->pObject )
-            Invalidate();
-        break;
-    }
-}
 /*-- 11.09.00 13:12:03---------------------------------------------------
 
   -----------------------------------------------------------------------*/
-uno::Reference< beans::XPropertySetInfo > SwXFootnote::getPropertySetInfo(  )
-    throw(uno::RuntimeException)
+uno::Reference< beans::XPropertySetInfo > SAL_CALL
+SwXFootnote::getPropertySetInfo()
+throw (uno::RuntimeException)
 {
-    static uno::Reference< beans::XPropertySetInfo >  xRef = aSwMapProvider.GetPropertySet(PROPERTY_MAP_FOOTNOTE)->getPropertySetInfo();
-    return xRef;
+    vos::OGuard g(Application::GetSolarMutex());
+    static uno::Reference< beans::XPropertySetInfo > xRet =
+        aSwMapProvider.GetPropertySet(PROPERTY_MAP_FOOTNOTE)
+            ->getPropertySetInfo();
+    return xRet;
 }
+
 /*-- 11.09.00 13:12:04---------------------------------------------------
 
   -----------------------------------------------------------------------*/
-void SwXFootnote::setPropertyValue( const ::rtl::OUString&,
-    const uno::Any& )
-        throw(beans::UnknownPropertyException, beans::PropertyVetoException,
-            lang::IllegalArgumentException, lang::WrappedTargetException, uno::RuntimeException)
+void SAL_CALL
+SwXFootnote::setPropertyValue(const ::rtl::OUString&, const uno::Any&)
+throw (beans::UnknownPropertyException, beans::PropertyVetoException,
+        lang::IllegalArgumentException, lang::WrappedTargetException,
+        uno::RuntimeException)
 {
     //no values to be set
     throw lang::IllegalArgumentException();
@@ -525,28 +644,34 @@ void SwXFootnote::setPropertyValue( const ::rtl::OUString&,
 /*-- 11.09.00 13:12:04---------------------------------------------------
 
   -----------------------------------------------------------------------*/
-uno::Any SwXFootnote::getPropertyValue( const OUString& rPropertyName )
-    throw(beans::UnknownPropertyException, lang::WrappedTargetException, uno::RuntimeException)
+uno::Any SAL_CALL
+SwXFootnote::getPropertyValue(const OUString& rPropertyName)
+throw (beans::UnknownPropertyException, lang::WrappedTargetException,
+        uno::RuntimeException)
 {
     vos::OGuard aGuard(Application::GetSolarMutex());
+
     uno::Any aRet;
     if (! ::sw::GetDefaultTextContentValue(aRet, rPropertyName))
     {
-        if(rPropertyName.equalsAsciiL(SW_PROP_NAME(UNO_NAME_START_REDLINE))||
-                rPropertyName.equalsAsciiL(SW_PROP_NAME(UNO_NAME_END_REDLINE)))
+        if (rPropertyName.equalsAsciiL(SW_PROP_NAME(UNO_NAME_START_REDLINE)) ||
+            rPropertyName.equalsAsciiL(SW_PROP_NAME(UNO_NAME_END_REDLINE)))
         {
             //redline can only be returned if it's a living object
-            if(!m_bIsDescriptor)
+            if (!m_pImpl->m_bIsDescriptor)
+            {
                 aRet = SwXText::getPropertyValue(rPropertyName);
+            }
         }
-        else if(rPropertyName.equalsAsciiL( SW_PROP_NAME(UNO_NAME_REFERENCE_ID)))
+        else if (rPropertyName.equalsAsciiL(
+                    SW_PROP_NAME(UNO_NAME_REFERENCE_ID)))
         {
-            const SwFmtFtn*  pFmt = FindFmt();
-            if(pFmt)
+            SwFmtFtn const*const pFmt = m_pImpl->GetFootnoteFormat();
+            if (pFmt)
             {
-                const SwTxtFtn* pTxtFtn = pFmt->GetTxtFtn();
+                SwTxtFtn const*const pTxtFtn = pFmt->GetTxtFtn();
                 DBG_ASSERT(pTxtFtn, "no TextNode?");
-                aRet <<= (sal_Int16)pTxtFtn->GetSeqRefNo();
+                aRet <<= static_cast<sal_Int16>(pTxtFtn->GetSeqRefNo());
             }
         }
         else
@@ -558,37 +683,51 @@ uno::Any SwXFootnote::getPropertyValue( const OUString& rPropertyName )
     }
     return aRet;
 }
+
 /*-- 11.09.00 13:12:04---------------------------------------------------
 
   -----------------------------------------------------------------------*/
-void SwXFootnote::addPropertyChangeListener( const OUString& /*aPropertyName*/,
-    const uno::Reference< beans::XPropertyChangeListener >& /*xListener*/ )
-        throw(beans::UnknownPropertyException, lang::WrappedTargetException, uno::RuntimeException)
+void SAL_CALL
+SwXFootnote::addPropertyChangeListener(
+        const ::rtl::OUString& /*rPropertyName*/,
+        const uno::Reference< beans::XPropertyChangeListener >& /*xListener*/)
+throw (beans::UnknownPropertyException, lang::WrappedTargetException,
+    uno::RuntimeException)
 {
+    OSL_ENSURE(false,
+        "SwXFootnote::addPropertyChangeListener(): not implemented");
 }
-/*-- 11.09.00 13:12:04---------------------------------------------------
 
-  -----------------------------------------------------------------------*/
-void SwXFootnote::removePropertyChangeListener( const OUString& /*aPropertyName*/,
-    const uno::Reference< beans::XPropertyChangeListener >& /*aListener*/ )
-        throw(beans::UnknownPropertyException, lang::WrappedTargetException, uno::RuntimeException)
+void SAL_CALL
+SwXFootnote::removePropertyChangeListener(
+        const ::rtl::OUString& /*rPropertyName*/,
+        const uno::Reference< beans::XPropertyChangeListener >& /*xListener*/)
+throw (beans::UnknownPropertyException, lang::WrappedTargetException,
+    uno::RuntimeException)
 {
+    OSL_ENSURE(false,
+        "SwXFootnote::removePropertyChangeListener(): not implemented");
 }
-/*-- 11.09.00 13:12:04---------------------------------------------------
 
-  -----------------------------------------------------------------------*/
-void SwXFootnote::addVetoableChangeListener( const OUString& /*PropertyName*/,
-    const uno::Reference< beans::XVetoableChangeListener >& /*aListener*/ )
-        throw(beans::UnknownPropertyException, lang::WrappedTargetException, uno::RuntimeException)
+void SAL_CALL
+SwXFootnote::addVetoableChangeListener(
+        const ::rtl::OUString& /*rPropertyName*/,
+        const uno::Reference< beans::XVetoableChangeListener >& /*xListener*/)
+throw (beans::UnknownPropertyException, lang::WrappedTargetException,
+    uno::RuntimeException)
 {
+    OSL_ENSURE(false,
+        "SwXFootnote::addVetoableChangeListener(): not implemented");
 }
-/*-- 11.09.00 13:12:05---------------------------------------------------
 
-  -----------------------------------------------------------------------*/
-void SwXFootnote::removeVetoableChangeListener( const OUString& /*PropertyName*/,
-    const uno::Reference< beans::XVetoableChangeListener >& /*aListener*/ )
-        throw(beans::UnknownPropertyException, lang::WrappedTargetException, uno::RuntimeException)
+void SAL_CALL
+SwXFootnote::removeVetoableChangeListener(
+        const ::rtl::OUString& /*rPropertyName*/,
+        const uno::Reference< beans::XVetoableChangeListener >& /*xListener*/)
+throw (beans::UnknownPropertyException, lang::WrappedTargetException,
+        uno::RuntimeException)
 {
+    OSL_ENSURE(false,
+        "SwXFootnote::removeVetoableChangeListener(): not implemented");
 }
 
-
diff --git a/sw/source/core/unocore/unoobj2.cxx b/sw/source/core/unocore/unoobj2.cxx
index 918ab49d45f0..53b0da94b12d 100644
--- a/sw/source/core/unocore/unoobj2.cxx
+++ b/sw/source/core/unocore/unoobj2.cxx
@@ -1344,13 +1344,7 @@ CreateParentXText(SwDoc & rDoc, const SwPosition& rPos)
                 if (pSttNode == pTxtFtn->GetStartNode()->GetNode().
                                     FindSttNodeByType(SwFootnoteStartNode))
                 {
-                    xParentText =
-                        static_cast<SwUnoCallBack*>(rDoc.GetUnoCallBack())->
-                                                            GetFootnote(rFtn);
-                    if (!xParentText.is())
-                    {
-                        xParentText = new SwXFootnote(&rDoc, rFtn);
-                    }
+                    xParentText = SwXFootnote::CreateXFootnote(rDoc, rFtn);
                     break;
                 }
             }