[jboss-svn-commits] JBL Code SVN: r19989 - in labs/jbossesb/branches/JBESB_4_2_1_GA_CP/product/rosetta: tests/src/org/jboss/soa/esb/notification and 1 other directory.

jboss-svn-commits at lists.jboss.org jboss-svn-commits at lists.jboss.org
Sat May 17 15:12:47 EDT 2008


Author: tcunning
Date: 2008-05-17 15:12:47 -0400 (Sat, 17 May 2008)
New Revision: 19989

Modified:
   labs/jbossesb/branches/JBESB_4_2_1_GA_CP/product/rosetta/src/org/jboss/soa/esb/notification/NotifySqlTable.java
   labs/jbossesb/branches/JBESB_4_2_1_GA_CP/product/rosetta/tests/src/org/jboss/soa/esb/notification/NotifySqlTableUnitTest.java
Log:
bug:JBESB-1623
Merge changes over to GA_CP branch, requested by forum post here :
http://www.jboss.com/index.html?module=bb&op=viewtopic&t=135591


Modified: labs/jbossesb/branches/JBESB_4_2_1_GA_CP/product/rosetta/src/org/jboss/soa/esb/notification/NotifySqlTable.java
===================================================================
--- labs/jbossesb/branches/JBESB_4_2_1_GA_CP/product/rosetta/src/org/jboss/soa/esb/notification/NotifySqlTable.java	2008-05-17 17:24:02 UTC (rev 19988)
+++ labs/jbossesb/branches/JBESB_4_2_1_GA_CP/product/rosetta/src/org/jboss/soa/esb/notification/NotifySqlTable.java	2008-05-17 19:12:47 UTC (rev 19989)
@@ -124,7 +124,7 @@
 		{
 			String sIn = (i1 == 0) ? "(" : ",";
 			String sCurrCol = saCols[i1];
-			sbCol.append(sIn).append("\"").append(sCurrCol).append("\"");
+			sbCol.append(sIn).append(sCurrCol);
 			sbPrm.append(sIn).append("'").append(m_oCols.getProperty(sCurrCol))
 					.append("'");
 			// REVIEW: Does the value being inserted not need to be escaped e.g.

Modified: labs/jbossesb/branches/JBESB_4_2_1_GA_CP/product/rosetta/tests/src/org/jboss/soa/esb/notification/NotifySqlTableUnitTest.java
===================================================================
--- labs/jbossesb/branches/JBESB_4_2_1_GA_CP/product/rosetta/tests/src/org/jboss/soa/esb/notification/NotifySqlTableUnitTest.java	2008-05-17 17:24:02 UTC (rev 19988)
+++ labs/jbossesb/branches/JBESB_4_2_1_GA_CP/product/rosetta/tests/src/org/jboss/soa/esb/notification/NotifySqlTableUnitTest.java	2008-05-17 19:12:47 UTC (rev 19989)
@@ -59,7 +59,7 @@
 		assertEquals("colvalue2", nst.m_oCols.getProperty("colname2"));
 		assertEquals("colvalue3", nst.m_oCols.getProperty("colname3"));
 
-		assertEquals("insert into table (\"colname3\",\"colname2\",\"colname1\") values ('colvalue3','colvalue2','colvalue1')", nst.getInsertStmt());
+		assertEquals("insert into table (colname3,colname2,colname1) values ('colvalue3','colvalue2','colvalue1')", nst.getInsertStmt());
 	}
 	
 	public void test_sendNotificationNegative() throws SAXException, IOException, ConfigurationException




More information about the jboss-svn-commits mailing list