diff options
| author | Kohei Yoshida <kohei.yoshida@collabora.com> | 2014-06-26 14:51:47 -0400 |
|---|---|---|
| committer | Kohei Yoshida <kohei.yoshida@collabora.com> | 2014-06-26 22:49:35 -0400 |
| commit | ecde8155fefcdacb08177a10a4eddc24f6675b0b (patch) | |
| tree | c98c86c1202357d6db8b2b51a5463fce98810552 /chart2 | |
| parent | 822d0e9db610ce3cbee2cf8d3154ee41218ea562 (diff) | |
fdo#79151: No need to offset column index by category labels.
In fact, doing so would lead to a corrupt internal data table. The
internal data stores category labels separately from the column data, so
there is no reason why we need to offset for category labels here.
Change-Id: I29dd99e53837cba7a1e28d2a1ef39a5384378805
Diffstat (limited to 'chart2')
| -rw-r--r-- | chart2/source/controller/dialogs/DataBrowserModel.cxx | 5 |
1 files changed, 0 insertions, 5 deletions
diff --git a/chart2/source/controller/dialogs/DataBrowserModel.cxx b/chart2/source/controller/dialogs/DataBrowserModel.cxx index e2bff5b99691..16ac3a2b3eba 100644 --- a/chart2/source/controller/dialogs/DataBrowserModel.cxx +++ b/chart2/source/controller/dialogs/DataBrowserModel.cxx @@ -343,10 +343,6 @@ void DataBrowserModel::insertDataSeries( sal_Int32 nAfterColumnIndex ) if (!xChartType.is()) return; - sal_Int32 nOffset = 0; - if( xDiagram.is() && lcl_ShowCategories( xDiagram )) - nOffset = getCategoryColumnCount(); - // Get shared sequences of current series. Normally multiple data series // only share "values-x" sequences. (TODO: simplify this logic). Reference< chart2::XDataSeriesContainer > xSeriesCnt( xChartType, uno::UNO_QUERY ); @@ -367,7 +363,6 @@ void DataBrowserModel::insertDataSeries( sal_Int32 nAfterColumnIndex ) Sequence<Reference<chart2::data::XLabeledDataSequence> > aLSequences = xSource->getDataSequences(); sal_Int32 nSeqIdx = 0; sal_Int32 nSeqSize = aLSequences.getLength(); - nStartCol -= (nOffset - 1); // ??? for (sal_Int32 nIndex = nStartCol; nSeqIdx < nSeqSize; ++nSeqIdx) { lcl_tSharedSeqVec::const_iterator aSharedIt( |