416779 - Implement "language packs" on the "languages and dictionaries" page

Status: VERIFIED FIXED

(addons.mozilla.org Graveyard :: Public Pages, defect)

Description

[Stephen Donner [:stephend] Not actively reading bugmail]

Implement the "language packs" functionality as mocked-up in http://mozilla.focalcurve.com/categorylanding-languages.html

The staging page where we lack them---no DB data yet--is https://remora-reskin.stage.mozilla.com/en-US/firefox/browse/type:3

See https://bugzilla.mozilla.org/show_bug.cgi?id=414884#c3

Comment 1

[Fred Wenzel [:wenzel]]

All right, in today's meeting we decided to run some DB magic in order to mark the "extensions" that are actually language packs with the right add-on type.

However, they don't seem to have a uniform way of describing what locale they are focusing on (unlike dictionaries, most of which have a parsable GUID), so we'll be unable to put them into the list in the right order.

I have a feeling in order to do this right we need a new database table for "this dictionary or language pack covers locale x". Though this obviously needs additional GUI for the dev panel etc then :-/

Clouserw, can you comment?

Comment 2

[Fred Wenzel [:wenzel]]

A search for "language pack" is returning 38 possible candidates.

https://preview.addons.mozilla.org/en-US/firefox/search?q=language+pack&cat=all

Comment 3

[Basil Hashem [:baz]]

ping.

Given our discussion on today's call - we should try to get some resolution to this page and its design. If you need some manual figuring out of the locale for these 38 language packs, I'll be OK with doing that.

Comment 4

[Wil Clouser [:clouserw]]

We're putting "Language Pack" as a separate type when someone first uploads an add-on, right?  So, could we just put a textbox that asks for what locale there?  (Or better, put it in the .rdf)

Filling in the locales manually in the db would work for the short term

Comment 5

[Fred Wenzel [:wenzel]]

Okay, I'll add a new field to the DB that holds the "target locale" for dictionaries and language packs (and is NULL otherwise). Devs will be able to enter a language code in there.

Displaying the dictionaries/lang pack page is then very easy, because there's no guessing anymore what locale they belong to.

Protection against screwed up language codes and such comes from the sandbox: Only dictionaries/lang packs that have a correct locale code assigned may be pushed public.

Comment 6

[Fred Wenzel [:wenzel]]

clouserw says:
- for dictionaries we need to store 3 new things:  locale, additional info, primary flag
- the first two are up to the developer, the flag is decided by admins

Comment 7

[Fred Wenzel [:wenzel]]

Attachment: Allow adding target locale to dicts and lang packs

This patch allows adding target locale and locale disambiguation info to dictionaries and application language packs.

Does that look right, clouserw?

Note that this needs some l10n, I am not quite sure what to do about that: If we want to deploy this change by 3.2, we need to give the fields names in the dev cp, in spite of l10n being frozen. Suggestions?

Comment 8

[Fred Wenzel [:wenzel]]

Attachment: Incremental SQL

This is the SQL to be run on a current database, in order to add the two fields. It also fills in the target locale for all current dictionaries.

Comment 9

[Fred Wenzel [:wenzel]]

Attachment: Updated patch, including l10n

Updated patch: It now contains l10n, en-US only.

Will work on the frontend (dictionaries page) part once this is reviewed.

Comment 10

[Wil Clouser [:clouserw]]

Comment on attachment 308910 [details] [diff] [review]
Updated patch, including l10n

Looking good to me.  It would be nice if we could say something about what the target locale means, just to make sure we're all using iso 639-1 names with optional country codes afterwards.  Not sure how to make that sound simple though.  In the mean time, r+

Comment 11

[Fred Wenzel [:wenzel]]

Code changes (so far) are in r11250, l10n changes in r11251. The incremental SQL to be run is on the wiki, too.

Comment 12

[Fred Wenzel [:wenzel]]

Attachment: Screenshot: Multiple dictionaries per target locale

I implemented the front-end side now. The attachment shows what it looks like when you add two dictionaries to the same target locale now.

Note that language packs still don't show up because most of them (all?) have the wrong addon type and none have a target locale assigned atm. I filed bug 422636 to have this fixed.

Comment 13

[Fred Wenzel [:wenzel]]

This is in r11265, and fixed. Please reopen if you notice anything I may have overlooked.

Comment 14

[Stephen Donner [:stephend] Not actively reading bugmail]

Verified; I installed them all (13) using Firefox 2.0.0.12:

https://preview.addons.mozilla.org/en-US/firefox/browse/type:3#dictionaries