[jboss-jira] [JBoss JIRA] Updated: (JBRULES-1951) NullPointerException if no package name defined in package files

NullPointerException if no package name defined in package files ---------------------------------------------------------------- Key: JBRULES-1951 URL: https://jira.jboss.org/jira/browse/JBRULES-1951 Project: JBoss Drools Issue Type: Bug Security Level: Public(Everyone can see) Components: drools-compiler Affects Versions: 5.0.0.M5 Environment: All OS Environments Reporter: Shahad Ahmed Assignee: Edson Tirelli If you don't define a package name for a rule package then the drools eclipse plugin version 5.0M5 display the following NullPointerException error message against the associated rules files: Error: java.lang.NullPointerException For example, if your package is defined in a drl file, or Excel decision table and you haven't defined a package name in the file, or in an associated rules.package file then the error is reported - I've set the plugin to re-parse whenever a resource is changed in the plugin options. Similarly, if you're defining guided rules and haven't defined a package name in a rule.package file then the same NPE error is displayed. I've done a bit of debugging and found the NPE is thrown because the package name is still null internally and so the first attempt to use the package name (by JavaDialect.getUniqueLegalName) throws the NPE - a full stacktrace is given below. I've looked into a fix, and it appears as if the code to report a sensible error message has been commented out in org.drools.compiler.PackageBuilder. In the PackageBuilder.addPackage method the first line has a commented out call to method validatePackageName - this entire method has also been commented out. The problem can also be traced back to DroolsEclipsePlugin.getParsedResource, which calls PackageBuilder.addPackage in the first place. This constructs the PackageBuilder using a PackageBuilderConfiguration, which in turn calls this form of the constructor: PackageBuilder(RuleBase ruleBase, PackageBuilderConfiguration configuration). This constructor also has the following comments in it, which hints at some fixes that still need to be added: // FIXME, we need to get drools to support "default" namespace. //this.defaultNamespace = pkg.getName(); Note that addPackage does set the package name as the default namespace if the package has no name. However, as can be see from the commented out code above, because the default namespace is never set (i.e. is null) the package name is set to null in the PackageDescr and hence the NPE. StackTrace: java.lang.NullPointerException at org.drools.rule.builder.dialect.java.JavaDialect.getUniqueLegalName(J avaDialect.java:731) at org.drools.rule.builder.dialect.java.JavaDialect.init(JavaDialect.jav a:222) at org.drools.rule.builder.RuleBuildContext.<init>(RuleBuildContext.java :93) at org.drools.compiler.PackageBuilder.addRule(PackageBuilder.java:1153) at org.drools.compiler.PackageBuilder.addPackage(PackageBuilder.java:633 ) at org.drools.eclipse.DroolsEclipsePlugin.generateParsedResource(Unknown Source) at org.drools.eclipse.DroolsEclipsePlugin.parseBRLResource(Unknown Sourc e) at org.drools.eclipse.builder.DroolsBuilder.parseBRLFile(Unknown Source) at org.drools.eclipse.builder.DroolsBuilder.parseResource(Unknown Source ) at org.drools.eclipse.builder.DroolsBuilder$DroolsBuildVisitor.visit(Unk nown Source) at org.eclipse.core.internal.resources.Resource$2.visit(Resource.java:10 6) at org.eclipse.core.internal.resources.Resource$1.visitElement(Resource. java:58) at org.eclipse.core.internal.watson.ElementTreeIterator.doIteration(Elem entTreeIterator.java:81) at org.eclipse.core.internal.watson.ElementTreeIterator.doIteration(Elem entTreeIterator.java:85) at org.eclipse.core.internal.watson.ElementTreeIterator.doIteration(Elem entTreeIterator.java:85) at org.eclipse.core.internal.watson.ElementTreeIterator.doIteration(Elem entTreeIterator.java:85) at org.eclipse.core.internal.watson.ElementTreeIterator.doIteration(Elem entTreeIterator.java:85) at org.eclipse.core.internal.watson.ElementTreeIterator.iterate(ElementT reeIterator.java:126) at org.eclipse.core.internal.resources.Resource.accept(Resource.java:68) at org.eclipse.core.internal.resources.Resource.accept(Resource.java:104 ) at org.eclipse.core.internal.resources.Resource.accept(Resource.java:88) at org.drools.eclipse.builder.DroolsBuilder.fullBuild(Unknown Source) at org.drools.eclipse.builder.DroolsBuilder.incrementalBuild(Unknown Sou rce) at org.drools.eclipse.builder.DroolsBuilder.build(Unknown Source) at org.eclipse.core.internal.events.BuildManager$2.run(BuildManager.java :633) at org.eclipse.core.runtime.SafeRunner.run(SafeRunner.java:37) at org.eclipse.core.internal.events.BuildManager.basicBuild(BuildManager .java:170) at org.eclipse.core.internal.events.BuildManager.basicBuild(BuildManager .java:201) at org.eclipse.core.internal.events.BuildManager$1.run(BuildManager.java :253) at org.eclipse.core.runtime.SafeRunner.run(SafeRunner.java:37) at org.eclipse.core.internal.events.BuildManager.basicBuild(BuildManager .java:256) at org.eclipse.core.internal.events.BuildManager.basicBuildLoop(BuildMan ager.java:309) at org.eclipse.core.internal.events.BuildManager.build(BuildManager.java :341) at org.eclipse.core.internal.events.AutoBuildJob.doBuild(AutoBuildJob.ja va:140) at org.eclipse.core.internal.events.AutoBuildJob.run(AutoBuildJob.java:2 38) at org.eclipse.core.internal.jobs.Worker.run(Worker.java:55)