You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@click.apache.org by sa...@apache.org on 2010/06/03 00:50:24 UTC

svn commit: r950806 - /click/trunk/click/extras/src/org/apache/click/extras/control/Menu.java

Author: sabob
Date: Wed Jun  2 22:50:24 2010
New Revision: 950806

URL: http://svn.apache.org/viewvc?rev=950806&view=rev
Log:
remove duplicate id lookup

Modified:
    click/trunk/click/extras/src/org/apache/click/extras/control/Menu.java

Modified: click/trunk/click/extras/src/org/apache/click/extras/control/Menu.java
URL: http://svn.apache.org/viewvc/click/trunk/click/extras/src/org/apache/click/extras/control/Menu.java?rev=950806&r1=950805&r2=950806&view=diff
==============================================================================
--- click/trunk/click/extras/src/org/apache/click/extras/control/Menu.java (original)
+++ click/trunk/click/extras/src/org/apache/click/extras/control/Menu.java Wed Jun  2 22:50:24 2010
@@ -1062,9 +1062,8 @@ public class Menu extends AbstractContro
 
         // Note, the addLoadEvent script is recreated and checked if it
         // is contained in the headElement.
-        String menuId = getId();
         JsScript script = new JsScript();
-        script.setId(menuId + "_js_setup");
+        script.setId(id + "_js_setup");
         if (!headElements.contains(script)) {
             // Script must be executed as soon as browser dom is ready
             script.setExecuteOnDomReady(true);