sw/source/writerfilter/dmapper/DomainMapper.cxx |    1 +
 1 file changed, 1 insertion(+)

New commits:
commit c8dd921334864b3dc61230fb66d1ad1021edadcf
Author:     Justin Luth <[email protected]>
AuthorDate: Fri Feb 6 10:00:42 2026 -0500
Commit:     Justin Luth <[email protected]>
CommitDate: Mon Feb 9 15:21:56 2026 +0100

    tdf#149240 docx import: don't grabbag ShowingPlcHdr for runSDTs
    
    This improves my first SDT patch in LO 7.5
    
    This section is tricky to code-read.
    The 'break' is necessary to stop doing 'more stuff'.
    All of the other run properties are 'done'
    when they set the value,
    and only continue to grabbag them
    if they are in blockSDTs.
    
    Without this patch,
    runSDTs were getting a SdtPr grabbag collection
    with only this one ShowingPlcHdr property.
    
    Since the runSdt already knows the value
    it doesn't need to be grabbagged,
    and should 'break' just like all the others.
    
    Change-Id: I5777eea80289b012bb8464e7f45b1c6432f4e5a8
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/198847
    Reviewed-by: Justin Luth <[email protected]>
    Tested-by: Jenkins
    (cherry picked from commit 6068b676b6887bc14901f0da56048f71ce0a4a25)
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/198956

diff --git a/sw/source/writerfilter/dmapper/DomainMapper.cxx 
b/sw/source/writerfilter/dmapper/DomainMapper.cxx
index 98ed01303c07..64c5584cf5c6 100644
--- a/sw/source/writerfilter/dmapper/DomainMapper.cxx
+++ b/sw/source/writerfilter/dmapper/DomainMapper.cxx
@@ -3433,6 +3433,7 @@ void DomainMapper::sprmWithProps( Sprm& rSprm, const 
PropertyMapPtr& rContext )
             {
                 if (nIntValue)
                     m_pImpl->m_pSdtHelper->SetShowingPlcHdr();
+                break;
             }
 
             if (nSprmId == NS_ooxml::LN_CT_SdtPr_color)

Reply via email to