Launchpad itself

bazaar-portlet-importstats is cruft

Bug #138844 reported by Michael Hudson-Doyle
2
Affects Status Importance Assigned to Milestone
Launchpad itself
Fix Released
Low
Tim Penhey

Bug Description

There's a template (bazaar-portlet-importstats.pt) and associated zcml, view class methods, pagetests, etc that looks like it was once included on the code.launchpad.net homepage but certainly isn't now.

It should be deleted.

Tags: lp-code
Tim Penhey (thumper)
Changed in launchpad-bazaar:
assignee: nobody → thumper
importance: Undecided → Low
status: New → Confirmed
Tim Penhey (thumper)
Changed in launchpad-bazaar:
status: Confirmed → In Progress
Revision history for this message
Michael Hudson-Doyle (mwhudson) wrote :

You can delete more than is in your for-review branch I think -- certainly the testing_count etc methods of BazaarApplicationView and I think the status argument to import count (and as this leaves only one call to importcount, that could be replaced with .searchImports().count()...)

Revision history for this message
Tim Penhey (thumper) wrote :

Good point.

More cleaning done.

Revision history for this message
Tim Penhey (thumper) wrote :

Fixed in RF 5004.

Changed in launchpad-bazaar:
status: In Progress → Fix Committed
Tim Penhey (thumper)
Changed in launchpad-bazaar:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.
Subscribing...

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.