[reportlab-users] ReportLab 2.6 Bugs

Stephan Richter stephan.richter at gmail.com
Fri Dec 21 08:55:13 EST 2012


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"


More information about the reportlab-users mailing list