Hi, I have submitted a patch for review:
https://gerrit.libreoffice.org/3651 To pull it, you can do: git pull ssh://gerrit.libreoffice.org:29418/core refs/changes/51/3651/1 Fixing Indentation RTL bug Change-Id: Ie8c3d30cb28afb97d4ded4ac67241ed18ab9e50c --- M writerfilter/source/ooxml/OOXMLFastContextHandler.cxx 1 file changed, 13 insertions(+), 1 deletion(-) diff --git a/writerfilter/source/ooxml/OOXMLFastContextHandler.cxx b/writerfilter/source/ooxml/OOXMLFastContextHandler.cxx index 81ac271..0a0b63e 100644 --- a/writerfilter/source/ooxml/OOXMLFastContextHandler.cxx +++ b/writerfilter/source/ooxml/OOXMLFastContextHandler.cxx @@ -1609,7 +1609,19 @@ if (mpValue.get() == NULL) { - OOXMLValue::Pointer_t pValue(new OOXMLIntegerValue(0)); + // Value should not always be 'true' + //OOXMLValue::Pointer_t pValue(new OOXMLBooleanValue(true)); + bool bSet = true; + + // Paragraph properties are not always 'true' by default + // For example - RTL layout of paragraph is by default 'false' (look in this link for 'sprmPFBiDi') + // http://msdn.microsoft.com/en-us/library/dd923496%28v=office.12%29.aspx + if (getId() == 0x2441) + { + bSet = false; + } + + OOXMLValue::Pointer_t pValue(new OOXMLBooleanValue(bSet)); setValue(pValue); } } -- To view, visit https://gerrit.libreoffice.org/3651 To unsubscribe, visit https://gerrit.libreoffice.org/settings Gerrit-MessageType: newchange Gerrit-Change-Id: Ie8c3d30cb28afb97d4ded4ac67241ed18ab9e50c Gerrit-PatchSet: 1 Gerrit-Project: core Gerrit-Branch: master Gerrit-Owner: Adam Co <rattles2...@gmail.com> _______________________________________________ LibreOffice mailing list LibreOffice@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/libreoffice