From d99358c7192283e929ff5427ec62456e622b2e3c Mon Sep 17 00:00:00 2001
From: Michael M Slusarz <slusarz@curecanti.org>
Date: Thu, 15 Jul 2010 13:19:03 -0600
Subject: [PATCH] Completely remove sorting from config/nls.php

Do sorting only when we need it.
---
 fima/data.php        |  5 +++--
 horde/config/nls.php |  4 ----
 imp/compose.php      |  1 +
 imp/lib/Prefs/Ui.php | 10 ++++++++++
 turba/data.php       |  5 +++--
 5 files changed, 17 insertions(+), 8 deletions(-)

diff --git a/fima/data.php b/fima/data.php
index aa0ee84fd..771332b37 100644
--- a/fima/data.php
+++ b/fima/data.php
@@ -208,8 +208,9 @@ require FIMA_TEMPLATES . '/menu.inc';
 
 if ($next_step == Horde_Data::IMPORT_FILE) {
     /* Build the charset options. */
-    $charsets = Horde_Nls::$config['encodings'];
-    $all_charsets = Horde_Nls::$config['charsets'];
+    $charsets = $registry->nlsconfig['encodings'];
+    asort($charsets);
+    $all_charsets = $registry->nlsconfig['charsets'];
     natcasesort($all_charsets);
     foreach ($all_charsets as $charset) {
         if (!isset($charsets[$charset])) {
diff --git a/horde/config/nls.php b/horde/config/nls.php
index 96373e762..0183721be 100644
--- a/horde/config/nls.php
+++ b/horde/config/nls.php
@@ -211,10 +211,6 @@ $horde_nls_config = array(
     )
 );
 
-/* Sort encodings. Disabled by default - encoding list is already sorted by
- * default in English. */
-//asort($horde_nls_config['encodings']);
-
 /* BSD charsets. */
 if (strpos(PHP_OS, 'BSD') !== false) {
     $horde_nls_config['charsets'] = array_merge($horde_nls_config['charsets'], array(
diff --git a/imp/compose.php b/imp/compose.php
index 3568f9237..5ba6799c1 100644
--- a/imp/compose.php
+++ b/imp/compose.php
@@ -819,6 +819,7 @@ if ($redirect) {
         $t->set('charset_label', Horde::label('charset', _("C_harset")));
         $t->set('charset_tabindex', ++$tabindex);
         $charset_array = array();
+        asort($registry->nlsconfig['encodings']);
         foreach ($registry->nlsconfig['encodings'] as $charset => $label) {
             $charset_array[] = array('value' => $charset, 'selected' => (strtolower($charset) == strtolower($encoding)), 'label' => $label);
         }
diff --git a/imp/lib/Prefs/Ui.php b/imp/lib/Prefs/Ui.php
index 8a0797458..7a4c92703 100644
--- a/imp/lib/Prefs/Ui.php
+++ b/imp/lib/Prefs/Ui.php
@@ -86,6 +86,11 @@ class IMP_Prefs_Ui
             if (empty($conf['compose']['allow_receipts'])) {
                 $ui->suppress[] = 'disposition_request_read';
             }
+
+            /* Sort encodings. */
+            if (!$prefs->isLocked('sending_charset')) {
+                asort($registry->nlsconfig['encodings']);
+            }
             break;
 
         case 'delmove':
@@ -269,6 +274,11 @@ class IMP_Prefs_Ui
                 $ui->suppress[] = 'html_image_replacement';
                 $ui->suppress[] = 'html_image_addrbook';
             }
+
+            /* Sort encodings. */
+            if (!$prefs->isLocked('default_msg_charset')) {
+                asort($registry->nlsconfig['encodings']);
+            }
             break;
         }
 
diff --git a/turba/data.php b/turba/data.php
index b4bed9559..7a3e8a915 100644
--- a/turba/data.php
+++ b/turba/data.php
@@ -532,8 +532,9 @@ if ($next_step == Horde_Data::IMPORT_FILE) {
     }
 
     /* Build the charset options. */
-    $charsets = Horde_Nls::$config['encodings'];
-    $all_charsets = Horde_Nls::$config['charsets'];
+    $charsets = $registry->nlsconfig['encodings'];
+    asort($charsets);
+    $all_charsets = $registry->nlsconfig['charsets'];
     natcasesort($all_charsets);
     foreach ($all_charsets as $charset) {
         if (!isset($charsets[$charset])) {
-- 
2.11.0