Put "Everyone" and "Everyone at [local instance]" at the top of ToSelector

diff --git a/lib/toselector.php b/lib/toselector.php
index a295dcd8ab8ef5e6d50a751854f26ce937862d59..2026c1dfdd702bbf83bc0a00dabb8ceb991b2d07 100644 (file)
--- a/lib/toselector.php
+++ b/lib/toselector.php
@@ -80,16 +80,7 @@ class ToSelector extends Widget
     function show()
     {
         $choices = array();
-        $default = 'public:site';
-
-        if (!common_config('site', 'private')) {
-            // TRANS: Option in drop-down of potential addressees.
-            $choices['public:everyone'] = _m('SENDTO','Everyone');
-            $default = 'public:everyone';
-        }
-        // TRANS: Option in drop-down of potential addressees.
-        // TRANS: %s is a StatusNet sitename.
-        $choices['public:site'] = sprintf(_('Everyone at %s'), common_config('site', 'name'));
+        $default = common_config('site', 'private') ? 'public:site' : 'public:everyone';
 
         $groups = $this->user->getGroups();
 
@@ -125,6 +116,21 @@ class ToSelector extends Widget
         // alphabetical order
         asort($choices);
 
+        // Reverse so we can add entries at the end (can't unshift with a key)
+        $choices = array_reverse($choices);
+
+        // TRANS: Option in drop-down of potential addressees.
+        // TRANS: %s is a StatusNet sitename.
+        $choices['public:site'] = sprintf(_('Everyone at %s'), common_config('site', 'name'));
+
+        if (!common_config('site', 'private')) {
+            // TRANS: Option in drop-down of potential addressees.
+            $choices['public:everyone'] = _m('SENDTO','Everyone');
+        }
+
+        // Return the order
+        $choices = array_reverse($choices);
+
         $this->out->dropdown($this->id,
                              // TRANS: Label for drop-down of potential addressees.
                              _m('LABEL','To:'),