diff options
author | Justin Luth <justin_luth@sil.org> | 2021-07-26 14:39:17 +0200 |
---|---|---|
committer | Justin Luth <justin_luth@sil.org> | 2021-07-26 19:24:04 +0200 |
commit | 1441e001938d6a0b19743ef8170980238b88d350 (patch) | |
tree | b25feab27d47cf1bd14cf69635d84cac7beffbd5 /writerfilter | |
parent | dd43ed8610f2d3e4c90d4bfd8a6ab3c736e5b4b8 (diff) |
tdf#143517 partial revert "tdf#141964 writerfilter CN: get para...
...paragraph and inherited listLevel
IIRC, I changed GetCurrentNumberingCharStyle() on the premise that
no matter where the level is defined (paragraph or style),
any function looking for it should use the inherited/applied level.
But GetCurrentNumberingCharStyle() itself looks like a bogus
hack that probably should be eliminated completely.
Unfortunately, 2123ede032ca64f696ef54af4ad3238974ca2b5d
from 2012 which introduced it does not include a unit test,
so there is no easy way to test whether it still has any
function.
[Eliminating it passes all unit tests and sounds similar to
tdf#108518 revert OOo hack: Fix issue #i119405: Numbering text style.]
In any case, just revert the changes to this function for now,
since hopelessly faulty functions don't need to operate "better".
Another alternative would be to just use GetListlevel(pEntry)
to at least allow style inheritance, but since it seems like
a completely faulty function to begin with, it seems better to
leave the function completely untouched.
Change-Id: I2eba777fdd68615d68b63288005a30301fa6ea2f
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/119521
Tested-by: Jenkins
Reviewed-by: Justin Luth <justin_luth@sil.org>
Diffstat (limited to 'writerfilter')
-rw-r--r-- | writerfilter/source/dmapper/DomainMapper_Impl.cxx | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/writerfilter/source/dmapper/DomainMapper_Impl.cxx b/writerfilter/source/dmapper/DomainMapper_Impl.cxx index 78f7dbc16b24..bc63eae6c0b0 100644 --- a/writerfilter/source/dmapper/DomainMapper_Impl.cxx +++ b/writerfilter/source/dmapper/DomainMapper_Impl.cxx @@ -7657,7 +7657,7 @@ uno::Reference<container::XIndexAccess> DomainMapper_Impl::GetCurrentNumberingRu if (nListId < 0) return xRet; if (pListLevel) - *pListLevel = GetListLevel(pEntry, GetTopContextOfType(CONTEXT_PARAGRAPH)); + *pListLevel = pStyleSheetProperties->GetListLevel(); // So we are in a paragraph style and it has numbering. Look up the relevant numbering rules. auto const pList(GetListTable()->GetList(nListId)); |