[jboss-svn-commits] JBL Code SVN: r5366 - labs/jbossesb/trunk/product/core/common/src/org/jboss/soa/esb/notification

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

Author: tfennelly
Date: 2006-07-31 13:07:05 -0400 (Mon, 31 Jul 2006)
New Revision: 5366

Modified:
   labs/jbossesb/trunk/product/core/common/src/org/jboss/soa/esb/notification/NotifySqlTable.java
Log:
just added a comment

Modified: labs/jbossesb/trunk/product/core/common/src/org/jboss/soa/esb/notification/NotifySqlTable.java
===================================================================
--- labs/jbossesb/trunk/product/core/common/src/org/jboss/soa/esb/notification/NotifySqlTable.java	2006-07-31 17:04:21 UTC (rev 5365)
+++ labs/jbossesb/trunk/product/core/common/src/org/jboss/soa/esb/notification/NotifySqlTable.java	2006-07-31 17:07:05 UTC (rev 5366)
@@ -98,6 +98,7 @@
 			String sCurrCol = saCols[i1];
 			sbCol.append(sIn).append("\"").append(sCurrCol).append("\"");
 			sbPrm.append(sIn).append("'").append(m_oCols.getProperty(sCurrCol)).append("'");
+			// REVIEW: Does the value being inserted not need to be escaped e.g. what if it has a quote in its value??
 		}
 		return sbCol.append(")").append(sbPrm).append(")").toString();
 	} // __________________________________