[Libreoffice-commits] core.git: Branch 'libreoffice-5-2' - sc/source

[Noel Grandin]

 sc/source/core/tool/charthelper.cxx |   10 +++-------
 1 file changed, 3 insertions(+), 7 deletions(-)

New commits:
commit 390cc253e2e54d8df61bd21e01ff6a9decb69cc7
Author: Noel Grandin <noel.grandin at collabora.co.uk>
Date:   Thu Sep 22 09:54:34 2016 +0200

    tdf#101894 Copying a single sheet with a chart on it garbles the chart
    
    ..on the new sheet
    
    Bug introduced with
        commit 86e9ef21a06a9babefb778dfc3e02118d371d70d
        Author: Noel Grandin <noelgrandin at gmail.com>
        Date:   Tue Mar 8 20:41:21 2016 +0200
        sequence->vector in sc
    
    Change-Id: I7a6539005f42c7a7036c60b12a9ff2f77f71ba6b
    Reviewed-on: https://gerrit.libreoffice.org/29169
    Reviewed-by: Markus Mohrhard <markus.mohrhard at googlemail.com>
    Tested-by: Markus Mohrhard <markus.mohrhard at googlemail.com>

diff --git a/sc/source/core/tool/charthelper.cxx b/sc/source/core/tool/charthelper.cxx
index e75677c..55388cd 100644
--- a/sc/source/core/tool/charthelper.cxx
+++ b/sc/source/core/tool/charthelper.cxx
@@ -224,13 +224,9 @@ void ScChartHelper::GetChartRanges( const uno::Reference< chart2::XChartDocument
         uno::Reference< chart2::data::XDataSequence > xValues( xLabeledSequence->getValues());
 
         if (xLabel.is())
-             rRanges.push_back( xLabel->getSourceRangeRepresentation() );
-         else
-             rRanges.push_back( OUString() );
-         if (xValues.is())
-             rRanges.push_back( xValues->getSourceRangeRepresentation() );
-         else
-             rRanges.push_back( OUString() );
+            rRanges.push_back( xLabel->getSourceRangeRepresentation() );
+        if (xValues.is())
+            rRanges.push_back( xValues->getSourceRangeRepresentation() );
     }
 }