1. Open attachment 117736 [details] (from Bug 93229 and Bug 94431)
2. Note content is read-only except for gray input fields in the table.
3. Add some content to some of those input fields
4. Save as DOCX and reopen.
5. Note other content is still read-only, which is good, but:
Expected: All input fields and their content are retained.
Actual: Input fields retained only if there was content there, not if empty.
DOCX was OK from 5.2 to 5.4, unlike DOC that has it own Bug 52555, and RTF, which has old Bug 94442.
Looks like a recent regression in 6.0+. If confirmed, please set bibisectRequest.
Looks like this fix from 5.2 was Bug 99434.
This seems to have begun at the below commit.
Adding Cc: to Mike Kaganski; Could you possibly take a look at this one? Thanks
efc773ec387320dda701706434cd1d17092e8647 is the first bad commit
Author: Norbert Thiebaud <firstname.lastname@example.org>
Date: Sat Jul 29 05:38:45 2017 -0700
author Mike Kaganski <email@example.com> 2017-07-07 08:33:34 (GMT)
committer Mike Kaganski <firstname.lastname@example.org> 2017-07-07 10:52:00 (GMT)
commit 7c1a51516aaf2767e43b393259a1ad21570df5fb (patch)
parent 350fb0249af7c29c98c3e15415a151bda8fae7ee (diff)
tdf#108995: take xml:space attribute into account
The problem identified. My patch uses UOString::trim(), which seems to be trimming all Unicode codepoints with Space category. DOCX uses EN SPACE (U+2002) for placeholders in the fields, and these spaces get trimmed on import. (The exported file opens OK in Word btw.)
Will come with a fix tomorrow.
Patch under review: https://gerrit.libreoffice.org/41444
Mike Kaganski committed a patch related to this issue.
It has been pushed to "master":
tdf#111964: only trim XML whitespace
It will be available in 6.0.0.
The patch should be included in the daily builds available at
http://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More
information about daily builds can be found at:
Affected users are encouraged to test the fix and report feedback.
Please set as Fixed. If I do it, script counts me as a fixer, which is not correct. And then I can put Verified.