Default to NOT ask for current location for new users

It may be a bad experience for new users to immediately when trying
out the service be asked for their geographical position. Instead,
let them opt-in for this behaviour.

classes/User.php

@@ -874,13 +874,12 @@ class User extends Managed_DataObject
         } else if ($cfg == 'never') {
             return false;
         } else { // user
-            $share = true;
+            $share = common_config('location', 'sharedefault');
 
+            // Check if user has a personal setting for this
             $prefs = User_location_prefs::staticGet('user_id', $this->id);
 
-            if (empty($prefs)) {
-                $share = common_config('location', 'sharedefault');
-            } else {
+            if (!empty($prefs)) {
                 $share = $prefs->share_location;
                 $prefs->free();
             }

lib/default.php

@@ -294,7 +294,7 @@ $default =
         array('contentlimit' => null),
         'location' =>
         array('share' => 'user', // whether to share location; 'always', 'user', 'never'
-              'sharedefault' => true),
+              'sharedefault' => false),
         'omb' =>
         array('timeout' => 5), // HTTP request timeout in seconds when contacting remote hosts for OMB updates
         'logincommand' =>