Issue 89769

Summary: hidden columns/rows shouldn't be exported via RTF
Product: Calc Reporter: andreas.martens
Component: save-exportAssignee: AOO issues mailing list <issues>
Status: CONFIRMED --- QA Contact:
Severity: Trivial    
Priority: P3 CC: gbpacheco, issues, ooo
Version: OOo 3.0 BetaKeywords: needhelp
Target Milestone: ---   
Hardware: All   
OS: All   
Issue Type: PATCH Latest Confirmation in: ---
Developer Difficulty: ---
Attachments:
Description Flags
Patch will hide hidden items in rtfexport none

Description andreas.martens 2008-05-22 08:23:40 UTC 
Follow-up of issue 84699
Fill cell A1, B1 and C1
Hide column B
Select A1-C1
Copy into clipboard
Paste into new Writer document as RTF (paste special)
=> the B-column seems not to be inserted but if you save this document and
reload it, you will see the content of B. When issue 84699 is fixed (CWS
sw30bf05) you will see the hiddden column immediately after pasting (with
minimum width of a column).
Other scenario:
Hide a row (in Calc) and copy/paste (into Writer) a selection which contains
this hidden row
=> The hidden row will be inserted and visible in Writer.

Hidden rows and columns should not be exported in RTF.
Comment 1 gbpacheco 2010-06-23 03:08:49 UTC 
Hi!

Any news about this issue? 

Thanks!
Comment 2 daniel.rentz 2010-06-23 08:00:58 UTC 
Sorry, am busy with other stuff, and never worked on the RTF filter. Nobody
provided a patch, ...
@er: do you know anything about RTF export?
Comment 3 ooo 2010-06-23 12:42:58 UTC 
Implementation is in sc/source/filter/rtf/rtfexp.cxx and pretty straight
forward. Basically each row would have to be checked for hidden or filtered
before calling ScRTFExport::WriteRow() and there each column before calling
ScRTFExport::WriteCell(). Note that places that use pCellX will need attention
as well. In fact pCellX could be initialized in ScRTFExport::WriteTab() to set a
column width to 0 if a column is hidden and not write cells in columns of width 0.

Adding keyword easy2dev so this issue shows up in the query at the bottom of
http://sc.openoffice.org/ and someone can jump on it and provide a patch.
Comment 4 gbpacheco 2010-06-30 01:50:11 UTC 
Hi! 

 I think 21604 and 89769 are the same problem, correct?

Thanks!
Comment 5 ooo 2010-06-30 11:37:57 UTC 
*** Issue 21604 has been marked as a duplicate of this issue. ***
Comment 6 scf 2011-02-11 13:20:26 UTC 
Hi all,

this is my very first contibution to ooo, so please be patient :). I've created
a patch with hg diff, hope that works. Don't hesitate to mail me if something's
wrong, I'm here to learn.

Greetings
Comment 7 scf 2011-02-11 13:22:12 UTC 
Created attachment 75817 [details]
Patch will hide hidden items in rtfexport
Comment 8 daniel.rentz 2011-02-11 13:33:06 UTC 
Eike, do you know the RTF filter and can review the patch?
Comment 9 Rob Weir 2013-03-11 15:02:08 UTC 
I'm adding this comment to all open issues with Issue Type == PATCH.  We have 220 such issues, many of them quite old.  I apologize for that.  

We need your help in prioritizing which patches should be integrated into our next release, Apache OpenOffice 4.0.

If you have submitted a patch and think it is applicable for AOO 4.0, please respond with a comment to let us know.

On the other hand, if the patch is no longer relevant, please let us know that as well.

If you have any general questions or want to discuss this further, please send a note to our dev mailing list:  dev@openoffice.apache.org

Thanks!

-Rob
Comment 10 Marcus 2017-05-20 11:11:43 UTC 
Reset assigne to the default "issues@openoffice.apache.org".