Bug 93174 - Writer - images inserted as background (page or paragraph) are lost on save as OOXML (docx)
Summary: Writer - images inserted as background (page or paragraph) are lost on save a...
Status: RESOLVED DUPLICATE of bug 90273
Alias: None
Product: LibreOffice
Classification: Unclassified
Component: filters and storage (show other bugs)
Version:
(earliest affected)
4.4.2.2 release
Hardware: Other All
: medium normal
Assignee: Not Assigned
URL:
Whiteboard:
Keywords: bibisected, bisected, filter:docx, regression
Depends on:
Blocks:
 
Reported: 2015-08-06 04:47 UTC by yousifjkadom
Modified: 2016-10-13 21:51 UTC (History)
6 users (show)

See Also:
Crash report or crash signature:


Attachments
1 (99.29 KB, image/png)
2015-08-06 04:47 UTC, yousifjkadom
Details
2 (98.61 KB, image/png)
2015-08-06 04:47 UTC, yousifjkadom
Details
3 (141.54 KB, image/png)
2015-08-06 04:48 UTC, yousifjkadom
Details
4 (98.97 KB, image/png)
2015-08-06 04:48 UTC, yousifjkadom
Details
clip of 5.0.2.2 on Windows, insert as background from Gallery -- seems correct (147.53 KB, image/png)
2015-09-26 17:03 UTC, V Stuart Foote
Details
Vedio recording (3.40 MB, video/mp4)
2015-11-05 17:20 UTC, yousifjkadom
Details

Note You need to log in before you can comment on or make changes to this bug.
Description yousifjkadom 2015-08-06 04:47:16 UTC
Created attachment 117693 [details]
1

Hi. Please look to pictures from 1 to 4 one after other. Insert from gallery as background as page does not saved. 

While insert from gallery as background as pragraph does not worked at all. At a time it be like bellow but at other time does not appear at all. At present time it does not appear even before saving, so I did not take picture for this case.

Pictures here are for as "page" only.
Comment 1 yousifjkadom 2015-08-06 04:47:55 UTC
Created attachment 117694 [details]
2
Comment 2 yousifjkadom 2015-08-06 04:48:27 UTC
Created attachment 117695 [details]
3
Comment 3 yousifjkadom 2015-08-06 04:48:57 UTC
Created attachment 117696 [details]
4
Comment 4 Regina Henschel 2015-08-06 19:06:03 UTC
I cannot confirm it for pages. But I can confirm it for paragraph background.

I use Version: 5.1.0.0.alpha1+
Build ID: 62e2fae93e8894f73560a30ae1e752cbd4c001ad
TinderBox: Win-x86@62-merge-TDF, Branch:MASTER, Time: 2015-07-29_04:40:44
Locale: de-DE (de_DE)

It is OK in Version: 4.2.5.2
Build ID: 6ff819b65674ae6c83f3cbab9e4a4c2b292a7a94

It seems that this feature got lost, when the special paragraph background dialog was removed in favor of the general area dialog.
Comment 5 yousifjkadom 2015-09-26 16:38:51 UTC
I test version 5.0.2 This bug is still existing for BOTH paragraph & page.

I can not understand why it is not confirmed for page!!
Comment 6 V Stuart Foote 2015-09-26 17:03:43 UTC
Created attachment 119046 [details]
clip of 5.0.2.2 on Windows, insert as background from Gallery -- seems correct

On Windows 10 Pro 64-bit en-US with
Version: 5.0.2.2 (x64)
Build ID: 37b43f919e4de5eeaca9b9755ed688758a8251fe
Locale: en-US (en_US)

I also can not confirm.

Working from from Gallery, context menu of object...

1. Insert as Background for page is correct.

2. Insert as Background for Paragraph is also correct.

See attached clip

Start with A01 - Arrow-Gray-Left as page background

3rd paragraph, add line continuations, apply Bullet 15 - Arrow-Blue as paragraph background.

Then select multiple paragraphs (5) apply A06 - Arrow-Red-Right as paragraph background.
Comment 7 yousifjkadom 2015-09-27 08:58:35 UTC
I test bug on windows 7 ultimate SP-1 x32 bit.

Please check if bug limited to 32 bit version of windows, though this is unusual!
Comment 8 raal 2015-10-21 16:12:15 UTC
I can confirm with Version: 5.1.0.0.alpha1+
Build ID: 51df957e1a40d2f3511345c1600c05dd35f34b6b
TinderBox: Linux-rpm_deb-x86_64@70-TDF, Branch:master, Time: 2015-10-19_04:22:32
and LO 4.4.2

Insert as background - Page or paragraph
save as .docx (see reporter's printscreen)
reopen

background lost


For .odt with Version: 5.1.0.0.alpha1+
Insert as background - Page -  work

Insert as background - paragraph -  doesn't work

=>

@Regina, comment 4 -  it's another bug for format .odt. Can you create a new bug?

@yousifjkadom, is your bug for docx format?
Comment 9 yousifjkadom 2015-11-05 17:20:58 UTC
Created attachment 120297 [details]
Vedio recording
Comment 10 yousifjkadom 2015-11-05 17:24:11 UTC
Hi. Bug is still existing in version 5.0.3

Please see attached file number 120297 (a vedio record made by me) to confirm the bug.
Comment 11 raal 2015-11-19 13:29:19 UTC
(In reply to yousifjkadom from comment #10)
> Hi. Bug is still existing in version 5.0.3
> 
> Please see attached file number 120297 (a vedio record made by me) to
> confirm the bug.

Hello, bug is already confirmed = status NEW.
Comment 12 V Stuart Foote 2015-11-19 14:05:46 UTC
The background graphics are correctly inserted into the canvas of an open document in LibreOffice.

But export filter currently losses the inserted background images on save as in this table:

Image 
backgnd        export/save format
object   OOXML     DOC(97-2003)    ODF
         ------    ------------    ---
Page     lost      present         present
Para     lost      lost            lost

The background loss from paragraphs in ODF text documents as in comment 4 is far more troubling to me.
Comment 13 raal 2016-02-10 21:13:56 UTC
(In reply to V Stuart Foote from comment #12)
> The background graphics are correctly inserted into the canvas of an open
> document in LibreOffice.
> 
> But export filter currently losses the inserted background images on save as
> in this table:
> 
> Image 
> backgnd        export/save format
> object   OOXML     DOC(97-2003)    ODF
>          ------    ------------    ---
> Page     lost      present         present
> Para     lost      lost            lost
> 
> The background loss from paragraphs in ODF text documents as in comment 4 is
> far more troubling to me.

Bibisected - insert paragraph background from gallery; save as odt; reload; paragraph background lost

This seems to have begun at the below commit.
Adding Cc: to Armin Le Grand ; Could you possibly take a look at this one?
Thanks
 e642812606be49244f2f775e90169fbd1be29a97 is the first bad commit
commit e642812606be49244f2f775e90169fbd1be29a97
Author: Matthew Francis <mjay.francis@gmail.com>
Date:   Sat Mar 14 22:28:59 2015 +0800

    source-hash-7d9bb549d498d6beed2c4050c402d09643febdfa
    
    commit 7d9bb549d498d6beed2c4050c402d09643febdfa
    Author:     Armin Le Grand <alg@apache.org>
    AuthorDate: Mon Jun 2 15:00:50 2014 +0000
    Commit:     Miklos Vajna <vmiklos@collabora.co.uk>
    CommitDate: Tue Jul 1 13:30:09 2014 +0200
    
        Related: #i124638# Second step of DrawingLayer FillAttributes...
    
        for Writer objects, now added support for Paragraph and PageStyle (including
        Header and Footer) for direct attributes and style attributes
    
        (cherry picked from commit cc25c58f7052827bfebdc9fbeec668c8fa29ed1b)
    
        Conflicts:
        	cui/source/factory/dlgfact.cxx
        	cui/source/factory/dlgfact.hxx
        	cui/source/inc/bbdlg.hxx
        	cui/source/inc/cuires.hrc
        	cui/source/tabpages/bbdlg.cxx
        	cui/source/tabpages/bbdlg.src
        	cui/source/tabpages/page.cxx
        	cui/source/tabpages/tparea.cxx
        	include/svx/pagectrl.hxx
        	include/svx/svxdlg.hxx
        	include/svx/svxids.hrc
        	include/xmloff/PageMasterStyleMap.hxx
        	include/xmloff/prstylei.hxx
        	include/xmloff/txtprmap.hxx
        	svx/Package_inc.mk
        	svx/inc/svx/hdft.hxx
        	svx/source/dialog/hdft.cxx
        	svx/source/dialog/pagectrl.cxx
        	svx/source/tbxctrls/tbxcolorupdate.cxx
        	svx/source/unodraw/unobrushitemhelper.cxx
        	sw/Library_sw.mk
        	sw/inc/fillattributes.hxx
        	sw/inc/format.hxx
        	sw/inc/frmatr.hxx
        	sw/inc/frmfmt.hxx
        	sw/inc/hintids.hxx
        	sw/inc/hints.hxx
        	sw/inc/ndtxt.hxx
        	sw/inc/node.hxx
        	sw/inc/swunohelper.hxx
        	sw/inc/unobrushitemhelper.hxx
        	sw/inc/unoprnms.hxx
        	sw/source/core/attr/format.cxx
        	sw/source/core/attr/hints.cxx
        	sw/source/core/doc/docdesc.cxx
        	sw/source/core/doc/docdraw.cxx
        	sw/source/core/doc/docfmt.cxx
        	sw/source/core/doc/docnew.cxx
        	sw/source/core/doc/docredln.cxx
        	sw/source/core/doc/poolfmt.cxx
        	sw/source/core/doc/visiturl.cxx
        	sw/source/core/docnode/node.cxx
        	sw/source/core/inc/frame.hxx
        	sw/source/core/inc/frmtool.hxx
        	sw/source/core/inc/rolbck.hxx
        	sw/source/core/layout/atrfrm.cxx
        	sw/source/core/layout/fillattributes.cxx
        	sw/source/core/layout/findfrm.cxx
        	sw/source/core/layout/paintfrm.cxx
        	sw/source/core/txtnode/ndtxt.cxx
        	sw/source/core/txtnode/thints.cxx
        	sw/source/core/txtnode/txtedt.cxx
        	sw/source/core/undo/rolbck.cxx
        	sw/source/core/unocore/swunohelper.cxx
        	sw/source/core/unocore/unoframe.cxx
        	sw/source/core/unocore/unomap.cxx
        	sw/source/core/unocore/unoparagraph.cxx
        	sw/source/core/unocore/unoprnms.cxx
        	sw/source/core/unocore/unostyle.cxx
        	sw/source/ui/chrdlg/paradlg.src
        	sw/source/ui/chrdlg/pardlg.cxx
        	sw/source/ui/fmtui/tmpdlg.cxx
        	sw/source/ui/fmtui/tmpdlg.src
        	sw/source/uibase/app/docst.cxx
        	sw/source/uibase/app/docstyle.cxx
        	sw/source/uibase/frmdlg/colex.cxx
        	sw/source/uibase/shells/basesh.cxx
        	sw/source/uibase/shells/textsh1.cxx
        	sw/source/uibase/uiview/viewstat.cxx
        	sw/source/uibase/utlui/uitool.cxx
        	xmloff/inc/PageMasterImportContext.hxx
        	xmloff/inc/xmloff/XMLShapeStyleContext.hxx
        	xmloff/source/draw/XMLShapeStyleContext.cxx
        	xmloff/source/draw/sdpropls.hxx
        	xmloff/source/style/PageMasterExportPropMapper.cxx
        	xmloff/source/style/PageMasterImportContext.cxx
        	xmloff/source/style/PageMasterPropHdlFactory.cxx
        	xmloff/source/style/PageMasterStyleMap.cxx
        	xmloff/source/style/prstylei.cxx
        	xmloff/source/text/txtexppr.cxx
        	xmloff/source/text/txtprhdl.cxx
        	xmloff/source/text/txtprmap.cxx
        	xmloff/source/text/txtstyli.cxx
    
        Conflicts:
        	svx/source/dialog/hdft.cxx
        	sw/source/core/doc/visiturl.cxx
        	sw/source/core/txtnode/thints.cxx
        	sw/source/core/txtnode/txtatr2.cxx
        	sw/source/core/unocore/unostyle.cxx
    
        Change-Id: I7e8779db6c0cbd1e242b63eab888f468f2de509a
Comment 14 Thomas Bertels 2016-08-30 12:05:14 UTC
+docx keyword
Still there on 5.2.0.4
The image doesn't need to come from gallery.
Comment 15 Xisco Faulí 2016-10-13 21:51:42 UTC

*** This bug has been marked as a duplicate of bug 90273 ***