531883 - Implement per-locale localizer dashboard

Status: VERIFIED FIXED

(addons.mozilla.org Graveyard :: Localization, enhancement, P1)

Description

[Fred Wenzel [:wenzel]]

The mockup (attachment 414601 [details]) and content have widely been accepted in bug 531145. This bug is about implementing it.

Core scope for now: event log + feed, localizer contacts, statistics, and locale switching.

I'll file more bugs on the remaining features soon. Those should make it into the release, but can be postponed if necessary.

Comment 1

[Fred Wenzel [:wenzel]]

Attachment: Patch, rev. 1

Here's a first step (I am intentionally not making a 500k patch for this ;)). It mostly covers the event log. In order to import the viewvc RSS feed, run the new maintenance.php task "l10n_rss". We'll probably make that a once-per-hour cron job, I'd suggest.

The URL you want to look at is /en-US/localizers/de, for example.

Comment 2

[Wil Clouser [:clouserw]]

+ $lang_re = implode('\s*', array('/\$supported_languages', '=', 'array\(', '([^)]+)', '\);/m'));
+ $language_inc = file_get_contents(dirname(__FILE__).'/../site/app/config/language.inc.php');
+ preg_match($lang_re, $language_inc, $matches);
+ eval('$supported_languages=array('.$matches[1].');');

I haven't even scrolled past the first page.  Are you kidding! :)

Comment 3

[Fred Wenzel [:wenzel]]

(In reply to comment #2)
> I haven't even scrolled past the first page.  Are you kidding! :)

Are you saying I need to run a full-blown cake inside our maintenance scripts just to load a config file? Sigh...

All right, I see what else I can do.

Comment 4

[Fred Wenzel [:wenzel]]

Attachment: Patch, rev. 2

This is better. I noticed we don't need actually cake if we can pretend to have it around just barely enough to make a clean import.

Comment 5

[Fred Wenzel [:wenzel]]

(In reply to comment #4)
> we don't need actually cake
Clearly, my word order doesn't match what I am thinking due to a shortage of coffee.

Comment 6

[Wil Clouser [:clouserw]]

Comment on attachment 415362 [details] [diff] [review]
Patch, rev. 2

This is looking great.  Some thoughts:

1) There are entries from 2 years ago showing up in the list.  Where do those come from?  Are they not getting cleaned up with the gc script?

2) My gut says we should use a table called l10n_eventlog to separate this out.  Otherwise this table is going to get huge recording everything from the whole site.

3) "Recent Activity for fr" "the fr .po file" "in touch with the fr L10n" "Current fr Localizers" -- I think "fr" should be the full localized name in each of these cases.

Comment 7

[Fred Wenzel [:wenzel]]

(In reply to comment #6)
> This is looking great.  Some thoughts:
> 
> 1) There are entries from 2 years ago showing up in the list.  Where do those
> come from?  Are they not getting cleaned up with the gc script?

I don't think they are, but I think you fixed that in the meantime, IIRC.

> 2) My gut says we should use a table called l10n_eventlog to separate this out.
>  Otherwise this table is going to get huge recording everything from the whole
> site.

Done. I'll make that part of the gearman patch over in bug 532453.

> 3) "Recent Activity for fr" "the fr .po file" "in touch with the fr L10n"
> "Current fr Localizers" -- I think "fr" should be the full localized name in
> each of these cases.

We don't have the grammatically correct case for these sentences, but I could probably use the full names at the end of the sentence with a colon: 'Current Localizers for: French'

Comment 8

[Fred Wenzel [:wenzel]]

Another thing clouserw pointed out on IRC (just so I don't forget): we should use per-event icons that fit our l10n events better.

Comment 9

[Fred Wenzel [:wenzel]]

I checked in the first chunks of code into SVN revisions r57967 through r57975. This won't look great yet until we have the new l10n maintenance tasks running on preview. But I'll attach a screenshot for everybody's enjoyment in a minute.

Comment 10

[Fred Wenzel [:wenzel]]

Attachment: Current Status: Screen Shot

The screen shot, as promised.

Comment 11

[Fred Wenzel [:wenzel]]

I added a final maintenance task to calculate the translation status of pages (r58412).

Comment 12

[Fred Wenzel [:wenzel]]

"featured collections" was missing as L10n category. Added in r58788.

Comment 13

[Fred Wenzel [:wenzel]]

I removed "Applications" from the L10n categories (cf. bug 476184) in r58878. This doesn't actually remove it from the codebase, so this does not quite fix that bug yet.

This bug is fixed, but if there are any problems left, please file individual bugs.

Comment 14

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

Verified FIXED on, e.g.:

https://preview.addons.mozilla.org/en-US/localizers
https://preview.addons.mozilla.org/fr/localizers/fr, etc.