Opened 10 years ago

Closed 10 years ago

#13930 closed defect (fixed)

remove bad is_subcategory implementation

Reported by: Nils Bruin Owned by: Nicolas M. Thiéry
Priority: major Milestone: sage-5.6
Component: categories Keywords:
Cc: Simon King Merged in: sage-5.6.rc0
Authors: Nils Bruin Reviewers: Simon King
Report Upstream: N/A Work issues:
Branch: Commit:
Dependencies: Stopgaps:

Status badges


As discovered on #13400, there is a bad implementation of is_subcategory in categories/ It doesn't cache although it should, it doesn't have a proper docstring and it shadows a better implementation on a superclass. It simply needs to be removed.

Attachments (1)

trac_13930-del_is_subcat.patch (1.0 KB) - added by Nils Bruin 10 years ago.
Straightforward fix

Download all attachments as: .zip

Change History (6)

Changed 10 years ago by Nils Bruin

Straightforward fix

comment:1 Changed 10 years ago by Nils Bruin

Authors: Nils Bruin
Status: newneeds_review

comment:2 Changed 10 years ago by Simon King

Cc: Simon King added

comment:3 Changed 10 years ago by Nils Bruin

Simon and Nicholas would be very qualified to review this.

comment:4 Changed 10 years ago by Simon King

Reviewers: Simon King
Status: needs_reviewpositive_review

We have discussed on #13400 that the removal if this is_subcat method makes sense, the change is indirectly doctested by the fact that the existing tests don't break. Hence, the patch seems fine to me.

The patchbot is happy, and I am happy as well.

comment:5 Changed 10 years ago by Jeroen Demeyer

Merged in: sage-5.6.rc0
Resolution: fixed
Status: positive_reviewclosed
Note: See TracTickets for help on using tickets.