Project

General

Profile

Actions

Bug #15572

closed

Reporting context is not re-evaluated when using multiple directives of the same technique

Added by Félix DALLIDET over 5 years ago. Updated over 2 years ago.

Status:
Released
Priority:
N/A
Category:
Web - Technique editor
Target version:
Severity:
UX impact:
User visibility:
Effort required:
Priority:
0
Name check:
To do
Fix check:
Checked
Regression:

Description

I ended up with missing reports while defining quite simple configuration in the technique editor.
The faulty technique is attached to the ticket.

PB:
When defining multiple directives from this technique only the first instance executed was correctly reporting, all other directives
were reporting wrongly on the "Condition from variable existence" with a component_name set to "None".

This is due to the promiser of _method_reporting_context which was:

Condition from variable existence_context_0

Which is not re-evaluated at each bundle call since only depending on static values: the 0 for the promiser and the class_parameter for the parameters which is common to all directives.
To avoid this, we should keep the incremental suffix but also add an arbitrary value in the promiser, depending on the directive we are executing. For instance, the promiser id.

Which fixed the problem with a promiser like:

"Condition from variable existence_context_${report_data.directive_id}_0" 


Files

Disable kernel module disabled.json (1.12 KB) Disable kernel module disabled.json Félix DALLIDET, 2019-08-23 15:39

Subtasks 2 (0 open2 closed)

Bug #15573: Same a parent ticket for branch 5.1ReleasedVincent MEMBRÉActions
Bug #15576: Don't parse _${report_data.directive} as report componentReleasedNicolas CHARLESActions
Actions

Also available in: Atom PDF