Find the system locale from the browser or OS

pkg/intl/intl_standalone.dart

Index: pkg/intl/intl_standalone.dart
===================================================================
--- pkg/intl/intl_standalone.dart	(revision 0)
+++ pkg/intl/intl_standalone.dart	(revision 0)
@@ -0,0 +1,115 @@
+// Copyright (c) 2012, the Dart project authors.  Please see the AUTHORS file
+// for details. All rights reserved. Use of this source code is governed by a
+// BSD-style license that can be found in the LICENSE file.
+
+/**
+ * This provides facilities for Internationalization that are only available
+ * when running standalone. You should import only one of this or
+ * intl_browser.dart. Right now the only thing provided here is finding
+ * the operating system locale.
+ */
+
+#library("intl_standalone");
+
+#import("dart:io");
+#import("intl.dart");
+
+// TODO(alanknight): The need to do this by forcing the user to specially
+// import a particular library is a horrible hack, only done because there
+// seems to be no graceful way to do this at all. Either mirror access on
+// dart2js or the ability to do spawnUri in the browser would be promising
+// as ways to get rid of this requirement.
+/**
+ * Find the system locale, accessed via the appropriate system APIs, and
+ * set it as the default for internationalization operations in
+ * the [Intl.systemLocale] variable. To find it, we
+ * check the "LANG" environment variable on *nix, use the "systeminfo"
+ * command on Windows, and use "defaults read -g AppleLocale" on the Mac. This

[Emily Fortuna, 2012/09/17 21:26:38]

rephrase the mac section to say you fall back to AppleLocale if LANG isn't set
(I see that in the code, but I didn't get that from reading this)

[Alan Knight, 2012/09/17 22:16:18]

Done.

+ * is not an ideal version of getting a single system locale, even if that
+ * concept really made sense, but it's a reasonable first approximation that's
+ * not too difficult to get. If it can't find the locale information, it will
+ * not modify [Intl.systemLocale] and the Future will complete with null.
+ */
+Future<String> findSystemLocale() {
+  // On *nix systems we expect this is an environment variable, which is the
+  // easiest thing to check. On a Mac the environment variable may be present
+  // so always check it first.
+  String baseLocale = _checkEnvironmentVariable();
+  if (baseLocale != null) return _setLocale(baseLocale);
+  if (Platform.operatingSystem == 'windows') {
+    return _getWindowsSystemInfo();
+  }
+  if (Platform.operatingSystem == 'macos') {
+    return _getAppleDefaults();
+  }
+  // We can't find anything, don't set the system locale and return null.
+  return new Future.immediate(null);
+}
+
+/**
+ * Check to see if we have a "LANG" environment variable we can use and return
+ * it if found. Otherwise return null;
+ */
+String _checkEnvironmentVariable() {
+  try {
+    return Platform.environment['LANG'];
+  } catch (e) {};
+  return null;
+}
+
+/**
+ * Run the "defaults read -g AppleLocale" command and return the output in
+ * a future.
+ */
+Future _getAppleDefaults() {
+  var p = Process.run('defaults', ['read', '-g', 'AppleLocale']);
+  var myResult = p.chain((result) => _checkResult(result, _appleDefaultsRegex));
+  return myResult;
+}
+
+/**
+ * Run the "systemlocale" command and return the output in a future.
+ */
+Future _getWindowsSystemInfo() {
+  var p = Process.run('systeminfo', []);
+  var myResult = p.chain((result) => _checkResult(result, _sysInfoRegex));
+  return myResult;
+}
+
+/**
+ * Regular expression to match the expected output of systeminfo on
+ * Windows. e.g. System Locale:<tab>en_US;English (United States)
+ */
+RegExp _sysInfoRegex = const RegExp(@"System Locale:\t(\w\w-\w+);");

[Emily Fortuna, 2012/09/17 21:26:38]

I could be persuaded otherwise, but I prefer having all of our constants/fields
up at the top of the file.

[Alan Knight, 2012/09/17 22:16:18]

I prefer having the important bit at the top, which in this case is the only
public function. But put the constants right below it.

+
+/**
+ * Regular expression to match the expected output of reading the defaults
+ * database for AppleLanguages on Mac systems.
+ * e.g. {
+ *     en,
+ *     "pt-PT",
+ *     ...
+ */
+RegExp _appleDefaultsRegex = const RegExp(@'(\w\w_\w+)');
+
+/**
+ * Given [result], find its text and extract the locale from it using
+ * [regex], and set it as the system locale. If the process didn't run correctly
+ * then don't set the variable and return a future that completes with null.
+ */
+Future<String> _checkResult(ProcessResult result, RegExp regex) {
+  if (result.exitCode != 0) return new Future.immediate(null);
+  var match = regex.firstMatch(result.stdout);
+  if (match == null) return new Future.immediate(null);
+  var locale = match.group(1);
+  _setLocale(locale);
+  return new Future.immediate(locale);
+}
+
+/**
+ * Set [Intl.systemLocale] to be the canonicalizedLocale of [aLocale].
+ */
+Future<String> _setLocale(aLocale) {
+  Intl.systemLocale = Intl.canonicalizedLocale(aLocale);
+  return new Future.immediate(Intl.systemLocale);
+}