Bug 68020

Summary: handling of body of cell of table: inconsistent depending on type and style
Product: LibreOffice Reporter: Lionel Elie Mamane <lionel>
Component: WriterAssignee: Not Assigned <libreoffice-bugs>
Status: NEW --- QA Contact:
Severity: normal    
Priority: medium CC: jbfaure, jmadero.dev, mst.fdo, qubit
Version: 4.2.0.0.alpha0+ Master   
Hardware: All   
OS: All   
Whiteboard: filter:odt
i915 platform: i915 features:
Bug Depends on: 68022, 68023, 68024, 68026, 60842, 62147    
Bug Blocks: 67930    
Attachments: test case

Description Lionel Elie Mamane 2013-08-12 14:13:45 UTC 
@mstahl: continuation of the saga of bug 60842 and bug 62147.

Tracker bug.

Consider the attached document.

Compare row April Prime and April Secundo.

Why does the second column of April Prime display the office:value,
but in April Secundo the <text:p> child? The only difference between
the two is the order between the <text:variable-set> and the actual
PCDATA (text). Surely that should not make any difference.

I'm surprised that on row April Prime, the second and third column
display differently. Depending on the format, the office:value
attribute or the text:p child are used? I'd expect that to be a bug.
It should *always* take the office:value attribute.

However, in the first three rows, the body may not be always displayed,
but it is always *treated* in some way. In particular, <text:variable-set>
tags are treated. See that variables (foo|bar)Apr(Pri|Sec) as well as
(foo|bar)AugPri are set, and displayed in the page footer:
 f41-b41,f42-b42,f81-b81

On the other hand, for cells with office:value-type="string"
(rows August Secundo and August Tertio), as soon as there is
an office:string-value (that is, the second column), the
<text:variable-set> in the body are not evaluated; the variables
fooAug(Sec|Ter) stay at "0". This is what causes bug 67930. In the
absence of an office:string-value, however, the body is displayed
and executed, and the barAug(Sec|Ter) variables do get set.
Comment 1 Lionel Elie Mamane 2013-08-12 14:15:06 UTC 
Created attachment 83968 [details]
test case
Comment 2 Robinson Tryon (qubit) 2014-02-03 18:37:49 UTC 
Remove comma from whiteboard.
Comment 3 Robinson Tryon (qubit) 2014-02-03 18:40:40 UTC 
(In reply to comment #0)
> @mstahl: continuation of the saga of bug 60842 and bug 62147.
> 
> Tracker bug.

Lionel - Are you just waiting for info from mstahl here, or is there some other reason why this bug is tagged as NeedAdvice?

Thanks,
Comment 4 Lionel Elie Mamane 2014-02-04 06:46:13 UTC 
(In reply to comment #3)
> Lionel - Are you just waiting for info from mstahl here, or is there some
> other reason why this bug is tagged as NeedAdvice?

JB Faure tagged it as NeedAdvice, not sure why. I had hoped that mstahl would fix the bug :)
Comment 5 Jean-Baptiste Faure 2014-02-05 21:13:08 UTC 
(In reply to comment #4)
> (In reply to comment #3)
> > Lionel - Are you just waiting for info from mstahl here, or is there some
> > other reason why this bug is tagged as NeedAdvice?
> 
> JB Faure tagged it as NeedAdvice, not sure why. 

Because of the "why" at the fifth line of your bug description.
If I misunderstood (it is possible ;-) ), please feel free to remove the tag.

Best regards. JBF
Comment 6 Robinson Tryon (qubit) 2014-07-03 19:51:46 UTC 
Question answered, changing NEEDINFO -> NEW.

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.