Bug 40

Title: Nested expression fails since 0.7.1
Type: behavior Severity: major
Components: opendocument Versions:
Status: closed Resolution:
Dependencies: Superseder:
Assigned To: ced Nosy List: ced, nblock, roundup-bot
Priority: high Keywords: patch

Created on 2017-10-27 12:45 by nblock, last changed 2017-11-14 17:36 by roundup-bot.

File name Uploaded Description Edit Remove
purchase.odt nblock, 2017-10-30 08:16 Minimal purchase report
msg60 Author: [hidden] (nblock) Date: 2017-10-27 12:45
we just updated our Tryton 4.2 installation to the latest
minor patch release and relatorio was updated from 0.7.0 to
0.7.1 during this process. We noticed that the following
expression no longer works in all of our reports:

<if test="">
Phone: <>

The same is true for mobile, email and fax.

The issue was solved by reverting the following commit on
top of relatorio 0.7.1:
msg61 Author: [hidden] (ced) Date: 2017-10-27 16:25
Could you provide a minimal example?
msg62 Author: [hidden] (nblock) Date: 2017-10-30 08:16
The minimal purchase example attached requires that the
employee which creates the purchase report is linked to a
party. This party has the linked contact information set to
some value.

When this report is rendered with relatorio 0.7.1, only the
full name is printed, all other fields are missing.
msg63 Author: [hidden] (ced) Date: 2017-10-30 23:32
Indeed the problem comes from rev 7966fd8fce98, the test
about the length is placed on the wrong element which makes
the guess type apply even if the cell is composed of many
elements. So the cell receives the attribute of the first
element and hiding the others.
Here is review42791002 which should fix the problem.
msg64 Author: [hidden] (roundup-bot) Date: 2017-11-14 17:36
New changeset fe33115e001f by C├ędric Krier in branch 'default':
Use the cell element to test number of elements in the cell
Date User Action Args
2017-11-14 17:36:06roundup-botsetstatus: new -> closed
nosy: + roundup-bot
messages: + msg64
2017-10-30 23:32:55cedsetseverity: normal -> major
messages: + msg63
priority: high
assignee: ced
keywords: + patch
type: behavior
2017-10-30 08:16:28nblocksetfiles: + purchase.odt
messages: + msg62
2017-10-27 16:25:23cedsetnosy: + ced
messages: + msg61
2017-10-27 12:45:43nblockcreate