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 2006/05/23 15:51:44 UTC

svn commit: r408944 - /incubator/adffaces/trunk/adf-faces/adf-faces-impl/src/main/java/org/apache/myfaces/adfinternal/ui/laf/base/desktop/TreeRenderer.java

Author: awiner
Date: Tue May 23 08:51:44 2006
New Revision: 408944

URL: http://svn.apache.org/viewvc?rev=408944&view=rev
Log:
ADFFaces-12: af:tree Need better error message for missing form tag

Modified:
    incubator/adffaces/trunk/adf-faces/adf-faces-impl/src/main/java/org/apache/myfaces/adfinternal/ui/laf/base/desktop/TreeRenderer.java

Modified: incubator/adffaces/trunk/adf-faces/adf-faces-impl/src/main/java/org/apache/myfaces/adfinternal/ui/laf/base/desktop/TreeRenderer.java
URL: http://svn.apache.org/viewvc/incubator/adffaces/trunk/adf-faces/adf-faces-impl/src/main/java/org/apache/myfaces/adfinternal/ui/laf/base/desktop/TreeRenderer.java?rev=408944&r1=408943&r2=408944&view=diff
==============================================================================
--- incubator/adffaces/trunk/adf-faces/adf-faces-impl/src/main/java/org/apache/myfaces/adfinternal/ui/laf/base/desktop/TreeRenderer.java (original)
+++ incubator/adffaces/trunk/adf-faces/adf-faces-impl/src/main/java/org/apache/myfaces/adfinternal/ui/laf/base/desktop/TreeRenderer.java Tue May 23 08:51:44 2006
@@ -27,6 +27,7 @@
 
 import org.apache.myfaces.adf.component.UIXHierarchy;
 import org.apache.myfaces.adf.component.UIXTree;
+import org.apache.myfaces.adf.logging.ADFLogger;
 import org.apache.myfaces.adf.model.RowKeySet;
 
 import org.apache.myfaces.adfinternal.agent.AdfFacesAgent;
@@ -98,8 +99,11 @@
 //    List focusPath = getFocusPath(context, node);
     Object focusPath = tree.getFocusRowKey();
     String formName = getParentFormName(context);
-
-
+    if (formName == null)
+    {
+      _LOG.warning("The tree component must be used inside of a form.");
+      return;
+    }
 
 
     // Bug 3931544:  don't use colons in Javascript variable names.
@@ -983,4 +987,6 @@
   
   private static final String _PATH_PARAM = 
     "path";
+
+  private static final ADFLogger _LOG = ADFLogger.createADFLogger(TreeRenderer.class);
 }