[CodeQuality] Keep parentheses around Ternary operand in SimplifyConditionsRector

[TomasVotruba]

Fixes #8121.

SimplifyConditionsRector inverts a binary op when wrapped in a boolean not, e.g. ! (1 < X) → 1 >= X. When the operand X is a ternary, the format-preserving printer dropped the parentheses. Since the ternary has lower precedence than >=, the result changed meaning:

// before (input) — evaluates false
if (! (1 < (1 ? 2 : 1))) {}

// produced — evaluates true (wrong)
if (1 >= 1 ? 2 : 1) {}

Now createInversedBooleanOp() sets WRAPPED_IN_PARENTHESES on ternary operands, matching the existing precedent in UseIdenticalOverEqualWithSameTypeRector, producing the correct:

if (1 >= (1 ? 2 : 1)) {}

Bug report and failing fixture by @j-dobr in #8121.

Co-authored-by: Jan Dobrovodsky jan.dobrovodsky@promedcs.com

[TomasVotruba]

Thank you for reporting 👍