diff options
author | Mike Kaganski <mike.kaganski@collabora.com> | 2024-01-02 17:12:06 +0600 |
---|---|---|
committer | Mike Kaganski <mike.kaganski@collabora.com> | 2024-01-03 03:16:18 +0100 |
commit | 960e37af28807ed1b376e26c4504ab755a81dfd5 (patch) | |
tree | e587de5d3fc5eb23412d97dee29981b050a67de1 /editeng | |
parent | 9c9f5c9ed7a1d11fd34be6af5b09f836ed442211 (diff) |
tdf#158971: Only copy directly-set properties
Commit f09420fa189be5165b0311083ba127073500a121 (tdf#158855: Make sure
to not add extra paragraph after a table in a section, 2023-12-25) had
implemented copying all attributes of a paragraph using copyAllProps.
But the function didn't check the states of properties, so copied also
default values of properties, making them direct properties of the
paragraph. Due to tdf#158972, that caused an assertion when saving,
and the main problem was a set of direct properties in the paragraph.
Fix that by checking the properties states. Also make sure to work with
paragraphs, rather than with ranges, which treats any properties set on
paragraph level as default-state on the run level.
Change-Id: I7cd9c7fbb9313d666c46be201913f0223a6b4f5e
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/161539
Tested-by: Jenkins
Reviewed-by: Mike Kaganski <mike.kaganski@collabora.com>
Diffstat (limited to 'editeng')
0 files changed, 0 insertions, 0 deletions