[facontidavide]

Fixes #1246.

Problem

Since #1121 (first released in 4.9.1), a numeric literal passed to a SubTree, e.g. <SubTree ID="Sub" mode="1"/>, is stored in the child blackboard as int/int64_t/double instead of std::string. That locks the entry type, so:

• the same value can no longer be read by a std::string port: tree creation fails with "port [mode] was initially created with type [int] and, later type [std::string] was used";
• a double port reading "1" fails the same way, and custom convertFromString<T> parsing is bypassed;
• the spelling is lost ("007" becomes 7, "1e3" becomes 1000).

Fix

• xml_parsing.cpp: literals are stored as strings again, as before 4.9.1. Each port converts the text to its own type.
• scripting/operators.hpp: the goal of Preserve numeric types for literal subtree port values #1121 (arithmetic on numeric literals in Script) moves into the evaluator. In + - * /, a string operand mixed with a number is converted if the whole string is a number (integer first, then parseDouble with full consumption, the same grammar Preserve numeric types for literal subtree port values #1121 used). "42abc", "true" and " 42" are rejected, so "text" + 1 still throws. String+string, .., comparisons, bitwise and logic operators are unchanged.

The #1121 test SubTree.LiteralNumericPortsPreserveType is kept unchanged and passes.

Behavior change

Compared to 4.9.1 – 4.10.0, these go back to pre-4.9.1 behavior:

• two literal operands are strings: a + b with a="1" b="2" gives "12", and a < b with a="10" b="2" compares text;
• int_val += 1, -int_val, int_val & 1, int_val && 1 and a bare <Precondition if="int_val"> on such a literal need an explicit number again.

No API/ABI change: only the inline evaluator in a public header and a .cpp change.

Tests

• SubTree.LiteralPortsAreReadByAnyPortType: the issue's case. mode="1" is read by string, int and double ports, and code="007" keeps its spelling.
• ParserTest.NumericStringArithmetic: both operand orders, rejected strings, string+string concatenation unchanged.

Both new tests fail on master and pass with the fix. Full suite: 574 passed.

🤖 Generated with Claude Code

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud