chart2/source/model/template/ChartType.cxx | 1 + 1 file changed, 1 insertion(+)
New commits: commit d49fc1cfb76514e300a3670f18e751c0710d81a6 Author: Regina Henschel <[email protected]> AuthorDate: Sun Jun 15 11:33:03 2025 +0200 Commit: Xisco Fauli <[email protected]> CommitDate: Mon Jun 16 11:29:30 2025 +0200 tdf#166664 adds missing erase in deleteSeries commit 84f7b840 has moved deleteSeries from DataSeriesHelper to ChartType. But the move has lost the actual erase. No unit test, because I don't know how to use a dialog in a test. Change-Id: I92dc1d2bdae0fe0d63b6c3b012b50b8c66032334 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/186512 Tested-by: Jenkins Reviewed-by: Regina Henschel <[email protected]> Reviewed-by: Noel Grandin <[email protected]> (cherry picked from commit 3391dfad01efddb4bfd3b2bfef221a17ac11e43a) Reviewed-on: https://gerrit.libreoffice.org/c/core/+/186541 Reviewed-by: Xisco Fauli <[email protected]> diff --git a/chart2/source/model/template/ChartType.cxx b/chart2/source/model/template/ChartType.cxx index 7b91223a8915..f8157b4f945a 100644 --- a/chart2/source/model/template/ChartType.cxx +++ b/chart2/source/model/template/ChartType.cxx @@ -648,6 +648,7 @@ void ChartType::deleteSeries( const rtl::Reference< ::chart::DataSeries > & xSer return; ModifyListenerHelper::removeListener( *it, m_xModifyEventForwarder ); + m_aDataSeries.erase(it); fireModifyEvent(); createCalculatedDataSeries();
