[esb-issues] [JBoss JIRA] Commented: (JBESB-1855) Upper / lower case problems in SqlTableGatewayListener

[Kevin Conner (JIRA)]

    [ http://jira.jboss.com/jira/browse/JBESB-1855?page=comments#action_12419322 ] 
            
Kevin Conner commented on JBESB-1855:
-------------------------------------

This has already been fixed in CP3 and will be pulled across into trunk once the CP release has gone out.

We can leave this open for now, to remind us to check once it has been merged, but no work should be necessary,

> Upper / lower case problems in SqlTableGatewayListener
> ------------------------------------------------------
>
>                 Key: JBESB-1855
>                 URL: http://jira.jboss.com/jira/browse/JBESB-1855
>             Project: JBoss ESB
>          Issue Type: Bug
>      Security Level: Public(Everyone can see) 
>          Components: Transports
>    Affects Versions: 4.3
>            Reporter: Tom Cunningham
>         Assigned To: Tom Cunningham
>             Fix For: 4.4
>
>
> There are a few upper/lower case problems in trunk in SqlTableGatewayListener, where we are either looking for the upper-cased column name but try to find the lower-cased version, or we are looking for the lower cased version and send in the upper-cased version.       The Quickstart only uses upper-cased column names (something we should probably change), so these problems likely did not show up during integration testing.
> A workaround for now for users would be to use all upper-case letters in their column names in jboss-esb.xml.       I don't see this problem in CP branch.
> Index: src/org/jboss/soa/esb/listeners/gateway/SqlTableGatewayListener.java
> ===================================================================
> --- src/org/jboss/soa/esb/listeners/gateway/SqlTableGatewayListener.java	(revision 20835)
> +++ src/org/jboss/soa/esb/listeners/gateway/SqlTableGatewayListener.java	(working copy)
> @@ -663,7 +663,7 @@
>              	if (_currentRow.containsKey(sColName)) {
>                      val = String.valueOf(_currentRow.get(sColName));            		
>              	} else {
> -                    val = String.valueOf(_upperCurrentRow.get(sColName));
> +                    val = String.valueOf(_upperCurrentRow.get(sColName.toUpperCase()));
>              	}
>                  _PSdeleteRow.setString(iParm++, val);
>              }
> @@ -740,7 +740,7 @@
>              for (String sColName : _keys) {
>              	Object oVal = null;
>              	if (_currentRow.containsKey(sColName)) {
> -            		oVal = String.valueOf(_currentRow.get(sColName.toUpperCase()));
> +            		oVal = String.valueOf(_currentRow.get(sColName));
>              	} else {
>              		oVal = String.valueOf(_upperCurrentRow.get(sColName.toUpperCase()));
>              	}

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: http://jira.jboss.com/jira/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira