| commit | 989eb3461bd4ba8716da2c62386e4f044731ff16 | [log] |
|---|---|---|
| author | Eike Rathke <erack@redhat.com> | Wed Oct 20 13:41:03 2021 +0200 |
| committer | Mike Kaganski <mike.kaganski@collabora.com> | Tue Oct 26 12:11:55 2021 +0200 |
| tree | 521c2d35a392143585837b8d25ec450b7578afb0 | |
| parent | c34dd19ef0729504e89c8b2523c6228fbb716fda [diff] |
Resolves: tdf#145235 TEXTJOIN() clear last string also for referenced empty Change-Id: If6d20a1629e001708c700c5c25bef8a75fa34e25 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/123889 Reviewed-by: Eike Rathke <erack@redhat.com> Tested-by: Jenkins (cherry picked from commit f4f2c94513e7d06691a73d9f12707c33d131d537) Reviewed-on: https://gerrit.libreoffice.org/c/core/+/123866 Reviewed-by: Xisco Fauli <xiscofauli@libreoffice.org> Reviewed-by: Michael Stahl <michael.stahl@allotropia.de> Reviewed-by: Mike Kaganski <mike.kaganski@collabora.com> Tested-by: Mike Kaganski <mike.kaganski@collabora.com>
diff --git a/sc/source/core/tool/interpr8.cxx b/sc/source/core/tool/interpr8.cxx index 19cdd7e..6cc239d 100644 --- a/sc/source/core/tool/interpr8.cxx +++ b/sc/source/core/tool/interpr8.cxx
@@ -1740,17 +1740,14 @@ void ScInterpreter::ScTextJoin_MS() aAdr.SetRow( nRow ); aAdr.SetCol( nCol ); ScRefCellValue aCell( mrDoc, aAdr ); if ( !aCell.isEmpty() ) { if ( !aCell.hasEmptyValue() ) { svl::SharedString aSS; GetCellString( aSS, aCell); aStr = aSS.getString(); } } else if (aCell.hasEmptyValue()) aStr.clear(); else { svl::SharedString aSS; GetCellString( aSS, aCell); aStr = aSS.getString(); } if ( !aStr.isEmpty() || !bSkipEmpty ) { if ( !bFirst )