| Summary: | Previous bug-fix in tables.py not applied in FC15/16 | ||
|---|---|---|---|
| Product: | [Fedora] Fedora | Reporter: | Nils Smeds <nils.smeds> |
| Component: | python-reportlab | Assignee: | Konstantin Ryabitsev <icon> |
| Status: | CLOSED WONTFIX | QA Contact: | Fedora Extras Quality Assurance <extras-qa> |
| Severity: | medium | Docs Contact: | |
| Priority: | unspecified | ||
| Version: | 16 | CC: | icon |
| Target Milestone: | --- | ||
| Target Release: | --- | ||
| Hardware: | All | ||
| OS: | Linux | ||
| Whiteboard: | |||
| Fixed In Version: | Doc Type: | Bug Fix | |
| Doc Text: | Story Points: | --- | |
| Clone Of: | Environment: | ||
| Last Closed: | 2013-02-13 14:49:08 UTC | Type: | --- |
| Regression: | --- | Mount Type: | --- |
| Documentation: | --- | CRM: | |
| Verified Versions: | Category: | --- | |
| oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |
| Cloudforms Team: | --- | Target Upstream Version: | |
This message is a reminder that Fedora 16 is nearing its end of life. Approximately 4 (four) weeks from now Fedora will stop maintaining and issuing updates for Fedora 16. It is Fedora's policy to close all bug reports from releases that are no longer maintained. At that time this bug will be closed as WONTFIX if it remains open with a Fedora 'version' of '16'. Package Maintainer: If you wish for this bug to remain open because you plan to fix it in a currently maintained version, simply change the 'version' to a later Fedora version prior to Fedora 16's end of life. Bug Reporter: Thank you for reporting this issue and we are sorry that we may not be able to fix it before Fedora 16 is end of life. If you would still like to see this bug fixed and are able to reproduce it against a later version of Fedora, you are encouraged to click on "Clone This Bug" and open it against that version of Fedora. Although we aim to fix as many bugs as possible during every release's lifetime, sometimes those efforts are overtaken by events. Often a more recent Fedora release includes newer upstream software that fixes bugs or makes them obsolete. The process we are following is described here: http://fedoraproject.org/wiki/BugZappers/HouseKeeping Fedora 16 changed to end-of-life (EOL) status on 2013-02-12. Fedora 16 is no longer maintained, which means that it will not receive any further security or bug fix updates. As a result we are closing this bug. If you can reproduce this bug against a currently maintained version of Fedora please feel free to reopen this bug against that version. Thank you for reporting this bug and we are sorry it could not be fixed. |
Description of problem: Version-Release number of selected component (if applicable): Name : python-reportlab Version : 2.5 Release : 3.fc15 How reproducible: Everytime for specific inputs to the library. Steps to Reproduce: Requires data and a python code that can be submitted if needed, but as this is a previously known bug (see FC14 below) it is not needed. Actual results: Application crashes with an error due to summing of a None valued variable Expected results: Additional info: Here is one patch to resolve the issue: *** /usr/lib64/python2.7/site-packages/reportlab/platypus/tables.py 2012-03-20 14:51:29.242188950 +0100 --- /usr/lib64/python2.7/site-packages/reportlab/platypus/tables.py-orig 2011-02-09 08:34:11.000000000 +0100 *************** def spanFixDim(V0,V,spanCons,lim=None,FU *** 202,208 **** for (x0,x1),v in spanCons.iteritems(): if x0>=lim: continue x1 += 1 ! t = sum([ (V[x] or 0) +M.get(x,0) for x in xrange(x0,x1)]) if t>=v-FUZZ: continue #already good enough X = [x for x in xrange(x0,x1) if V0[x] is None] #variable candidates if not X: continue #something wrong here mate --- 202,208 ---- for (x0,x1),v in spanCons.iteritems(): if x0>=lim: continue x1 += 1 ! t = sum([V[x]+M.get(x,0) for x in xrange(x0,x1)]) if t>=v-FUZZ: continue #already good enough X = [x for x in xrange(x0,x1) if V0[x] is None] #variable candidates if not X: continue #something wrong here mate In FC14 the offending line is just commented out and replaced by: t = 0 Either solution works for the cases where my application runs into this problem.