From 8d760d684e4ed08be8c4b0a338f52cd7d17db590 Mon Sep 17 00:00:00 2001
From: Oliver Specht <os@openoffice.org>
Date: Tue, 5 Jun 2001 07:24:50 +0000
Subject: [PATCH] #87739# SwXDrawPage::add(): prevent adding of already
 inserted objects

---
 sw/inc/unodraw.hxx                 |  5 +++--
 sw/source/core/unocore/unodraw.cxx | 16 ++++++++++++----
 2 files changed, 15 insertions(+), 6 deletions(-)

diff --git a/sw/inc/unodraw.hxx b/sw/inc/unodraw.hxx
index 95e414b5e5aa..41b580b9708a 100644
--- a/sw/inc/unodraw.hxx
+++ b/sw/inc/unodraw.hxx
@@ -2,9 +2,9 @@
  *
  *  $RCSfile: unodraw.hxx,v $
  *
- *  $Revision: 1.7 $
+ *  $Revision: 1.8 $
  *
- *  last change: $Author: os $ $Date: 2001-05-31 10:13:10 $
+ *  last change: $Author: os $ $Date: 2001-06-05 08:24:50 $
  *
  *  The Contents of this file are made available subject to the terms of
  *  either of the following licenses
@@ -197,6 +197,7 @@ class SwXShape : public SwXShapeBaseClass,
     friend class SwHTMLImageWatcher;
     friend class SwHTMLParser;
     friend class SwXGroupShape;
+    friend class SwXDrawPage;
 
     ::com::sun::star::uno::Reference< ::com::sun::star::uno::XAggregation >                 xShapeAgg;
     SfxItemPropertySet          aPropSet;
diff --git a/sw/source/core/unocore/unodraw.cxx b/sw/source/core/unocore/unodraw.cxx
index c41b3cc22e6d..b33cfb457f12 100644
--- a/sw/source/core/unocore/unodraw.cxx
+++ b/sw/source/core/unocore/unodraw.cxx
@@ -2,9 +2,9 @@
  *
  *  $RCSfile: unodraw.cxx,v $
  *
- *  $Revision: 1.23 $
+ *  $Revision: 1.24 $
  *
- *  last change: $Author: os $ $Date: 2001-06-01 07:02:43 $
+ *  last change: $Author: os $ $Date: 2001-06-05 08:20:38 $
  *
  *  The Contents of this file are made available subject to the terms of
  *  either of the following licenses
@@ -554,8 +554,15 @@ void SwXDrawPage::add(const uno::Reference< drawing::XShape > & xShape)
         pSvxShape = (SvxShape*)xShapeTunnel->getSomething(SvxShape::getUnoTunnelId());
     }
 
-    if(!pShape)
-        throw uno::RuntimeException();
+    if(!pShape || pShape->GetRegisteredIn() || !pShape->m_bDescriptor )
+    {
+        RuntimeException aExcept;
+        if(pShape)
+            aExcept.Message = C2U("object already inserted");
+        else
+            aExcept.Message = C2U("illegal object");
+        throw aExcept;
+    }
     GetSvxPage()->add(xShape);
 
     uno::Reference< uno::XAggregation >     xAgg = pShape->GetAggregationInterface();
@@ -654,6 +661,7 @@ void SwXDrawPage::add(const uno::Reference< drawing::XShape > & xShape)
     SwFrmFmt* pFmt = ::FindFrmFmt( pObj );
     if(pFmt)
         pFmt->Add(pShape);
+    pShape->m_bDescriptor = sal_False;
 
     delete pPam;
     delete pInternalPam;