Register a single view (SeriesLanguageView) for all ISeriesLanguage-implementing objects

Bug #496361 reported by Adi Roiban
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Launchpad itself
Triaged
Low
Unassigned

Bug Description

------------------
From: Danilo
Subiect: Re: [Merge] lp:~adiroiban/launchpad/bug-427319 into lp:launchpad/devel
Data: Fri, 11 Dec 2009 19:13:29 -0000

Btw, nice refactoring Adi. Thanks for doing it! It'd be nice to move it even farther where we register only a single view (SeriesLanguageView) for all ISeriesLanguage-implementing objects (note, this interface doesn't exist yet :), but let's leave that for some other time :)

Also, now that these views are unified, it makes sense to also unify the view tests: i.e. I believe mostly setUp method would have to be changed to accommodate different ways to construct distroserieslanguages and productserieslanguages, but that would make it easy to extend these views to sourcepackagelanguage and projectlanguage views in the future.
----------------

Action plan:
* Create ISeriesLanguage
* Change Distro and Product seriesLanguage to use the new Interface
* Unify the view tests

Adi Roiban (adiroiban)
Changed in rosetta:
assignee: nobody → Adi Roiban (adiroiban)
Adi Roiban (adiroiban)
Changed in rosetta:
assignee: Adi Roiban (adiroiban) → nobody
Changed in rosetta:
status: New → Triaged
importance: Undecided → High
tags: added: tech-debt
Revision history for this message
Robert Collins (lifeless) wrote :

Still nice to do, not currently on the roadmap. Patches solicited!

Changed in launchpad:
importance: High → Low
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

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