[reportlab-users] ReportLab 2.6 Bugs

[Stephan Richter]

On Friday, December 21, 2012 10:41:44 AM Robin Becker wrote:

> I agree with the  pdfform & testshapes resets, but I'm not sure why we need

> to  set setSideLabels on the Pie3d class. Pie3d inherits from Pie and Pie

> has a default initialization of setSideLabels=0 which should overwrite any

> Pie3d class value for it.


That is correct, but the initialization code is not called. I am surprised 
that your Pie3d tests would pass at all. Basically, my z3c.rml testsuite fails 
for 3D pie charts without that fix.

Note how Pie3d overwrites __init__ without a super call, but Pie.__init__ 
clearly sets sideLabels.

I also think that there are no tests for Pie3d. Could it be that z3c.rml is 
the most comprehensive testsuite? I need to do a coverage test run to see how 
much code coverage I get in z3c.rml and reportlab. (I should really add 
support/tests for some of the iconic widgets to make it more complete.)

Regards,
Stephan
-- 
Entrepreneur and Software Geek
Google me. "Zope Stephan Richter"