violations-plugin icon indicating copy to clipboard operation
violations-plugin copied to clipboard

No violations found when opening file on Windows

Open domruf opened this issue 10 years ago • 1 comments

When running on windows and you try to open the page .../job/myjob/2/violations/file/path/to/myfile.py/ you get "No violations found"

The following patch fixes this. But I don't really know the code and there my be some other places where this problem pops up. So there might be a better place/way to fix this properly.

diff -r 17567a8c09f2 src/main/java/hudson/plugins/violations/ViolationsReport.java --- a/src/main/java/hudson/plugins/violations/ViolationsReport.java Wed Apr 16 20:21:26 2014 +0100 +++ b/src/main/java/hudson/plugins/violations/ViolationsReport.java Wed May 07 13:59:24 2014 +0200 @@ -211,7 +211,7 @@ if (name.startsWith("/")) { name = name.substring(1); }

  •    FileModelProxy proxy = getFileModelProxy(name);
    
  •    FileModelProxy proxy = getFileModelProxy(name.replace("/", File.separator));
     if (proxy != null) {
         return new RecurDynamic(
             "", name, proxy.build(build).contextPath(req.getContextPath()));
    

domruf avatar May 07 '14 12:05 domruf

I came across this issue as well while trying to parse results from subfolders in Windows. I think this is the best place to make the change, as the keys are all generated matching what is contained within the xml files and directory locations. I created pull request 81 for this change.

pfhorman avatar Dec 04 '15 20:12 pfhorman