Fix for #5638 - ComponentLocator does not use DOM hierarchy when a SubPartAware fails...

svn changeset:15073/svn branch:6.4

diff --git a/src/com/vaadin/terminal/gwt/client/ComponentLocator.java b/src/com/vaadin/terminal/gwt/client/ComponentLocator.java
index 0c11064707a55162c6b347e8486d2d29a9334ee8..6644a243d05a32b1566147a40ecb09a95f442a35 100644 (file)
--- a/src/com/vaadin/terminal/gwt/client/ComponentLocator.java
+++ b/src/com/vaadin/terminal/gwt/client/ComponentLocator.java
@@ -136,14 +136,15 @@ public class ComponentLocator {
              */
             String elementLocator = ((SubPartAware) w)
                     .getSubPartName(targetElement);
-            return path + SUBPART_SEPARATOR + elementLocator;
-        } else {
-            /*
-             * If everything else fails we use the DOM path to identify the
-             * target element
-             */
-            return path + getDOMPathForElement(targetElement, w.getElement());
+            if (elementLocator != null) {
+                return path + SUBPART_SEPARATOR + elementLocator;
+            }
         }
+        /*
+         * If everything else fails we use the DOM path to identify the target
+         * element
+         */
+        return path + getDOMPathForElement(targetElement, w.getElement());
     }
 
     /**