correct implementation

Change-Id: Ie195e46e1cb14b39875efd1c9689db0f13968e87

1 files changed, 26 insertions, 7 deletions

diff --git a/sc/source/ui/unoobj/condformatuno.cxx b/sc/source/ui/unoobj/condformatuno.cxx
index ce46c429edc1..8efdc9475514 100644
--- a/sc/source/ui/unoobj/condformatuno.cxx
+++ b/sc/source/ui/unoobj/condformatuno.cxx
@@ -332,16 +332,35 @@ void ScCondFormatsObj::Notify(SfxBroadcaster& /*rBC*/, const SfxHint& rHint)
     }
 }
 
-sal_Int32 ScCondFormatsObj::createByRange(const uno::Reference< sheet::XSheetCellRanges >& /*xRanges*/)
+sal_Int32 ScCondFormatsObj::createByRange(const uno::Reference< sheet::XSheetCellRanges >& xRanges)
     throw(uno::RuntimeException, std::exception)
 {
     SolarMutexGuard aGuard;
-    /*
-    ScCondFormatObj* pFormatObj = ScCondFormatObj::getImplementation(xCondFormat);
-    ScConditionalFormat* pFormat = pFormatObj->getCoreObject();
-    mpFormatList->InsertNew(pFormat);
-    */
-    return 0;
+    if (!mpDocShell)
+        throw lang::IllegalArgumentException();
+
+    if (!xRanges.is())
+        throw lang::IllegalArgumentException();
+
+    uno::Sequence<table::CellRangeAddress> aRanges =
+        xRanges->getRangeAddresses();
+
+    ScRangeList aCoreRange;
+    for (sal_Int32 i = 0, n = aRanges.getLength(); i < n; ++i)
+    {
+        ScRange aRange;
+        ScUnoConversion::FillScRange(aRange, aRanges[i]);
+        aCoreRange.Join(aRange);
+    }
+
+    if (aCoreRange.empty())
+        throw lang::IllegalArgumentException();
+
+    SCTAB nTab = aCoreRange[0]->aStart.Tab();
+
+    ScConditionalFormat* pNewFormat = new ScConditionalFormat(0, &mpDocShell->GetDocument());
+    pNewFormat->SetRange(aCoreRange);
+    return mpDocShell->GetDocument().AddCondFormat(pNewFormat, nTab);
 }
 
 void ScCondFormatsObj::removeByID(const sal_Int32 nID)