I haven't checked out the code yet, but isn't `value` a string here, and therefore `!isNaN(value)` will _always_ return true (because a string is not `NaN`)?