JsonScalarExpression.VisitChildren() has wrong loop logic

[roji]

In JsonScalarExpression.VisitChildren():

for (var j = 0; j < i; i++)
{
    newPath[j] = Path[j];
}

Note the i/j confusion in the for loop (🤦)

[roji]

Fixed for 10.0 via #37190 (to make the 10.0.1 deadline).

This bug exists also in 8.0 and 9.0, but is quite an edge case: a JSON array index expression needs to be changed in a visitor, triggering the rewriting the JsonScalarExpression. As we've received zero user reports so far, I'll go ahead and close this - but we can decide to backport in the future.