Create an interface (ITranslatedLangaugeAggregator) for object providing a collection of ITranslatedLanguage

Bug #512735 reported by Adi Roiban
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Launchpad itself
Invalid
High
Unassigned

Bug Description

This is a follow up for bug 492375

There is another page that should be filtered for preferred languages: https://translations.edge.launchpad.net/ubuntu/karmic/+source/gnome-codec-install/+pots/gnome-codec-install

Since preferred languages are filtered in more than 4 places, maybe we can refactor the current code and only have the filtering in one place.

This work is about defining a ITranslatedLanguageAggregate interface and changing the current views to use it.

Merging the views into a common mixin should be done in a different branch.

Adi Roiban (adiroiban)
Changed in rosetta:
assignee: nobody → Adi Roiban (adiroiban)
Revision history for this message
Данило Шеган (danilo) wrote :

Yes, we should refactor all "has languages with translations" containers to implement a shared interface and then provide filtering over such interface.

tags: added: tech-debt ui
Changed in rosetta:
status: New → Triaged
importance: Undecided → High
Adi Roiban (adiroiban)
Changed in rosetta:
status: Triaged → In Progress
Adi Roiban (adiroiban)
summary: - At first diplay only preferred languages for a POFile from a
- POFileSubset
+ Create an interface for per language statistics
Adi Roiban (adiroiban)
description: updated
Adi Roiban (adiroiban)
summary: - Create an interface for per language statistics
+ Create an interface (ITranslatedLangaugeAggregator) for object providing
+ a collection of ITranslatedLanguage
Curtis Hovey (sinzui)
Changed in launchpad:
assignee: Adi Roiban (adiroiban) → nobody
status: In Progress → Triaged
Revision history for this message
Robert Collins (lifeless) wrote :

I'm going to close this - its an interesting speculation about how the code could be structured but these things age poorly as other maintenance happens.

Changed in launchpad:
status: Triaged → Invalid
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.