Bug 60529 - FILESAVE: Shape/smiley created on a brand new file isn't exported in docx
Summary: FILESAVE: Shape/smiley created on a brand new file isn't exported in docx
Status: CLOSED WORKSFORME
Alias: None
Product: LibreOffice
Classification: Unclassified
Component: Writer (show other bugs)
Version:
(earliest affected)
3.6.5.2 release
Hardware: All All
: medium normal
Assignee: Not Assigned
URL:
Whiteboard: BSA (target:4.2.0) (target:4.3.0)
Keywords:
Depends on:
Blocks:
 
Reported: 2013-02-09 06:22 UTC by Julien Nabet
Modified: 2013-12-29 09:02 UTC (History)
2 users (show)

See Also:
Crash report or crash signature:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Julien Nabet 2013-02-09 06:22:05 UTC 
Problem description: 
On pc Debian x86-64 with 3.6 sources and master sources updated yesterday, shapes are not saved when exporting in docx

Steps to reproduce:
1. Open a brand new file on Writer
2. Enable drawing toolbar: Menu View/Toolbars/drawing must be checked
3. Add a basic shape or a smiley
4. Menu File/Save as, choose file name and docx extension then ok
5. Reopen the docx file

Current behavior:
No shape/smiley in docx file

Expected behavior:
Shape/smiley present in docx file

Notice that when exporting in .doc, it's ok.

I also noticed these logs during docx export with master sources:
 TODO VMLExport::Commit(), unimplemented id: 135, value: 1, data: [0, (nil)]
TODO VMLExport::Commit(), unimplemented id: 511, value: 524296, data: [0, (nil)]
TODO VMLExport::Commit(), unimplemented id: 450, value: 5282507, data: [0, (nil)]
warn:legacy.tools:6290:1:filter/source/msfilter/eschesdo.cxx:835: ImplEscherExSdr::~ImplEscherExSdr: unwritten SolverContainer
warn:legacy.tools:6290:1:svx/source/unodraw/unopage.cxx:87: SvxDrawPage must be disposed!

              
Operating System: Debian
Version: 3.6.5.2 release
Comment 1 Julien Nabet 2013-02-09 06:32:00 UTC 
Forgot to say it was with a brand new file.

I also reproduced this with Win7 and 3.6.5.2 version and a brand new file, no specific extensions or fonts.

So put it All for envs and New
Comment 2 Julien Nabet 2013-02-09 06:34:38 UTC 
Miklos: one for you? (considering git history of oox/source/export/vmlexport.cxx)
Comment 3 Jorendc 2013-12-29 00:39:06 UTC 
@Julien: looks fixed to me, tested using Windows 8.1 with LibreOffice Version: 4.3.0.0.alpha0+
Build ID: 5be7ec4193b892e5643ff5f3f2e6755319569190
TinderBox: Win-x86@39, Branch:master, Time: 2013-12-27_23:55:02

Can you please verify :)?

Kind regards,
Joren
Comment 4 Julien Nabet 2013-12-29 09:02:07 UTC 
With 4.1.3.2 Debian package it still doesn't work but with master and with 4.2 sources both updated 2 days ago it's ok.