summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorMichael Stahl <mstahl@redhat.com>2014-01-29 22:38:21 +0100
committerMiklos Vajna <vmiklos@collabora.co.uk>2014-01-30 08:47:44 +0000
commitf72af83f3a68e0683f8d48dd02c90306b8b2d6f8 (patch)
tree3f04ebc03b4e5299b67a0914d166edab99f22c09
parent9a19e8d838753128504274e1885eb3ce8ec1dbb8 (diff)
fdo#63553: sw: fix copy/paste of cross references
If one of aIds or aDestIds contains nSeqNo then adding an identity mapping for it will actually re-define a previous mapping. Likely this can only happen when clipboard document is the source. (regression from 44f971506c0ed37928c48e55d8007f24b0c43a5f) (also it looks like 4b0b0a5b3045902d9a6b95510ec30a3027181e29 has subtly changed what the "else" branch does when it is erroneously taken - results are differently wrong) Change-Id: I34e2979e45cffeb62ebf3d7b3e5ac6d2afff0c2b (cherry picked from commit bb665affbd8870652ade3951d626d76e99143f67) Reviewed-on: https://gerrit.libreoffice.org/7729 Reviewed-by: Miklos Vajna <vmiklos@collabora.co.uk> Tested-by: Miklos Vajna <vmiklos@collabora.co.uk>
-rw-r--r--sw/source/core/fields/reffld.cxx18
1 files changed, 13 insertions, 5 deletions
diff --git a/sw/source/core/fields/reffld.cxx b/sw/source/core/fields/reffld.cxx
index 96de7834e448..e20accbeb30e 100644
--- a/sw/source/core/fields/reffld.cxx
+++ b/sw/source/core/fields/reffld.cxx
@@ -992,7 +992,7 @@ void _RefIdsMap::Init( SwDoc& rDoc, SwDoc& rDestDoc, bool bField )
for ( std::set<sal_uInt16>::iterator pIt = aDstIds.begin(); pIt != aDstIds.end(); ++pIt )
AddId( GetFirstUnusedId(aIds), *pIt );
- // Change the Sequence number of all the SetExp fields in the destination document
+ // Change the Sequence number of all SetExp fields in the source document
SwFieldType* pType = rDoc.GetFldType( RES_SETEXPFLD, aName, false );
if( pType )
{
@@ -1046,16 +1046,24 @@ void _RefIdsMap::Check( SwDoc& rDoc, SwDoc& rDestDoc, SwGetRefField& rFld,
{
Init( rDoc, rDestDoc, bField);
- // dann teste mal, ob die Nummer schon vergeben ist
- // oder ob eine neue bestimmt werden muss.
sal_uInt16 nSeqNo = rFld.GetSeqNo();
- if( aIds.count( nSeqNo ) && aDstIds.count( nSeqNo ))
+
+ // Check if the number is used in both documents
+ // Note: For fields, aIds contains both the ids of SetExp from rDestDoc
+ // and the targets of the already remapped ones from rDoc.
+ // It is possible that aDstIds contains numbers that aIds does not contain!
+ // For example, copying a selection to clipboard that does not contain
+ // the first SwSetExpField will result in id 0 missing, then pasting that
+ // into empty document gives a mapping 1->0 ... N->N-1 (fdo#63553).
+ if (aIds.count(nSeqNo) || aDstIds.count(nSeqNo))
{
// Number already taken, so need a new one.
if( sequencedIds.count(nSeqNo) )
rFld.SetSeqNo( sequencedIds[nSeqNo] );
else
{
+ assert(!bField || !aDstIds.count(nSeqNo)); // postcond of Init
+
sal_uInt16 n = GetFirstUnusedId( aIds );
// die neue SeqNo eintragen, damit die "belegt" ist
@@ -1077,7 +1085,7 @@ void _RefIdsMap::Check( SwDoc& rDoc, SwDoc& rDestDoc, SwGetRefField& rFld,
}
else
{
- AddId( nSeqNo, nSeqNo );
+ AddId( nSeqNo, nSeqNo ); // this requires that nSeqNo is unused in both!
}
}