Inheriting code is always enlightening 🙂

For the issue with the expression on the connection string, there's no real value in having it set up that way if the expression is just static text. Unless that connection string is being dynamically built in the expression, it's adding some complexity without any added value.

On the variable issue - That's got to be an oversight when it was first developed. That expression doesn't really do anything.