[Libreoffice-commits] core.git: sc/source

[Katarina Behrens]

 sc/source/filter/oox/condformatbuffer.cxx |    4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

New commits:
commit 317967af601797b3b1193db691330fcf6cb100e5
Author: Katarina Behrens <bubli at bubli.org>
Date:   Tue Feb 10 22:46:24 2015 +0100

    mbNeg means: a different colour for negative values is defined
    
    It is totally orthogonal to whether there's a gradient fill or not
    
    Change-Id: I32ab465d9116c3dc31cde3611cd19d3ec10ac5d6

diff --git a/sc/source/filter/oox/condformatbuffer.cxx b/sc/source/filter/oox/condformatbuffer.cxx
index 34519f1..a984202 100644
--- a/sc/source/filter/oox/condformatbuffer.cxx
+++ b/sc/source/filter/oox/condformatbuffer.cxx
@@ -1111,9 +1111,6 @@ void ExtCfRule::finalizeImport()
                 pDataBar->meAxisPosition = databar::MIDDLE;
             else
                 pDataBar->meAxisPosition = databar::AUTOMATIC;
-            // TODO: the following line makes very little sense, figure out
-            // how it used to be (XML_negativeBarColorSameAsPositive)
-            pDataBar->mbNeg = !maModel.mbGradient;
             pDataBar->mbGradient = maModel.mbGradient;
             break;
         }
@@ -1171,6 +1168,7 @@ void ExtCfRule::importNegativeFillColor( const AttributeList& rAttribs )
 {
      mnRuleType = NEGATIVEFILLCOLOR;
      maModel.mnNegativeColor = rAttribs.getIntegerHex( XML_rgb, API_RGB_TRANSPARENT );
+     maModel.mbNeg = true;
 }
 
 void ExtCfRule::importAxisColor( const AttributeList& rAttribs )