You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@myfaces.apache.org by ar...@apache.org on 2009/06/01 19:37:33 UTC

svn commit: r780743 - /myfaces/trinidad/branches/1.2.11.3-branch/trinidad-api/src/main/java/org/apache/myfaces/trinidad/component/UIXCollection.java

Author: arobinson74
Date: Mon Jun  1 17:37:33 2009
New Revision: 780743

URL: http://svn.apache.org/viewvc?rev=780743&view=rev
Log:
TRINIDAD-1493

Appling patch from Kamran Kashanian

Modified:
    myfaces/trinidad/branches/1.2.11.3-branch/trinidad-api/src/main/java/org/apache/myfaces/trinidad/component/UIXCollection.java

Modified: myfaces/trinidad/branches/1.2.11.3-branch/trinidad-api/src/main/java/org/apache/myfaces/trinidad/component/UIXCollection.java
URL: http://svn.apache.org/viewvc/myfaces/trinidad/branches/1.2.11.3-branch/trinidad-api/src/main/java/org/apache/myfaces/trinidad/component/UIXCollection.java?rev=780743&r1=780742&r2=780743&view=diff
==============================================================================
--- myfaces/trinidad/branches/1.2.11.3-branch/trinidad-api/src/main/java/org/apache/myfaces/trinidad/component/UIXCollection.java (original)
+++ myfaces/trinidad/branches/1.2.11.3-branch/trinidad-api/src/main/java/org/apache/myfaces/trinidad/component/UIXCollection.java Mon Jun  1 17:37:33 2009
@@ -1052,18 +1052,26 @@
         else
         {
           String currencyString = postId.substring(0, sepIndex);
-          Object oldRowKey = getRowKey();
-          try
+          Object rowKey = getClientRowKeyManager().getRowKey(context, this, currencyString);
+          
+          // A non-null rowKey here means we are on a row and we should set currency,  otherwise 
+          // the client id is for a non-stamped child component in the table/column header/footer.
+          if (rowKey != null)
           {
-            setCurrencyString(currencyString);
-            
-            return invokeOnChildrenComponents(context, clientId, callback);
-          }
-          finally
-          {
-            // And restore the currency
-            setRowKey(oldRowKey);
+            Object oldRowKey = getRowKey();
+            try
+            {
+              setRowKey(rowKey);              
+              return invokeOnChildrenComponents(context, clientId, callback);
+            }
+            finally
+            {
+              // And restore the currency
+              setRowKey(oldRowKey);
+            }
           }
+          else
+            return invokeOnChildrenComponents(context, clientId, callback);            
         }
       }
       finally