[jboss-svn-commits] JBL Code SVN: r33275 - labs/jbossrules/trunk/drools-eclipse/org.drools.eclipse/src/main/java/org/drools/eclipse/refactoring.

[jboss-svn-commits at lists.jboss.org]

Author: KrisVerlaenen
Date: 2010-05-31 21:10:18 -0400 (Mon, 31 May 2010)
New Revision: 33275

Modified:
   labs/jbossrules/trunk/drools-eclipse/org.drools.eclipse/src/main/java/org/drools/eclipse/refactoring/DRLProjectDetector.java
Log:
JBRULES-2450: Nullpointer Exception when refacroring in unreleted projectsa
 - made sure no NPE can occur in this part of the code anymore


Modified: labs/jbossrules/trunk/drools-eclipse/org.drools.eclipse/src/main/java/org/drools/eclipse/refactoring/DRLProjectDetector.java
===================================================================
--- labs/jbossrules/trunk/drools-eclipse/org.drools.eclipse/src/main/java/org/drools/eclipse/refactoring/DRLProjectDetector.java	2010-06-01 00:57:12 UTC (rev 33274)
+++ labs/jbossrules/trunk/drools-eclipse/org.drools.eclipse/src/main/java/org/drools/eclipse/refactoring/DRLProjectDetector.java	2010-06-01 01:10:18 UTC (rev 33275)
@@ -28,15 +28,17 @@
 	}
 
 	private void detect(IResource[] members) throws CoreException {
+		if (members == null) {
+			return;
+		}
 		for (int i = 0; i < members.length; i++) {
 			if (members[i] instanceof IFolder) {
-				IFolder folder = (IFolder)members[i];
+				IFolder folder = (IFolder) members[i];
 				if (!folder.isDerived())
-					detect(((IFolder)members[i]).members());
-			}
-			if (members[i] instanceof IFile) {
-				IFile file = (IFile)members[i];
-				if (file.getFileExtension().equalsIgnoreCase("drl"))
+					detect(folder.members());
+			} else if (members[i] instanceof IFile) {
+				IFile file = (IFile) members[i];
+				if (file.getFileExtension() != null && file.getFileExtension().equalsIgnoreCase("drl"))
 					if (file.isAccessible() && !file.isReadOnly() && !file.isDerived())
 						resources.add(file);
 			}