diff options
| author | Justin Luth <justin.luth@collabora.com> | 2023-03-02 19:32:27 -0500 |
|---|---|---|
| committer | Justin Luth <jluth@mail.com> | 2023-03-07 14:03:44 +0000 |
| commit | beae0eaf6a7534181a0fbd11d071aaef8617d05f (patch) | |
| tree | cf4de4939b356b0ae6f059c209709d893e4e2b17 /writerfilter | |
| parent | 505abcce16134d30ffb842fa569126198ac31388 (diff) | |
tdf#153613 writerfilter: handle defered break before SDT start
If there are deferred column or page breaks before an SDT,
then before starting the SDT we need to determine if the
existing run needs to split - same as everywhere else.
There was only one existing unit test that had any defered
page or column breaks, so only two example documents to work from.
That was tdf125778_lostPageBreakTOX.docx.
Change-Id: I02d71577af13399d8aaf3ba0ce40fc6bed5e6e3c
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/148148
Tested-by: Jenkins
Reviewed-by: Justin Luth <jluth@mail.com>
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/148362
Tested-by: Justin Luth <jluth@mail.com>
Diffstat (limited to 'writerfilter')
| -rw-r--r-- | writerfilter/source/dmapper/DomainMapper.cxx | 31 |
1 files changed, 31 insertions, 0 deletions
diff --git a/writerfilter/source/dmapper/DomainMapper.cxx b/writerfilter/source/dmapper/DomainMapper.cxx index 61b4d6d114d4..46060b2d9950 100644 --- a/writerfilter/source/dmapper/DomainMapper.cxx +++ b/writerfilter/source/dmapper/DomainMapper.cxx @@ -1074,7 +1074,9 @@ void DomainMapper::lcl_attribute(Id nName, Value & val) break; case NS_ooxml::LN_CT_SdtBlock_sdtContent: case NS_ooxml::LN_CT_SdtRun_sdtContent: + { m_pImpl->m_pSdtHelper->SetSdtType(nName); + if (m_pImpl->m_pSdtHelper->getControlType() == SdtControlType::unknown) { // Still not determined content type? and it is even not unsupported? Then it is plain text field @@ -1092,11 +1094,40 @@ void DomainMapper::lcl_attribute(Id nName, Value & val) } } + PropertyMapPtr pContext = m_pImpl->GetTopContextOfType(CONTEXT_PARAGRAPH); + if (pContext && m_pImpl->isBreakDeferred(PAGE_BREAK)) + { + if (!m_pImpl->GetFootnoteContext() && !m_pImpl->IsInShape() + && !m_pImpl->IsInComments()) + { + if (m_pImpl->GetIsFirstParagraphInSection() || !m_pImpl->IsFirstRun()) + { + m_pImpl->m_bIsSplitPara = true; + finishParagraph(); + lcl_startParagraphGroup(); + } + pContext->Insert(PROP_BREAK_TYPE, uno::Any(style::BreakType_PAGE_BEFORE)); + m_pImpl->clearDeferredBreaks(); + } + } + else if (pContext && m_pImpl->isBreakDeferred(COLUMN_BREAK)) + { + if (m_pImpl->GetIsFirstParagraphInSection() || !m_pImpl->IsFirstRun()) + { + mbIsSplitPara = true; + finishParagraph(); + lcl_startParagraphGroup(); + } + pContext->Insert(PROP_BREAK_TYPE, uno::Any(style::BreakType_COLUMN_BEFORE)); + m_pImpl->clearDeferredBreaks(); + } + m_pImpl->m_pSdtHelper->setControlType(SdtControlType::richText); m_pImpl->PushSdt(); break; } m_pImpl->SetSdt(true); + } break; case NS_ooxml::LN_CT_SdtBlock_sdtEndContent: case NS_ooxml::LN_CT_SdtRun_sdtEndContent: |