| commit | fb2b0e1836431f9c16d7125dfd3f596436899134 | [log] |
|---|---|---|
| author | Eike Rathke <erack@redhat.com> | Wed Oct 20 13:41:03 2021 +0200 |
| committer | Xisco Fauli <xiscofauli@libreoffice.org> | Wed Oct 20 18:27:16 2021 +0200 |
| tree | 86d2186ba765627658f976be81f7289b5e77c21b | |
| parent | 879f73cf5181f92b582b9cd2a38a9cf27a9621ef [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/+/123864 Reviewed-by: Xisco Fauli <xiscofauli@libreoffice.org>
diff --git a/sc/source/core/tool/interpr8.cxx b/sc/source/core/tool/interpr8.cxx index 64bb3d1..05b9101 100644 --- a/sc/source/core/tool/interpr8.cxx +++ b/sc/source/core/tool/interpr8.cxx
@@ -1742,17 +1742,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 )