Author: sumedha
Date: Wed Jan  2 01:05:02 2008
New Revision: 11738

Log:

backward compatibility for stored procedure ordinals

Modified:
   
trunk/commons/data-services/src/main/java/org/wso2/ws/dataservice/beans/Param.java
   
trunk/commons/data-services/src/main/java/org/wso2/ws/dataservice/beans/Query.java

Modified: 
trunk/commons/data-services/src/main/java/org/wso2/ws/dataservice/beans/Param.java
==============================================================================
--- 
trunk/commons/data-services/src/main/java/org/wso2/ws/dataservice/beans/Param.java
  (original)
+++ 
trunk/commons/data-services/src/main/java/org/wso2/ws/dataservice/beans/Param.java
  Wed Jan  2 01:05:02 2008
@@ -50,7 +50,7 @@
                this.name = name;
                this.sqlType = sqlType;
                this.type = type;
-               this.ordinal = Integer.valueOf(ordinal).intValue();;
+               this.ordinal = Integer.valueOf(ordinal).intValue();
        }
        public Param(String name,String sqlType,String type){
                this.name = name;

Modified: 
trunk/commons/data-services/src/main/java/org/wso2/ws/dataservice/beans/Query.java
==============================================================================
--- 
trunk/commons/data-services/src/main/java/org/wso2/ws/dataservice/beans/Query.java
  (original)
+++ 
trunk/commons/data-services/src/main/java/org/wso2/ws/dataservice/beans/Query.java
  Wed Jan  2 01:05:02 2008
@@ -34,13 +34,23 @@
        
        public Query(Iterator paramItr){
                Param param;
-               ArrayList paramList = new ArrayList();
+               ArrayList paramList = new ArrayList();          
+               int ordinal = 0; 
                while (paramItr.hasNext()) {
+                       ordinal++;
                        OMElement paramElement = (OMElement) paramItr.next();
+
+                       //start: work-a-round to maintain backward compatibility
+                       String userSetOrdinalValue = 
paramElement.getAttributeValue(new QName("ordinal"));
+                       if(userSetOrdinalValue == null || 
userSetOrdinalValue.trim().length() == 0){
+                               userSetOrdinalValue = String.valueOf(ordinal);
+                       }
+                       //end: work-a-round to maintain backward compatibility
+                       
                        param = new Param(paramElement.getAttributeValue(new 
QName("name")),
                                                
paramElement.getAttributeValue(new QName("sqlType")),
                                                
paramElement.getAttributeValue(new QName("type")),
-                                               
paramElement.getAttributeValue(new QName("ordinal")));
+                                               userSetOrdinalValue);
                        paramList.add(param);
                }
                try{

_______________________________________________
Commons-dev mailing list
[email protected]
http://wso2.org/cgi-bin/mailman/listinfo/commons-dev

Reply via email to