sc/source/core/data/bcaslot.cxx | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
New commits:
commit a1f21d7094deb6f1ae5388718f2bc28eecd5737a
Author: Eike Rathke <er...@redhat.com>
AuthorDate: Wed Feb 16 16:56:52 2022 +0100
Commit: Eike Rathke <er...@redhat.com>
CommitDate: Wed Feb 16 18:45:25 2022 +0100
Resolves: tdf#147398 Test Intersects() instead of Contains(), tdf#119083
Regression from
commit 8406139062d9ffe1daed32aefe4e261c6c55d63e
CommitDate: Mon Dec 6 15:45:35 2021 +0100
process broadcasts for adjacent cells together (tdf#119083)
that changed single cell broadcasts to blocks of rows broadcasts and
- if (rAreaRange.Contains( rAddress))
+ if (rAreaRange.Contains( rRange))
but a block of rows may be distributed over several broadcast
areas so rRange is not contained within one rAreaRange and thus
was not broadcasted. Testing for intersection instead fixes this.
Change-Id: I10700296ebc897e4b7b7752e0e6bcb480085b487
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/130027
Reviewed-by: Eike Rathke <er...@redhat.com>
Tested-by: Jenkins
diff --git a/sc/source/core/data/bcaslot.cxx b/sc/source/core/data/bcaslot.cxx
index 3126918570d8..6869ca24e120 100644
--- a/sc/source/core/data/bcaslot.cxx
+++ b/sc/source/core/data/bcaslot.cxx
@@ -281,7 +281,7 @@ bool ScBroadcastAreaSlot::AreaBroadcast( const ScHint&
rHint)
ScBroadcastArea* pArea = (*aIter).mpArea;
const ScRange& rAreaRange = pArea->GetRange();
- if (rAreaRange.Contains( rRange))
+ if (rAreaRange.Intersects( rRange))
{
if (pArea->IsGroupListening())
{
