It should be ```that.saveSettings(that.get("settings", {}))``` because ```that``` refer to the user object and we should save the all settings.

But I think the best solution should be to change ```saveSettings``` to:

```
// Save user settings
        saveSettings: function(data) {
            var that = this;
            data = data || this.get("settings");
            return loading.show(rpc.execute("auth/settings", data).then(function(settings) {
                that.set("settings", settings);
            }), "Saving Settings");
        }
```