[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