Bug 152038 - assert and mangled/missing field on import from docx (structured document tag?)
Summary: assert and mangled/missing field on import from docx (structured document tag?)
Status: NEW
Alias: None
Product: LibreOffice
Classification: Unclassified
Component: Writer (show other bugs)
Version:
(earliest affected)
7.3.7.2 release
Hardware: All All
: medium normal
Assignee: Not Assigned
URL:
Whiteboard:
Keywords: bisected
Depends on:
Blocks: Fields
  Show dependency treegraph
 
Reported: 2022-11-14 10:52 UTC by Caolán McNamara
Modified: 2022-11-17 12:46 UTC (History)
4 users (show)

See Also:
Crash report or crash signature:


Attachments
comparisons of word and LibreOffice 7.2 and 7.3 (267.97 KB, image/png)
2022-11-14 10:53 UTC, Caolán McNamara
Details
source document (19.58 KB, application/vnd.openxmlformats-officedocument.wordprocessingml.document)
2022-11-14 10:53 UTC, Caolán McNamara
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Caolán McNamara 2022-11-14 10:52:45 UTC
Description:
The attached document asserts in dbgutil for me with:
sw/source/core/txtnode/swfont.cxx:1064: Size SwSubFont::GetTextSize_(SwDrawTextInfo &): Assertion `!"this is presumably dead code"'

Steps to Reproduce:
1. Load Document

Actual Results:
dbgutil build asserts, release build shows fieldname and tags as content

Expected Results:
Ideally, field result of "February 14, 2014" as seen in msword screenshot


Reproducible: Always


User Profile Reset: No

Additional Info:
screenshot of word, LibreOffice 7.2 and 7.3 attached.

bisect of the LibreOffice 7.2 to 7.3 changes gives:

commit a4432eb0946c0bc775b3d30b634bef5d66544f8d
Date:   Wed Nov 24 14:50:12 2021 +0300

    tdf#104823: support for sdt plain text fields
    
    This is a squashed commit containing set of changes:
Comment 1 Caolán McNamara 2022-11-14 10:53:30 UTC
Created attachment 183585 [details]
comparisons of word and LibreOffice 7.2 and 7.3
Comment 2 Caolán McNamara 2022-11-14 10:53:51 UTC
Created attachment 183586 [details]
source document
Comment 3 Miklos Vajna 2022-11-14 11:11:35 UTC
Yes, I can reproduce the problem. The document has an <w:sdt> but then inside that, there is a FORMTEXT field.

Additionally, the assert is added in commit 19a559b0ec9b806519c405651d6d2b2e14712b4a (sw: SwSubFont::GetTextSize_() assert on presumably dead code, 2019-10-14), perhaps Michael S and Vasily should agree if that assert is valid or not.
Comment 4 Vasily Melenchuk (CIB) 2022-11-17 11:36:11 UTC
Problem was also reproduced.
Not sure about assert by itself: it looks unrelated, because we have problems much earlier during insertion of field into sdt field: this wasn't expected.

No suggestions on fixing yet.