diff options
| author | Tünde Tóth <toth.tunde@nisz.hu> | 2020-08-14 13:45:38 +0200 |
|---|---|---|
| committer | László Németh <nemeth@numbertext.org> | 2020-08-19 10:31:51 +0200 |
| commit | a8950d08a33e7aded31d4591c98a03d1edf8c5a4 (patch) | |
| tree | 7b8d50403c2207c25e6dd5ff75e1f25297389ecf /sc | |
| parent | 6724040e61c1efbee0b1456e3480b1e7b0827732 (diff) | |
tdf#123647 Chart XLSX export: fix missing empty chart
Save empty chart as empty bar chart, because a chart without chart type
is invalid.
Change-Id: Ieb9cc4300070b5280565fe814ba0343640dfff0a
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/100844
Tested-by: Jenkins
Tested-by: László Németh <nemeth@numbertext.org>
Reviewed-by: László Németh <nemeth@numbertext.org>
Diffstat (limited to 'sc')
| -rw-r--r-- | sc/source/filter/xcl97/xcl97rec.cxx | 10 |
1 files changed, 6 insertions, 4 deletions
diff --git a/sc/source/filter/xcl97/xcl97rec.cxx b/sc/source/filter/xcl97/xcl97rec.cxx index 213dfe77d4ae..0f9ba1288d33 100644 --- a/sc/source/filter/xcl97/xcl97rec.cxx +++ b/sc/source/filter/xcl97/xcl97rec.cxx @@ -202,9 +202,11 @@ bool IsValidObject( const XclObj& rObj ) if (!xCooSysContainer.is()) return false; - const uno::Sequence<uno::Reference<chart2::XCoordinateSystem> > xCooSysSeq = xCooSysContainer->getCoordinateSystems(); - if (!xCooSysSeq.hasElements()) - return false; + const uno::Sequence<uno::Reference<chart2::XCoordinateSystem>> xCooSysSeq = xCooSysContainer->getCoordinateSystems(); + + // tdf#123647 Save empty charts too. + // if (!xCooSysSeq.hasElements()) + // return false; for (const auto& rCooSys : xCooSysSeq) { @@ -212,7 +214,7 @@ bool IsValidObject( const XclObj& rObj ) if (!xChartTypeCont.is()) return false; - uno::Sequence<uno::Reference<chart2::XChartType> > xChartTypeSeq = xChartTypeCont->getChartTypes(); + uno::Sequence<uno::Reference<chart2::XChartType>> xChartTypeSeq = xChartTypeCont->getChartTypes(); if (!xChartTypeSeq.hasElements()) // No chart type. Not good. return false; |