[jboss-cvs] JBossAS SVN: r109982 - trunk/connector/src/main/java/org/jboss/resource/adapter/jdbc/local.

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

Author: weston.price at jboss.com
Date: 2010-12-17 04:27:33 -0500 (Fri, 17 Dec 2010)
New Revision: 109982

Modified:
   trunk/connector/src/main/java/org/jboss/resource/adapter/jdbc/local/LocalManagedConnectionFactory.java
Log:
[JBAS-8559] Fixed issue with non-existent property and data sources. Currently we just ignore and WARN if the property does not exist. This will require a follow up CTS JIRA issue to get clarification for the specification as well as these particular TCK tests.

Modified: trunk/connector/src/main/java/org/jboss/resource/adapter/jdbc/local/LocalManagedConnectionFactory.java
===================================================================
--- trunk/connector/src/main/java/org/jboss/resource/adapter/jdbc/local/LocalManagedConnectionFactory.java	2010-12-17 09:20:02 UTC (rev 109981)
+++ trunk/connector/src/main/java/org/jboss/resource/adapter/jdbc/local/LocalManagedConnectionFactory.java	2010-12-17 09:27:33 UTC (rev 109982)
@@ -608,7 +608,8 @@
 	                  }
 	                  catch(NoSuchMethodException nsme)
 	                  {
-	                     type = String.class;
+	                	  log.warn("Property with name " + name + " could not be found for datasource class" + getDriverClass() + ". This property will be ignored.");
+	                	  continue;
 	                  }
 	               }