Skip to content

Commit 22efebd

Browse files
committed
Fix source links for ChatZilla and Venkman
Also put en-US folders in alphabetical order.
1 parent a73a977 commit 22efebd

File tree

2 files changed

+68
-32
lines changed

2 files changed

+68
-32
lines changed

app/classes/Transvision/VersionControl.php

Lines changed: 56 additions & 32 deletions
Original file line numberDiff line numberDiff line change
@@ -82,8 +82,10 @@ public static function hgPath($locale, $repo, $path)
8282
if (Strings::startsWith($repo, 'gaia')
8383
|| in_array(
8484
$base_folder,
85-
['apps', 'shared', 'showcase_apps',
86-
'test_apps', 'test_external_apps', ]
85+
[
86+
'apps', 'shared', 'showcase_apps',
87+
'test_apps', 'test_external_apps',
88+
]
8789
)
8890
) {
8991
$locale = Project::getLocaleInContext($locale, $repo);
@@ -98,47 +100,47 @@ public static function hgPath($locale, $repo, $path)
98100
return $url . $path . '/' . $entity_file;
99101
}
100102

101-
$en_US_Folder_Mess = [
103+
$en_US_folder_mess = [
104+
'b2g/',
102105
'b2g/branding/official/',
103106
'b2g/branding/unofficial/',
104-
'b2g/',
105-
'netwerk/',
106-
'embedding/android/',
107-
'testing/extensions/community/chrome/',
108-
'intl/',
109-
'extensions/spellcheck/',
110-
'services/sync/',
111-
'mobile/android/branding/aurora/',
112-
'mobile/android/branding/official/',
113-
'mobile/android/branding/nightly/',
114-
'mobile/android/branding/unofficial/',
115-
'mobile/android/branding/beta/',
116-
'mobile/android/base/',
117-
'mobile/android/',
118-
'mobile/',
119-
'security/manager/',
120-
'toolkit/content/tests/fennec-tile-testapp/chrome/',
121-
'toolkit/',
107+
'browser/',
122108
'browser/branding/aurora/',
123-
'browser/branding/official/',
124109
'browser/branding/nightly/',
110+
'browser/branding/official/',
125111
'browser/branding/unofficial/',
126-
'browser/',
127112
'browser/extensions/pocket/',
113+
'calendar/',
114+
'chat/',
128115
'devtools/client/',
129116
'devtools/shared/',
130-
'layout/tools/layout-debug/ui/',
131117
'dom/',
132-
'webapprt/',
133-
'chat/',
134-
'suite/',
135-
'other-licenses/branding/thunderbird/',
118+
'editor/ui/',
119+
'embedding/android/',
120+
'extensions/spellcheck/',
121+
'intl/',
122+
'layout/tools/layout-debug/ui/',
123+
'mail/',
136124
'mail/branding/aurora/',
137125
'mail/branding/nightly/',
138-
'mail/',
139126
'mail/test/resources/mozmill/mozmill/extension/',
140-
'editor/ui/',
141-
'calendar/',
127+
'mobile/',
128+
'mobile/android/',
129+
'mobile/android/base/',
130+
'mobile/android/branding/aurora/',
131+
'mobile/android/branding/beta/',
132+
'mobile/android/branding/nightly/',
133+
'mobile/android/branding/official/',
134+
'mobile/android/branding/unofficial/',
135+
'netwerk/',
136+
'other-licenses/branding/thunderbird/',
137+
'security/manager/',
138+
'services/sync/',
139+
'suite/',
140+
'testing/extensions/community/chrome/',
141+
'toolkit/',
142+
'toolkit/content/tests/fennec-tile-testapp/chrome/',
143+
'webapprt/',
142144
];
143145

144146
// Desktop repos
@@ -149,6 +151,28 @@ public static function hgPath($locale, $repo, $path)
149151
$url .= '/releases/l10n/mozilla-' . $repo . '/' . $locale . '/file/default/';
150152
}
151153
} else {
154+
// Chatzilla and Venkman are in separate repositories
155+
if ($base_folder == 'extensions') {
156+
switch ($exploded_path[1]) {
157+
case 'irc':
158+
$url .= '/chatzilla';
159+
$found_extension = true;
160+
break;
161+
case 'venkman':
162+
$url .= '/venkman';
163+
$found_extension = true;
164+
break;
165+
default:
166+
$found_extension = false;
167+
break;
168+
}
169+
170+
if ($found_extension) {
171+
return "{$url}/file/default/locales/en-US/chrome/{$entity_file}";
172+
}
173+
}
174+
175+
// comm-central folders
152176
if (in_array(
153177
$base_folder,
154178
['calendar', 'chat', 'editor', 'ldap', 'mail', 'mailnews', 'suite']
@@ -167,7 +191,7 @@ public static function hgPath($locale, $repo, $path)
167191
$loop = true;
168192

169193
while ($loop && count($exploded_path) > 0) {
170-
if (in_array(implode('/', $exploded_path) . '/', $en_US_Folder_Mess)) {
194+
if (in_array(implode('/', $exploded_path) . '/', $en_US_folder_mess)) {
171195
$path_part1 = implode('/', $exploded_path) . '/locales/en-US';
172196
$pattern = preg_quote(implode('/', $exploded_path), '/');
173197
$path = preg_replace('/' . $pattern . '/', $path_part1, $path, 1);

tests/units/Transvision/VersionControl.php

Lines changed: 12 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -129,6 +129,18 @@ public function hgFileDP()
129129
'browser/chrome/browser/browser-pocket.properties:maxtaglength',
130130
'https://hg.mozilla.org/releases/mozilla-beta/file/default/browser/locales/en-US/chrome/browser/browser-pocket.properties',
131131
],
132+
[
133+
'en-US',
134+
'beta',
135+
'extensions/venkman/chrome/venkman.dtd:Help.about',
136+
'https://hg.mozilla.org/venkman/file/default/locales/en-US/chrome/venkman.dtd',
137+
],
138+
[
139+
'en-US',
140+
'aurora',
141+
'extensions/irc/chrome/chatzilla.properties:msg.save.files.folder',
142+
'https://hg.mozilla.org/chatzilla/file/default/locales/en-US/chrome/chatzilla.properties',
143+
],
132144
];
133145
}
134146

0 commit comments

Comments
 (0)