SF.net SVN: matplotlib:[8673] branches/v1_0_maint/lib/matplotlib

[<ef...@us...>]

Revision: 8673
          http://matplotlib.svn.sourceforge.net/matplotlib/?rev=8673&view=rev
Author:   efiring
Date:     2010-09-01 19:44:36 +0000 (Wed, 01 Sep 2010)

Log Message:
-----------
[3054467] fix bug in PatchCollection; restore attribute access to Patch.fill

Modified Paths:
--------------
    branches/v1_0_maint/lib/matplotlib/collections.py
    branches/v1_0_maint/lib/matplotlib/patches.py

Modified: branches/v1_0_maint/lib/matplotlib/collections.py
===================================================================
--- branches/v1_0_maint/lib/matplotlib/collections.py	2010-08-30 15:20:42 UTC (rev 8672)
+++ branches/v1_0_maint/lib/matplotlib/collections.py	2010-09-01 19:44:36 UTC (rev 8673)
@@ -1034,7 +1034,7 @@
 
         if match_original:
             def determine_facecolor(patch):
-                if patch.fill:
+                if patch.get_fill():
                     return patch.get_facecolor()
                 return [0, 0, 0, 0]
 

Modified: branches/v1_0_maint/lib/matplotlib/patches.py
===================================================================
--- branches/v1_0_maint/lib/matplotlib/patches.py	2010-08-30 15:20:42 UTC (rev 8672)
+++ branches/v1_0_maint/lib/matplotlib/patches.py	2010-09-01 19:44:36 UTC (rev 8673)
@@ -312,13 +312,18 @@
 
         ACCEPTS: [True | False]
         """
-        self._fill = b
+        self._fill = bool(b)
         self.set_facecolor(self._original_facecolor)
 
     def get_fill(self):
         'return whether fill is set'
         return self._fill
 
+    # Make fill a property so as to preserve the long-standing
+    # but somewhat inconsistent behavior in which fill was an
+    # attribute.
+    fill = property(get_fill, set_fill)
+
     def set_hatch(self, hatch):
         """
         Set the hatching pattern


This was sent by the SourceForge.net collaborative development platform, the world's largest Open Source development site.