writerfilter/source/dmapper/PropertyMap.cxx | 10 +++++++---
1 file changed, 7 insertions(+), 3 deletions(-)
New commits:
commit 3bdc8b82c067d467efc9d3f4d9780b6302edf6a4
Author: Justin Luth <justin.l...@collabora.com>
AuthorDate: Thu Dec 10 16:48:06 2020 +0300
Commit: Gabor Kelemen <kelemen.gab...@nisz.hu>
CommitDate: Tue Mar 2 16:30:14 2021 +0100
tdf#118711 writerfilter: don't hardcode default page description
well, at least not at the beginning of the document.
The problem with specifying is that it becomes a property
of the paragraph - and so if the paragraph is copied or moved,
the page break comes along with it and the user has to remove
the unnecessary page break wherever it lands.
Since the default will be applied anyway,
there is no value in specifying it.
I'm not sure how, but at least one test document started
with a Converted1 style, so I had to explicitly confirm
that the stylename really did match the default
for a first section.
A unit test also indicated that
it needs to be specified if an explict page number is assigned
because RES_PAGEDESC contains MID_PAGEDESC_PAGEDESCNAME
as well as MID_PAGEDESC_PAGENUMOFFSET.
Change-Id: I0d0f6b70767f7daaf300e09c0d31ac4b17b91ed1
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/107555
Tested-by: Jenkins
Reviewed-by: Justin Luth <justin_l...@sil.org>
Reviewed-by: Miklos Vajna <vmik...@collabora.com>
(cherry picked from commit 8787a45f9cbb5dce61b20817ef0e549b5a227a95)
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/111787
Tested-by: Gabor Kelemen <kelemen.gab...@nisz.hu>
Reviewed-by: Gabor Kelemen <kelemen.gab...@nisz.hu>
diff --git a/writerfilter/source/dmapper/PropertyMap.cxx
b/writerfilter/source/dmapper/PropertyMap.cxx
index 921a77eabfc0..0974b151e5ad 100644
--- a/writerfilter/source/dmapper/PropertyMap.cxx
+++ b/writerfilter/source/dmapper/PropertyMap.cxx
@@ -1797,12 +1797,16 @@ void SectionPropertyMap::CloseSectionGroup(
DomainMapper_Impl& rDM_Impl )
{
// Avoid setting page style in case of autotext: so inserting
the autotext at the
// end of the document does not introduce an unwanted page
break.
- if (!rDM_Impl.IsReadGlossaries() &&
!rDM_Impl.IsInFootOrEndnote())
+ // Also avoid setting the page style at the very beginning if
it still is the default page style.
+ const OUString sPageStyle = m_bTitlePage ?
m_sFirstPageStyleName : m_sFollowPageStyleName;
+ if (!rDM_Impl.IsReadGlossaries()
+ && !rDM_Impl.IsInFootOrEndnote()
+ && !(m_bIsFirstSection && sPageStyle == getPropertyName(
PROP_STANDARD ) && m_nPageNumber < 0)
+ )
{
xRangeProperties->setPropertyValue(
getPropertyName( PROP_PAGE_DESC_NAME ),
- uno::makeAny( m_bTitlePage ? m_sFirstPageStyleName
- : m_sFollowPageStyleName ) );
+ uno::makeAny(sPageStyle) );
}
if (0 <= m_nPageNumber)
_______________________________________________
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits
