Merge lp:~wgrant/launchpad/bug-750949 into lp:launchpad
Status: | Merged | ||||
---|---|---|---|---|---|
Approved by: | Robert Collins | ||||
Approved revision: | no longer in the source branch. | ||||
Merged at revision: | 12748 | ||||
Proposed branch: | lp:~wgrant/launchpad/bug-750949 | ||||
Merge into: | lp:launchpad | ||||
Diff against target: |
124 lines (+41/-7) 4 files modified
lib/canonical/launchpad/webapp/publication.py (+2/-0) lib/lp/bugs/browser/bugattachment.py (+3/-0) lib/lp/bugs/browser/tests/test_bugtask.py (+28/-3) lib/lp/bugs/model/bug.py (+8/-4) |
||||
To merge this branch: | bzr merge lp:~wgrant/launchpad/bug-750949 | ||||
Related bugs: |
|
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Robert Collins (community) | Approve | ||
Review via email: mp+56295@code.launchpad.net |
Commit message
[r=lifeless][bug=750949] Preload attachment LibraryFileContents to eliminate another BugTask:+index scaling issue.
Description of the change
Bug comments display the size of any associated attachments, which requires the LibraryFileContent. This branch expands the LFA preloading to LFCs too, and adds a test.
Ahhh, the test. Browsing to a bug page caused canonical_