diff options
| author | Ouyang Leyan <ouyang.leyan@hotmail.com> | 2020-11-21 15:42:19 +0100 |
|---|---|---|
| committer | Noel Grandin <noel.grandin@collabora.co.uk> | 2020-11-23 07:31:06 +0100 |
| commit | fd5f3b212a38ec6ef35d2686ce5a7cc39cb7d377 (patch) | |
| tree | 8b8a84dc95d9c05e9c0a3b8fe0d21dbaf51cbc81 | |
| parent | d767d0c968d243a891999df5685e177d8777fe7c (diff) | |
tdf#134059 Always include hidden cells in chart series label
Change-Id: I92ff60b26af713a49fefd8b2ce8042f4e8e4d179
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/106317
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
| -rw-r--r-- | chart2/source/controller/dialogs/tp_DataSource.cxx | 3 | ||||
| -rw-r--r-- | sc/source/ui/unoobj/chart2uno.cxx | 3 | ||||
| -rw-r--r-- | xmloff/source/chart/SchXMLSeries2Context.cxx | 8 |
3 files changed, 13 insertions, 1 deletions
diff --git a/chart2/source/controller/dialogs/tp_DataSource.cxx b/chart2/source/controller/dialogs/tp_DataSource.cxx index 6ac626957517..90eb974ca0d5 100644 --- a/chart2/source/controller/dialogs/tp_DataSource.cxx +++ b/chart2/source/controller/dialogs/tp_DataSource.cxx @@ -841,6 +841,9 @@ bool DataSourceTabPage::updateModelFromControl(const weld::Entry* pField) aRange = xNewSeq->getSourceRangeRepresentation(); Reference< beans::XPropertySet > xProp( xNewSeq, uno::UNO_QUERY_THROW ); xProp->setPropertyValue( "Role" , uno::Any( OUString(lcl_aLabelRole) )); + + //Labels should always include hidden cells, regardless of the setting chosen + xProp->setPropertyValue( "IncludeHiddenCells", uno::Any(true)); xLabeledSeq->setLabel( xNewSeq ); } } diff --git a/sc/source/ui/unoobj/chart2uno.cxx b/sc/source/ui/unoobj/chart2uno.cxx index ffbcba6c234f..004729176c2e 100644 --- a/sc/source/ui/unoobj/chart2uno.cxx +++ b/sc/source/ui/unoobj/chart2uno.cxx @@ -1031,7 +1031,8 @@ uno::Reference< chart2::data::XLabeledDataSequence > lcl_createLabeledDataSequen } if ( bHasLabel ) { - uno::Reference< chart2::data::XDataSequence > xLabelSeq( new ScChart2DataSequence( pDoc, std::move(aLabelTokens), bIncludeHiddenCells ) ); + //Labels should always include hidden cells, regardless of the bIncludeHiddenCells setting + uno::Reference< chart2::data::XDataSequence > xLabelSeq( new ScChart2DataSequence( pDoc, std::move(aLabelTokens), true ) ); xResult->setLabel( xLabelSeq ); } } diff --git a/xmloff/source/chart/SchXMLSeries2Context.cxx b/xmloff/source/chart/SchXMLSeries2Context.cxx index 434018276e93..992a03d64113 100644 --- a/xmloff/source/chart/SchXMLSeries2Context.cxx +++ b/xmloff/source/chart/SchXMLSeries2Context.cxx @@ -480,6 +480,14 @@ void SchXMLSeries2Context::StartElement( const uno::Reference< xml::sax::XAttrib xSequenceLabel.set(SchXMLTools::CreateDataSequenceWithoutConvert(aSeriesLabelString, mxNewDoc)); } } + + //Labels should always include hidden cells + Reference<beans::XPropertySet> xSeqLabelProp(xSequenceLabel, uno::UNO_QUERY); + if (xSeqLabelProp.is() && xSeqLabelProp->getPropertySetInfo()->hasPropertyByName("IncludeHiddenCells")) + { + xSeqLabelProp->setPropertyValue( "IncludeHiddenCells", uno::Any(true)); + } + xLabeledSeq->setLabel(xSequenceLabel); // Note: Even if we have no label, we have to register the label |