https://bugs.documentfoundation.org/show_bug.cgi?id=163154

Justin L <[email protected]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Severity|normal                      |minor
           Priority|medium                      |low

--- Comment #1 from Justin L <[email protected]> ---
Interesting - the difference is that the first bookmark comes after the field
has already started [like how LO exports]
    <w:fldChar w:fldCharType="begin">
        <w:ffData>
            <w:name w:val="Text1"/>
    ...
    <w:bookmarkStart w:id="0" w:name="Text1"/>

while the other fields all start with the bookmark and then define the field
[which is how MS Word exports], and thus they didn't pass if(IsOpenField()).

If I understand this bug report correctly, there is no functionality break. It
simply is a convenient fact that the legacy fields (which are not entities in
the navigator sidebar) are typically wrapped with a bookmark (which IS an
navigable entity).

On the other hand, the avoidance of reading the presumably-LO-exported bookmark
was also a convenience factor that tried to mitigate previous bad export
behaviour.

So I don't really see any reason why not to revert that commit...

-- 
You are receiving this mail because:
You are the assignee for the bug.

Reply via email to