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

Unclear error messages: Error in plugin, 0 #269

Closed
mjclemente opened this issue May 19, 2017 · 12 comments
Closed

Unclear error messages: Error in plugin, 0 #269

mjclemente opened this issue May 19, 2017 · 12 comments
Assignees
Labels
Milestone

Comments

@mjclemente
Copy link

I'm seeing some strange behavior when using 1.1.0, from the master branch (it appears it may be related to use of underscores). I'll do my best to describe it, and provide a screenshot of the resulting output.

I'm including the errors thrown for two files; the same basic two errors are being thrown for each:

  • Error in plugin
  • 0

For the Error in plugin, the expression referenced is variables._.reverse(categoryList). No expression, line, or function is referenced for the 0 error.

Possibly related to these, I'm also get the following errors:

  • Message code: METHOD_ALLCAPS_NAME is not configured correctly."
  • Message code: METHOD_TOO_SHORT is not configured correctly.

Is this an issue with my configuration or related to cflint? Thanks for any help or insight you can provide.

(In the first instance, it's a file in which I am using underscoreCFC, and in the second, it's a FW/1 method.

image

@ryaneberly
Copy link
Contributor

cfparser does not know how to parse the underscore. I will look into that.

@mjclemente
Copy link
Author

Thanks! Sorry for the bother; as I've said before, love the project. Let me know if there's anything that I can do to help or clarify further.

@ryaneberly
Copy link
Contributor

ryaneberly commented Jun 2, 2017 via email

ryaneberly added a commit to cfparser/cfparser that referenced this issue Jun 16, 2017
@ryaneberly
Copy link
Contributor

@mjclemente could you test against dev. I cannot duplicate your issue. If you still see it, could you provide a code sample.

@mjclemente
Copy link
Author

Sure. Happy to.

When I went to install the dev version, I got a build error:

[INFO]
[INFO] --- maven-war-plugin:2.2:war (default-war) @ CFLint ---
[INFO] Packaging webapp
[INFO] Assembling webapp [CFLint] in [/Users/matthew/www/tools/CFLint-dev/target/CFLint-1.1.0]
[INFO] Processing war project
[INFO] Webapp assembled in [129 msecs]
[INFO] Building war: /Users/matthew/www/tools/CFLint-dev/target/CFLint-1.1.0.war
[INFO] ------------------------------------------------------------------------
[INFO] BUILD FAILURE
[INFO] ------------------------------------------------------------------------
[INFO] Total time: 29.029 s
[INFO] Finished at: 2017-06-22T06:36:32-04:00
[INFO] Final Memory: 32M/588M
[INFO] ------------------------------------------------------------------------
[ERROR] Failed to execute goal org.apache.maven.plugins:maven-war-plugin:2.2:war (default-war) on project CFLint: Error assembling WAR: webxml attribute is required (or pre-existing WEB-INF/web.xml if executing in update mode) -> [Help 1]
[ERROR]
[ERROR] To see the full stack trace of the errors, re-run Maven with the -e switch.
[ERROR] Re-run Maven using the -X switch to enable full debug logging.
[ERROR]
[ERROR] For more information about the errors and possible solutions, please read the following articles:
[ERROR] [Help 1] http://cwiki.apache.org/confluence/display/MAVEN/MojoExecutionException

I feel like I should probably be able to figure out and correct this, but I'm a complete novice when it comes to Maven. Is there a change I can make to run the build successfully?

@mjclemente
Copy link
Author

Sorry, just saw the updated instructions on installing (https://github.com/cflint/CFLint/wiki/How-Do-I-Install-This-Tool%3F). I'll try that route to see if it resolves the issue.

@mjclemente
Copy link
Author

Okay, ran the build via Gradle. It says "BUILD SUCCESSFUL" at the end. However, I can't find the executablecflint. (There's no target or appassembler, folders, as there used to be following a Maven install. Not sure how to proceed.

@denuno
Copy link
Collaborator

denuno commented Jun 22, 2017 via email

@mjclemente
Copy link
Author

Thanks. That fixed the build error.

I built CFLint from the latest dev, but the parsing issue is still occurring for me.

I put together some code to reproduce it: https://github.com/mjclemente/cflint-underscore-test

Please let me know if there's more that I can do, or if there's any more information that I can provide.

@ryaneberly ryaneberly added this to the 1.2.0 milestone Jun 27, 2017
@ryaneberly ryaneberly self-assigned this Jun 28, 2017
ryaneberly added a commit that referenced this issue Jun 28, 2017
@ryaneberly
Copy link
Contributor

ValidNames checker has an issue with the underscore. It should be fixed.

@mjclemente
Copy link
Author

Awesome! Just downloaded from the dev branch and ran it. Worked perfectly!

Thanks guys; this project is amazing.

@ryaneberly
Copy link
Contributor

@mjclemente . Thanks for testing it.

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

4 participants