Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Running on folder causes nonexistent issues to be produced #463

Closed
rcastagno opened this issue Aug 30, 2017 · 4 comments
Closed

Running on folder causes nonexistent issues to be produced #463

rcastagno opened this issue Aug 30, 2017 · 4 comments
Assignees
Labels
Milestone

Comments

@rcastagno
Copy link
Contributor

Attached files, if cflinted singularly produced expected result, but when evaluated together produced a bogus error:

  <issue severity="WARNING" id="EXCESSIVE_ARGUMENTS" message="EXCESSIVE_ARGUMENTS" category="CFLint" abbrev="EA">
    <location file="C:\temp\tesisquare\vcs\cores\framework\ttt\test_something.cfc" fileName="test_something.cfc" function="" column="0" line="3" message="Function  has too many arguments. Should be less than 10." variable="">
      <Expression><![CDATA[<cfcomponent name="another">&#x000D;&#x000A;&#x000D;&#x000A;</cfcomponent>]]></Expression>
    </location>
  </issue>

apparantly due to some kind of interference between the test() function of the "b" component and the component named test_something.cfc.

renamig either the function or the test_something component removing the "interference" makes the test pass

removing the commented <cfargument in the b component makes the error go away

b.cfc.txt
test_something.cfc.txt

@rcastagno
Copy link
Contributor Author

This issue is blocking on SonarQube coldfusion plugin because it uses xml report to create issues on SonarQube. The existence of bogus errors causes the plugin to abruptly crash because it tries to reference a non-existent line in the file.

@rcastagno rcastagno added the bug label Aug 30, 2017
@ryaneberly ryaneberly self-assigned this Dec 16, 2017
@ryaneberly ryaneberly added this to the 1.3.0 milestone Dec 16, 2017
@ryaneberly
Copy link
Contributor

@rcastagno. Thanks for the examples. I can't duplicate the issue in the latest dev branch. Can this be closed?

@rcastagno
Copy link
Contributor Author

Ok for me, will try again on 1.3 when it's out, so far I've managed to proceed on sonarqube with a workaround on the plugin side.

@ryaneberly
Copy link
Contributor

Thanks @rcastagno. Closing for now. Reopen if it persists past 1.3.0

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants