summaryrefslogtreecommitdiff
path: root/sw/source/core/doc
diff options
context:
space:
mode:
authorMichael Stahl <mstahl@redhat.com>2017-07-21 22:55:51 +0200
committerMichael Stahl <mstahl@redhat.com>2017-07-21 23:26:18 +0200
commit850795942b3e168cab8ce88b4f2b421945ff29ca (patch)
tree4bf7e9e11fe9a83d707425860fa46fbcd268f267 /sw/source/core/doc
parent37185c03e1d66c494b0da298b22aca557d8a3f3c (diff)
tdf#99692 sw: fix bookmark positions in tables at start of redlines
The code assumes that if it can move the cursor backward in line 2038, that move can be "inverted" by moving the cursor forward after the content has been moved - but if the cursor moved back a node, and the moved content does not start with a SwTextNode, the cursor will move forward skipping over the non-text nodes, so offsets in the aSaveBkmks (and aSaveRedl, presumably) are going to be wrong. Just don't use Move() if it leaves the current node. Change-Id: I95278a10c14aeba9f76558486bb2712f6726dbcb
Diffstat (limited to 'sw/source/core/doc')
-rw-r--r--sw/source/core/doc/DocumentContentOperationsManager.cxx13
1 files changed, 10 insertions, 3 deletions
diff --git a/sw/source/core/doc/DocumentContentOperationsManager.cxx b/sw/source/core/doc/DocumentContentOperationsManager.cxx
index 57204ff80b5a..7df812d1cdd5 100644
--- a/sw/source/core/doc/DocumentContentOperationsManager.cxx
+++ b/sw/source/core/doc/DocumentContentOperationsManager.cxx
@@ -2033,12 +2033,19 @@ bool DocumentContentOperationsManager::MoveRange( SwPaM& rPaM, SwPosition& rPos,
// Put back the Pam by one "content"; so that it's always outside of
// the manipulated range.
- // If there's no content anymore, set it to the StartNode (that's
- // always there).
- const bool bNullContent = !aSavePam.Move( fnMoveBackward, GoInContent );
+ // tdf#99692 don't Move() back if that would end up in another node
+ // because moving backward is not necessarily the inverse of forward then.
+ const bool bNullContent = aSavePam.GetPoint()->nContent == 0;
if( bNullContent )
{
aSavePam.GetPoint()->nNode--;
+ aSavePam.GetPoint()->nContent.Assign(aSavePam.GetContentNode(), 0);
+ }
+ else
+ {
+ bool const success(aSavePam.Move(fnMoveBackward, GoInContent));
+ assert(success);
+ (void) success;
}
// Copy all Bookmarks that are within the Move range into an array,