superruzafa / nb-coding-standard-validator Goto Github PK

Be able to sort the coding standard violations in the window by line or message 
by clicking on the column header.

Allow to validate recursively the coding standard of all files in a folder. 
Show a progress dialog when it is a long time operation.

Use a ProgressMonitor or something like that to notify the user a validation is 
in progress.

This could be applied both to single files to folders (issue 4).

Be able to parse XML reports of Code Sniffer.

What steps will reproduce the problem?
1. selected a custom (autodetected) coding standard
2. loaded a file.
3. refreshed the standards window

What is the expected output? What do you see instead?
just the results

What version of the product are you using? On what operating system?
Netbeans 7Beta2 on Win7

Please provide any additional information below.

java.lang.IllegalStateException: Problem in some module which uses Window 
System: Window System API is required to be called from AWT thread only, see 
http://core.netbeans.org/proposals/threading/
    at org.netbeans.core.windows.WindowManagerImpl.warnIfNotInEDT(WindowManagerImpl.java:1520)
    at org.netbeans.core.windows.WindowManagerImpl.getMainWindow(WindowManagerImpl.java:160)
    at org.superruzafa.codingstandardvalidator.ui.CodingStandardValidatorAction$1.doInBackground(CodingStandardValidatorAction.java:61)
    at org.superruzafa.codingstandardvalidator.ui.CodingStandardValidatorAction$1.doInBackground(CodingStandardValidatorAction.java:48)
    at javax.swing.SwingWorker$1.call(Unknown Source)
    at java.util.concurrent.FutureTask$Sync.innerRun(Unknown Source)
    at java.util.concurrent.FutureTask.run(Unknown Source)
    at javax.swing.SwingWorker.run(Unknown Source)
    at java.util.concurrent.ThreadPoolExecutor$Worker.runTask(Unknown Source)
    at java.util.concurrent.ThreadPoolExecutor$Worker.run(Unknown Source)
    at java.lang.Thread.run(Unknown Source)

Move the revalidate button at first position and change the icon. Use double 
"play" symbol button.

Be able to automatically validate the code and show highlightning marks on 
coding standard violations.

When there are violations and the user hides all messages (by clicking the side 
buttons) a empty list of violations is shown.

An error message should be displayed notifying the code style doesn't complains 
the coding standard (if there are strict violations) instead of an empty list 
of violations.

Also, when there are no strict violations but warnings, a warning message 
should be displayed notifying this.

Show current coding standard on violations window's title.

Currently if you provided a non-installed coding standard the plugin validates 
the file saying there aren't any error.

java.lang.IllegalStateException: Problem in some module which uses Window 
System: Window System API is required to be called from AWT thread only, see 
http://core.netbeans.org/proposals/threading/
    at org.netbeans.core.windows.WindowManagerImpl.warnIfNotInEDT(WindowManagerImpl.java:1531)
    at org.netbeans.core.windows.WindowManagerImpl.getMainWindow(WindowManagerImpl.java:171)
    at org.superruzafa.codingstandardvalidator.ui.CodingStandardValidatorAction$1.doInBackground(CodingStandardValidatorAction.java:61)
    at org.superruzafa.codingstandardvalidator.ui.CodingStandardValidatorAction$1.doInBackground(CodingStandardValidatorAction.java:48)
    at javax.swing.SwingWorker$1.call(SwingWorker.java:277)
    at java.util.concurrent.FutureTask$Sync.innerRun(FutureTask.java:303)
    at java.util.concurrent.FutureTask.run(FutureTask.java:138)
    at javax.swing.SwingWorker.run(SwingWorker.java:316)
    at java.util.concurrent.ThreadPoolExecutor$Worker.runTask(ThreadPoolExecutor.java:886)
    at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:908)
    at java.lang.Thread.run(Thread.java:680)

When the method CodeSnifferReportParser::parse() returns false, the validator 
reports that the code complains the coding standard.

When false returned it means that the parsing couldn't be done.

Show in a small bottom text a summary with total errors/warning occurred.

Currently just the line where the violation was occurred is used. Add also a 
reference to the column (where applicable, like the XML parser)

Users usually modify the code files and then launch the validation process 
without save their work. So CodeSniffer validates the old non-saved version of 
the file.

Validator should save all files before validate them.

I have new functionality suggestion

Will be nice add some after save hook. Will be nice validate file immediately 
after file save and show for example red exclamation mark on the left side of 
editor (like PHP errors).

A menu should be shown to (at least) copy the coding standard violations text.

When somehow the Code Sniffer script is properly set but not the coding 
standard then the coding standard should be "Default" or something like that.

Add a button at the side of the violations window to relaunch the process on 
the last validated file or folder.

A message box should be shown when the user doesn't specify the path to the 
code sniffer script or when that path doesn't exists.