Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 323160 - JS validator should ignore derived resources by default
Summary: JS validator should ignore derived resources by default
Status: RESOLVED FIXED
Alias: None
Product: JSDT
Classification: WebTools
Component: General (show other bugs)
Version: unspecified   Edit
Hardware: All All
: P3 normal (vote)
Target Milestone: 3.2.3   Edit
Assignee: Nitin Dahyabhai CLA
QA Contact: Nitin Dahyabhai CLA
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-08-19 11:13 EDT by Jim Zhang CLA
Modified: 2010-09-28 17:06 EDT (History)
1 user (show)

See Also:
cmjaun: review+


Attachments
proposed patch (2.79 KB, patch)
2010-09-06 21:35 EDT, Nitin Dahyabhai CLA
no flags Details | Diff
updated patch (2.89 KB, patch)
2010-09-27 18:03 EDT, Nitin Dahyabhai CLA
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Jim Zhang CLA 2010-08-19 11:13:56 EDT
Build Identifier: WTP 3.2

there are cases where I have javascript code alongside my java code.  as such, these js files will be existing inside the java output folder, for instance web-inf/classes, after build.  js validation should ignore these files by default since they are derived resources.  Currently validator does go through these files and flag errors if there are validation problems.

Reproducible: Always

Steps to Reproduce:
1. create a dynamic web project
2. add a js file with validation errors (such as missing "}") to a java source package
3. build the project
4. notice the error on the js file inside the web-inf/classes folder
Comment 1 Nitin Dahyabhai CLA 2010-09-06 21:35:15 EDT
Created attachment 178289 [details]
proposed patch
Comment 2 Nitin Dahyabhai CLA 2010-09-27 18:03:45 EDT
Created attachment 179695 [details]
updated patch
Comment 3 Nitin Dahyabhai CLA 2010-09-27 18:07:23 EDT
Committed to HEAD.
Comment 4 Chris Jaun CLA 2010-09-28 17:06:07 EDT
Fixed in 3.2.3 and HEAD.