Minor improvements for the HTML report

The HTML report is a bit more verbose than it needs to be

Variable user_id is not declared with a var statement. Use var or the local scope, or otherwise clarify the scope

In file C:\Users\zac\Documents\GitHub\CFLint\build\libs.\linenumber.cfm, line 2

At line: 2, at column: 2

In function test()

Expression: user_id

• line number is repeated twice, may add the column number to the file reference and remove the At line: 2, at column: 2 line and extra new line?
• the variable user_id could be made bold for clarity
• expression (i.e. code snippets) could be wrapped in <pre></pre>
• ‘In file’ is kind redundant before to the file path
• does the warning need to be repeated in the details section, it’s already the warning column?

Variable user_id is not declared with a var statement. Use var or the local scope, or otherwise clarify the scope

C:\Users\zac\Documents\GitHub\CFLint\build\libs.\linenumber.cfm, line 2, column 2

In function test()

Expression: user_id

[TheRealAgentK]

Fixed that @zspitzer

[TheRealAgentK]

Ha, nothing is easy sometimes.

So, the HTML report gets created by XSLT from Findbugs XML which itself comes out of CFLint XML by XSLT. sigh 😃

In a nutshell - some of the blurbs I get to get into the HTML report can’t be easily changed.

I made two small fixes:

• does the warning need to be repeated in the details section, it’s already the warning column?
• ‘In file’ is kind redundant before to the file path

The other items I’ll get back to in the output/report rewrite for 2.0.