VAR_INVALID_NAME false positive with number-suffixed names

[timbeadle]

if (urlScope.keyExists('id2') && isNumeric(urlScope.id2)) {
	route &= '/:id2';
}

[INFO]: [cflint] VAR_INVALID_NAME: Variable id2 is not a valid name. Please use camelCase or underscores.

Using CFLint v1.3.0.

[ryaneberly]

Does :id2 dereference id2 automatically?

[KamasamaK]

That is just concatenating a string literal, but that would be beside the point. The issue is simply that variable names like the example above or address1 are triggering the *_INVALID_NAME rules.

[ryaneberly]

Thanks @KamasamaK , I agree, i was just curios, since that code will also trigger a var unused message.

[ryaneberly]

This has been a 'feature' almost since the beginning. I had to disable or change almost a dozen tests after I fixed this one. :) However, I agree that ending a number should not be considered invalid naming.

[timbeadle]

Thanks @ryaneberly for the quick turnaround 😄