CFLint

CFLint copied to clipboard

Literal get occurs several times in one or more files. Consider giving it a name and not hard coding values.cflint(GLOBAL_LITERAL_VALUE_USED_TOO_OFTEN) reports lines getRandom, getMaax, getMin, getMedian, but not before that....

dawesi

Migrate from Travis CI to GitHub Actions

11

I will start by saying I'm not really familiar with Travis CI. I noticed the builds were failing when I submitted a PR and saw the results of [Build 1390](https://travis-ci.org/cflint/CFLint/builds/658988680)....

KamasamaK

Suggestion: exit code

9

Given input that produces no warnings, cflint would exit with code 0. Given input that produces warnings, cflint would exit with a non-zero code, perhaps 1. This would allow cflint...

jaredbeck

I am running the latest DEV branch, and here is how I am calling CFLINT `java -jar CFLint-1.5.1-SNAPSHOT-all.jar -folder ./ -filterFile cflintexclude.json -d` .cflintrc is in the folder `/Users/aj/Projects/test/` ```...

andyj

Report trailing comma after last array element

5

This feature proposal refers to [the issue i submitted](https://github.com/KamasamaK/vscode-cfml/issues/34) to vscode-cfml. Since the issue my proposal addresses is engine-specific, it is rather an improvement for CFLint than for the CFML...

dominikbraun

In CommandBox: try { application action="update" datasources="#datasourceStruct#"; } catch (any err) { print.line(err.message); } cflint threw "cfml.parsing.reporting.ParseException: try statement must include at least one catch clause or a finally clause."...

djcard

Make offsets more accurate

2

ryaneberly

new rules: AVOID_QUERY_IN_LOOP

1

AVOID_QUERY_IN_LOOP, It's not a good practice to query a DB inside loop like `` and ` ` - that increase load to DB and response time. https://codeutopia.net/blog/2010/10/07/optimizing-sql-removing-queries-inside-loops/

Vintic

The word "case" inside of a for loop causes a NPE

1

Is it possible to update the parser to allow the word "case" to be used as variable names and inside for loops. Take the example code below, it will cause...

Yamaha32088

Given the code ```cfml ``` The rules `VAR_HAS_PREFIX_OR_POSTFIX` and `VAR_IS_TEMPORARY` are reported not only for the assignment but for all usages of `jsonObj`. I've only included one usage for simplicity....

KamasamaK