Code review comment for lp:~intellectronica/launchpad/patch-badge-performance
Revision history for this message
| Eleanor Berger (intellectronica) wrote | # |
© 2004
Canonical Ltd.
•
Terms of use
•
Data privacy
•
Contact Launchpad Support
•
Blog
•
Careers
•
System status
•
35a32c5
(Get the code!)
This branch optimizes the way we display the patch badge in bug listings by extending the decorator for bugtask listing items to include has_patch.
If the diff looks a bit short it's because:
1. The SQL for getting has_patch has already been added in the original branch (just not used).
2. I have not added any additional tests (since existing tests cover the functionality). I did test manually that the inefficient code path is not being entered when using this version.