Bug 80410 - FILESAVE: Embedded Open XML DOCX object inside DOCX converts to picture when opened in MSO
Summary: FILESAVE: Embedded Open XML DOCX object inside DOCX converts to picture when ...
Status: RESOLVED FIXED
Alias: None
Product: LibreOffice
Classification: Unclassified
Component: Writer (show other bugs)
Version: 3.6.7.2 release
Hardware: Other All
: medium normal
Assignee: Michael Stahl
QA Contact:
URL:
Whiteboard: BSA interoperability target:4.4.0 tar...
Keywords:
: 80408 (view as bug list)
Depends on:
Blocks:
 
Reported: 2014-06-23 15:24 UTC by Mark Kaplan
Modified: 2014-07-16 13:46 UTC (History)
2 users (show)

See Also:
i915 platform:
i915 features:


Attachments
A DOCX file which has an embedded DOCX object (Open XML Word 2010 document) (24.15 KB, application/vnd.openxmlformats-officedocument.wordprocessingml.document)
2014-06-23 15:24 UTC, Mark Kaplan
Details

Description Mark Kaplan 2014-06-23 15:24:39 UTC
Created attachment 101597 [details]
A DOCX file which has an embedded DOCX object (Open XML Word 2010 document)

Problem description:
When loading a DOCX into Writer that has an embedded DOCX object, and saving it to a NEW.DOCX - the embedded DOCX is being converted to a picture (when opening the NEW.DOCX in Word 2010 and selecting the embedded object, the 'Picture Tools' contextual menu appears). 

Steps to reproduce:
1. Open attached DOCX in LO
2. Save as NEW.DOCX
3. Open NEW.DOCX in Word - the embedded DOCX had been converted to a picture

Current behavior:
Embedded DOCX object is converted to a picture when round tripped in LO

Expected behavior:
Embedded DOCX object should be preserved when round tripped in LO
              
Operating System: All
Version: 4.4.0.0.alpha0+ Master
Comment 1 Jay Philips 2014-06-24 23:14:36 UTC
Confirmed that the export from Linux Mint in 3.6 - 4.2, 4.3.0 and master, reopened in MSO 2013 shows as a picture.

Opening the saved .docx from 4.1 and 4.2 in MSO shows two pictures rather than one.

Reopening the saved .docx back in LibO doesnt show the OLE in version 3.6 - 4.2, including 4.2.6. Only reopening the saved .docx in 4.3 and master works.
Comment 2 Jay Philips 2014-06-25 02:13:54 UTC
*** Bug 80408 has been marked as a duplicate of this bug. ***
Comment 3 Commit Notification 2014-07-10 21:06:07 UTC
Michael Stahl committed a patch related to this issue.
It has been pushed to "master":

http://cgit.freedesktop.org/libreoffice/core/commit/?id=3ffb06e76be9be7499f12973139949ddbf6297b9

fdo#80410: sw: DOCX export: support DOCX embedded objects



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:
http://wiki.documentfoundation.org/Testing_Daily_Builds
Affected users are encouraged to test the fix and report feedback.
Comment 4 Michael Stahl 2014-07-10 21:06:52 UTC
this doesn't look like a regression to me, because this never worked:
the OLE was always either lost or degraded to a picture.

but since it was so easy i had a fix already before noticing that...
Comment 5 Jay Philips 2014-07-10 21:39:06 UTC
Well the regression was pointed at 4.1 & 4.2 shows two pictures rather than one, as it worked in 4.0, and hoped that a 4.2 fix might be possible. :)
Comment 6 Commit Notification 2014-07-11 14:37:07 UTC
Michael Stahl committed a patch related to this issue.
It has been pushed to "master":

http://cgit.freedesktop.org/libreoffice/core/commit/?id=46d9724369ba4f248eab0133d396bd8f6a4453c6

fdo#80410: let's add a unit test too...



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:
http://wiki.documentfoundation.org/Testing_Daily_Builds
Affected users are encouraged to test the fix and report feedback.
Comment 7 Michael Stahl 2014-07-11 14:58:23 UTC
> Well the regression was pointed at 4.1 & 4.2 shows two pictures rather than one,
> as it worked in 4.0, and hoped that a 4.2 fix might be possible. :)

i've looked at relevant changes and there were a lot of commits
related to OLE export between 4.2 and 4.3, i don't think it's
a good idea to backport that much to the 4.2 branch now.

but i've backported a couple bugfixes to the 4.3 branch to get
various file types properly exported.
Comment 8 Commit Notification 2014-07-11 15:22:39 UTC
Michael Stahl committed a patch related to this issue.
It has been pushed to "libreoffice-4-3":

http://cgit.freedesktop.org/libreoffice/core/commit/?id=589b256452e729e05d9610f10fb2408810e56eb6&h=libreoffice-4-3

fdo#80410: sw: DOCX export: support DOCX embedded objects


It will be available in LibreOffice 4.3.1.

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:
http://wiki.documentfoundation.org/Testing_Daily_Builds
Affected users are encouraged to test the fix and report feedback.
Comment 9 Jay Philips 2014-07-16 07:08:38 UTC
Hi Michael,

I had closed bug 80408 as it was exactly the same as this one, but instead had an embedded doc file. Was this also fixed for embedded doc files?
Comment 10 Michael Stahl 2014-07-16 13:46:12 UTC
(In reply to comment #9)
> I had closed bug 80408 as it was exactly the same as this one, but instead
> had an embedded doc file. Was this also fixed for embedded doc files?

it was, but by a different commit; every file type needs to be explicitly recognized and the right mime-type set; i'll add some fixed bugs to see also


Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.