tdf#138544 sw LoadUserSettings: default DoNotJustifyLinesWithManualBreak

This is the second patch in the series.
When you start a new document, Writer compat setting
"Expand word space on lines with manual breaks
in justified paragraphs" is on.
Now, when saving and reloading it with LoadUserSettings disabled,
it is still the program default of "on", instead of "off".

So this one is slightly more dangerous because it will prevent
setting old behaviours for MULTIPLE settings which are
not listed in the UI.
(Anything that trusts that ConsiderWrapOnObjPos to indicate a
very old document.)

The fact that we are explicitly not loading ConsiderWrapOnObjPos
from the document should not imply that this document is an old one.
At this far point away from StarOffice 8, it is probably safer
to assume the document is new, not old.


P.S. I'm pretty sure the reset of ExpandWordSpace was a complete
error during a refactor.
(At that point, it was the only item that defaulted to true.)
The only scenario I can imagine is that at one point in time,
ExpandWordSpace was not part of the SvtCompatibilityOptions,
and so this would add it. But now it certainly is one of the
options, and so this pointless pre-init is simply overwritten.

Change-Id: Ibe160099839bcf40d5659f7d5246854c7c4fb5e8
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/108546
Tested-by: Justin Luth <justin_luth@sil.org>
Reviewed-by: Justin Luth <justin_luth@sil.org>
Reviewed-by: Miklos Vajna <vmiklos@collabora.com>

1 files changed, 0 insertions, 1 deletions

diff --git a/sw/source/ui/config/optcomp.cxx b/sw/source/ui/config/optcomp.cxx
index a3c3837006ce..2cb5a8558eb3 100644
--- a/sw/source/ui/config/optcomp.cxx
+++ b/sw/source/ui/config/optcomp.cxx
@@ -208,7 +208,6 @@ void SwCompatibilityOptPage::InitControls( const SfxItemSet& rSet )
     const Sequence< Sequence< PropertyValue > > aList = m_aConfigItem.GetList();
 
     SvtCompatibilityEntry aEntry;
-    aEntry.setValue<bool>( SvtCompatibilityEntry::Index::ExpandWordSpace, false );
 
     for ( const Sequence< PropertyValue >& rEntry : aList )
     {