diff options
author | Justin Luth <justin.luth@collabora.com> | 2018-04-21 11:35:31 +0300 |
---|---|---|
committer | Aron Budea <aron.budea@collabora.com> | 2018-11-01 18:16:41 +0100 |
commit | fcba99488b2193b385bd8dd7d3a85f7e46e2e0ce (patch) | |
tree | 276c992146cd080cdf16999314ae2e9102cd1065 /writerfilter | |
parent | 1e9f2450aa839520290311bc3c4b156f0624adfc (diff) |
tdf#117137: DOCX import: don't try to set grab-bag as UNO prop
This modifies commit a6f2199e9888cb75960f1d35034bd44fb45e5565
"DOCX import: don't try to set grab-gag props as UNO props"
Perhaps that commit should simply be reverted, but I will trust
that the commit was mostly OK and simply adjust the logic so
that *InteropGrabBag is ignored as before.
Doing this resolves MSO being unable to open a specific document
and LO missing some numbering during LO round-tripping.
Probably these are just side-effects from other locations in the
code that couldn't deal with these unexpected properties. For
example, the numbering.xml file is malformed, since it is
missing the w14: namespace.
Unfortunately, I failed in my attempt to create a minimal
test document.
Change-Id: Idf88cd09d96546b7f03d326afb5f6e58439bcf20
Reviewed-on: https://gerrit.libreoffice.org/53271
Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Justin Luth <justin_luth@sil.org>
(cherry picked from commit ac27f4e7abf5339f71d4f5f3fc09a13b25669fe4)
Reviewed-on: https://gerrit.libreoffice.org/62736
Reviewed-by: Aron Budea <aron.budea@collabora.com>
Tested-by: Aron Budea <aron.budea@collabora.com>
Diffstat (limited to 'writerfilter')
-rw-r--r-- | writerfilter/source/dmapper/NumberingManager.cxx | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/writerfilter/source/dmapper/NumberingManager.cxx b/writerfilter/source/dmapper/NumberingManager.cxx index d9dfeb3d8efe..89e910102ac0 100644 --- a/writerfilter/source/dmapper/NumberingManager.cxx +++ b/writerfilter/source/dmapper/NumberingManager.cxx @@ -217,7 +217,7 @@ uno::Sequence< beans::PropertyValue > ListLevel::GetCharStyleProperties( ) { if (IgnoreForCharStyle(aValIter->Name)) continue; - else + else if ( aValIter->Name != "CharInteropGrabBag" && aValIter->Name != "ParaInteropGrabBag" ) rProperties.emplace_back(aValIter->Name, 0, aValIter->Value, beans::PropertyState_DIRECT_VALUE); } |