Merge pull request #13998 from dstansby/patch-contains

timhoffm · web-flow · commit 1956b1d5e7d5 · 2019-04-22T17:06:46.000+02:00
Fix patch contains logic for patches that don't have any codes
diff --git a/lib/matplotlib/patches.py b/lib/matplotlib/patches.py
@@ -123,25 +123,33 @@ def _process_radius(self, radius):
         return _radius
 
     def contains(self, mouseevent, radius=None):
-        """Test whether the mouse event occurred in the patch.
+        """
+        Test whether the mouse event occurred in the patch.
 
-        Returns T/F, {}
+        Returns
+        -------
+        (bool, empty dict)
         """
         if self._contains is not None:
             return self._contains(self, mouseevent)
         radius = self._process_radius(radius)
         codes = self.get_path().codes
-        vertices = self.get_path().vertices
-        # if the current path is concatenated by multiple sub paths.
-        # get the indexes of the starting code(MOVETO) of all sub paths
-        idxs, = np.where(codes == Path.MOVETO)
-        # Don't split before the first MOVETO.
-        idxs = idxs[1:]
-        return any(
+        if codes is not None:
+            vertices = self.get_path().vertices
+            # if the current path is concatenated by multiple sub paths.
+            # get the indexes of the starting code(MOVETO) of all sub paths
+            idxs, = np.where(codes == Path.MOVETO)
+            # Don't split before the first MOVETO.
+            idxs = idxs[1:]
+            subpaths = map(
+                Path, np.split(vertices, idxs), np.split(codes, idxs))
+        else:
+            subpaths = [self.get_path()]
+        inside = any(
             subpath.contains_point(
                 (mouseevent.x, mouseevent.y), self.get_transform(), radius)
-            for subpath in map(
-                Path, np.split(vertices, idxs), np.split(codes, idxs))), {}
+            for subpath in subpaths)
+        return inside, {}
 
     def contains_point(self, point, radius=None):
         """
