diff options
| author | Eike Rathke <erack@redhat.com> | 2022-02-16 16:56:52 +0100 |
|---|---|---|
| committer | Eike Rathke <erack@redhat.com> | 2022-02-16 18:45:25 +0100 |
| commit | a1f21d7094deb6f1ae5388718f2bc28eecd5737a (patch) | |
| tree | bb912068a66e391804cbc5bb9c104d5a17c4acdc | |
| parent | 72c5c23defee2d542b0e4a57018fc86638d15137 (diff) | |
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 <erack@redhat.com>
Tested-by: Jenkins
| -rw-r--r-- | sc/source/core/data/bcaslot.cxx | 2 |
1 files changed, 1 insertions, 1 deletions
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()) { |