You are viewing a plain text version of this content. The canonical link for it is here.
Posted to adffaces-commits@incubator.apache.org by aw...@apache.org on 2007/03/09 21:54:07 UTC

svn commit: r516575 - /incubator/adffaces/trunk/trinidad/trinidad-api/src/main/java/org/apache/myfaces/trinidad/model/XMLMenuModel.java

Author: awiner
Date: Fri Mar  9 13:54:06 2007
New Revision: 516575

URL: http://svn.apache.org/viewvc?view=rev&rev=516575
Log:
ADFFACES-393: one hopes, the final patch on this issue from Gary Kind!

Modified:
    incubator/adffaces/trunk/trinidad/trinidad-api/src/main/java/org/apache/myfaces/trinidad/model/XMLMenuModel.java

Modified: incubator/adffaces/trunk/trinidad/trinidad-api/src/main/java/org/apache/myfaces/trinidad/model/XMLMenuModel.java
URL: http://svn.apache.org/viewvc/incubator/adffaces/trunk/trinidad/trinidad-api/src/main/java/org/apache/myfaces/trinidad/model/XMLMenuModel.java?view=diff&rev=516575&r1=516574&r2=516575
==============================================================================
--- incubator/adffaces/trunk/trinidad/trinidad-api/src/main/java/org/apache/myfaces/trinidad/model/XMLMenuModel.java (original)
+++ incubator/adffaces/trunk/trinidad/trinidad-api/src/main/java/org/apache/myfaces/trinidad/model/XMLMenuModel.java Fri Mar  9 13:54:06 2007
@@ -431,6 +431,15 @@
   {
     _setCurrentlySelectedNode(currentNode);
     _setRequestMethod(_METHOD_POST);
+
+    
+    // Do this in the case where a menu item is selected
+    // that has the same viewId as the previous menu item
+    // that is selected.  If not, the test at the beginning
+    // of getFocusRowKey() (currentViewId == _prevViewId)
+    // is true and just returns, even though we have selected
+    // a new node and the focus path should change.
+    _prevViewId = null;
   }
 
   /**