Actions
Bug #15167
closedInvalid detection of empty strings for edition in Techniques
Pull Request:
Severity:
UX impact:
User visibility:
Effort required:
Priority:
0
Name check:
To do
Fix check:
Checked
Regression:
Description
In the classical agent, we detect if we are to edit files if string are not empty
So, we have a lot of
"edit_content" not => strcmp("${content}", "");
(this is mostly due to historical reasons)
However, in recent version of agents, this fails if the content contains a variable, as promise is not evaluated
for instance
"str1" string => "${foo bar baz}"; "cmp1" expression => strcmp("${str1}", ""); "notcmp1" not => strcmp("${str1}", "");
neither cmp1 nor notcmp1 is defined
we should replace all not => strcmp by the opposite logic in all techniques/generic method
Actions