[424131] JSDT Parser doesn't respect "Enable Semantic Validation" preference value

Problem reported is made to respect Enable Semantic Validation preference value for the preferences, so the problems market as
ProblemSeverities.Optional (configurable problems) aren't reported in case of Semantic Validation is disabled.
With this fix the build validation and as-you-type validation (errpr/warning annotations) are showing the same results,
not configurable error/warning messages do not depend on the value of "Enable JavaScript semantic validation" preference value.
All the JUnit Test cases of DualParseSyntaxErrorTest and SyntaxErrorTest passed successfully.

Signed-off-by: vrubezhny <vrubezhny@exadel.com>

1 files changed, 4 insertions, 1 deletions

diff --git a/bundles/org.eclipse.wst.jsdt.core/src/org/eclipse/wst/jsdt/internal/compiler/problem/ProblemReporter.java b/bundles/org.eclipse.wst.jsdt.core/src/org/eclipse/wst/jsdt/internal/compiler/problem/ProblemReporter.java
index c780b3404..baaf07fe1 100644
--- a/bundles/org.eclipse.wst.jsdt.core/src/org/eclipse/wst/jsdt/internal/compiler/problem/ProblemReporter.java
+++ b/bundles/org.eclipse.wst.jsdt.core/src/org/eclipse/wst/jsdt/internal/compiler/problem/ProblemReporter.java
@@ -616,7 +616,10 @@ public int computeSeverity(int problemID){
 	if (irritant != 0) {
 		if ((problemID & IProblem.Javadoc) != 0 && !this.options.docCommentSupport)
 			return ProblemSeverities.Ignore;
-		return this.options.getSeverity(irritant);
+		int severity = this.options.getSeverity(irritant);
+		return (!options.enableSemanticValidation && ((severity & ProblemSeverities.Optional) != 0) ?
+					ProblemSeverities.Ignore : 
+						severity);
 	}
 	return ProblemSeverities.Error | ProblemSeverities.Fatal;
 }