ml_AttributeIsString() defines String as 1 and Integer as 0.
The switch/case was just doing the opposite...