Actions
Bug #15572
closedReporting context is not re-evaluated when using multiple directives of the same technique
Status:
Released
Priority:
N/A
Assignee:
Category:
Web - Technique editor
Target version:
Pull Request:
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
Actions