Description: Cutting image in frame in table saved as DOCX doesn't work Steps to Reproduce: 1. Open the attached file 2. Save as DOCX 3. File reload 4. Select the image frame in table (figure 1) & CTRL+X Actual Results: Image still present Expected Results: Disappearing Reproducible: Always User Profile Reset: No Additional Info: Found in 7.1 6.0 4.4.7.2 cut crashes in 4.3 export somewhat broken in 4.2
Created attachment 164282 [details] Example file
I assume the flaw started somewhere around the crash or should at minimum give some pointer
Fine with DOC
Created attachment 167801 [details] The docx versions of the original file in 7.0 and 7.2alpha after cutting the frame Seems to work now in: Version: 7.2.0.0.alpha0+ (x64) Build ID: 4e63ec27b69fa01ff610c894c9fbf05c377a6179 CPU threads: 4; OS: Windows 6.3 Build 9600; UI render: default; VCL: win Locale: en-US (hu_HU); UI: en-US Calc: CL Was bad in: Version: 7.0.0.3 (x64) Build ID: 8061b3e9204bef6b321a21033174034a5e2ea88e CPU szálak: 4; OS: Windows 6.3 Build 9600; Felületmegjelenítés: Skia/Raster; VCL: win Locale: hu-HU (hu_HU); UI: hu-HU Calc: CL
Seems to be fixed since: https://cgit.freedesktop.org/libreoffice/core/commit/?id=da4f9b77a6cd39b1ae5babdd476d1575c8b9371c author Miklos Vajna <vmiklos@collabora.com> 2020-09-07 21:23:07 +0200 committer Miklos Vajna <vmiklos@collabora.com> 2020-09-08 09:11:24 +0200 tdf#135149 sw: fix deleting textbox of as-char shapes Thanks Miklos for fixing this one too :) *** This bug has been marked as a duplicate of bug 135149 ***
@Xisco Is a separate unit test required for this case or is this covered by the unit test for bug 135149.
(In reply to Telesto from comment #6) > @Xisco > Is a separate unit test required for this case or is this covered by the > unit test for bug 135149. noup, see the commit // Without the accompanying fix in place, this test would have failed with: // - Expected: 0 // - Actual : 2 // i.e. the fly frame of the shape and the inner Writer image was not deleted. CPPUNIT_ASSERT_EQUAL(static_cast<sal_Int32>(0), nActual);