svx/source/table/svdotable.cxx | 3 +++
1 file changed, 3 insertions(+)
New commits:
commit f6a4b8ca7f79126569db14b030b46733309dbda6
Author: Stepas Toliautas <stepas.toliau...@gmail.com>
AuthorDate: Tue Dec 17 11:09:33 2019 +0200
Commit: Thorsten Behrens <thorsten.behr...@cib.de>
CommitDate: Wed Dec 18 17:48:38 2019 +0100
tdf#127481: reset active cell reference during table copy
During cloning of 1x1 table, SdrTableObjImpl.mxActiveCell reference
is set incorrectly, keeping an address of a temporary cell.
I clear that reference so it gets assigned to a cell of a new table
on first request. An alternative is to find the position of an active
cell in source table and set it to a corresponding cell in cloned
table (after call to new TableModel()), but I have no opinion
on whether newly-cloned table should have active cell marker,
and the assignment seems to work in some other fashion for multi-cell
tables. So I just zero-init temp object, which should be done anyway.
Change-Id: I965b7afbfedf2e57262d3388be1d045687521eee
Reviewed-on: https://gerrit.libreoffice.org/85275
Tested-by: Jenkins
Reviewed-by: Thorsten Behrens <thorsten.behr...@cib.de>
diff --git a/svx/source/table/svdotable.cxx b/svx/source/table/svdotable.cxx
index 68386971fb88..2538b798c55c 100644
--- a/svx/source/table/svdotable.cxx
+++ b/svx/source/table/svdotable.cxx
@@ -410,6 +410,9 @@ SdrTableObjImpl& SdrTableObjImpl::operator=( const
SdrTableObjImpl& rSource )
mxTable.clear();
}
+ // tdf#127481: reset active cell reference
+ mxActiveCell.clear();
+
// copy TableStyle (short internal data)
maTableStyle = rSource.maTableStyle;
_______________________________________________
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits
