From 160b9cd1f888acfed4afee77816b7310c50a20e7 Mon Sep 17 00:00:00 2001
From: marcelveldt <marcelvanderveldt@MacBook-Silvia.local>
Date: Wed, 16 Oct 2019 09:29:31 +0200
Subject: [PATCH] fix config

---
 music_assistant/homeassistant.py              |  3 +-
 music_assistant/playerproviders/chromecast.py |  2 +-
 music_assistant/web/pages/config.vue.js       | 29 +++++++------------
 3 files changed, 13 insertions(+), 21 deletions(-)

diff --git a/music_assistant/homeassistant.py b/music_assistant/homeassistant.py
index de9f8d12..c4b3c5c5 100644
--- a/music_assistant/homeassistant.py
+++ b/music_assistant/homeassistant.py
@@ -59,7 +59,8 @@ class HomeAssistant():
         else:
             self._use_ssl = False
             self._host = url.replace('http://','').split('/')[0]
-        LOGGER.info('Homeassistant integration is enabled')
+        if self.enabled:
+            LOGGER.info('Homeassistant integration is enabled')
 
     async def setup(self):
         ''' perform async setup '''
diff --git a/music_assistant/playerproviders/chromecast.py b/music_assistant/playerproviders/chromecast.py
index 203dce86..2e9539fc 100644
--- a/music_assistant/playerproviders/chromecast.py
+++ b/music_assistant/playerproviders/chromecast.py
@@ -23,7 +23,7 @@ PROV_NAME = 'Chromecast'
 PROV_CLASS = 'ChromecastProvider'
 
 CONFIG_ENTRIES = [
-    (CONF_ENABLED, False, CONF_ENABLED),
+    (CONF_ENABLED, True, CONF_ENABLED),
     ]
 
 PLAYER_CONFIG_ENTRIES = [
diff --git a/music_assistant/web/pages/config.vue.js b/music_assistant/web/pages/config.vue.js
index c6b5eebf..0cf2bcf7 100755
--- a/music_assistant/web/pages/config.vue.js
+++ b/music_assistant/web/pages/config.vue.js
@@ -47,24 +47,24 @@ var Config = Vue.component('Config', {
                               </template>
                               <div v-for="conf_item_key in conf.player_settings[key].__desc__" v-if="conf.player_settings[key].enabled">
                                   <v-list-tile>
-                                        <v-switch v-if="typeof(conf_item_key[1]) == 'boolean'" v-model="conf.player_settings[key][conf_item_key[0]]" :label="$t('conf.'+conf_item_key[2])"></v-switch>
-                                        <v-text-field v-else-if="conf_item_key[1] == '<password>'" v-model="conf.player_settings[key][conf_item_key[0]]" :label="$t('conf.'+conf_item_key[2])" box type="password"></v-text-field>
-                                        <v-select v-else-if="conf_item_key[1] == '<player>'" v-model="conf.player_settings[key][conf_item_key[0]]" :label="$t('conf.'+conf_item_key[2])" 
+                                        <v-switch v-if="typeof(conf_item_key[1]) == 'boolean'" v-model="conf.player_settings[key][conf_item_key[0]]" :label="$t('conf.'+conf_item_key[2])" @change="confChanged('player_settings', key, conf.player_settings[key])"></v-switch>
+                                        <v-text-field v-else-if="conf_item_key[1] == '<password>'" v-model="conf.player_settings[key][conf_item_key[0]]" :label="$t('conf.'+conf_item_key[2])" box type="password" @change="confChanged('player_settings', key, conf.player_settings[key])"></v-text-field>
+                                        <v-select v-else-if="conf_item_key[1] == '<player>'" v-model="conf.player_settings[key][conf_item_key[0]]" :label="$t('conf.'+conf_item_key[2])" @change="confChanged('player_settings', key, conf.player_settings[key])"
                                           :items="playersLst"
                                           item-text="name"
                                           item-value="id" box>
                                         </v-select>
-                                        <v-select v-else-if="conf_item_key[0] == 'max_sample_rate'" v-model="conf.player_settings[key][conf_item_key[0]]" :label="$t('conf.'+conf_item_key[2])" :items="sample_rates" box></v-select>
-                                        <v-slider v-else-if="conf_item_key[0] == 'crossfade_duration'" v-model="conf.player_settings[key][conf_item_key[0]]" :label="$t('conf.'+conf_item_key[2])" min=0 max=10 box thumb-label></v-slider>
-                                        <v-text-field v-else v-model="conf.player_settings[key][conf_item_key[0]]" :label="$t('conf.'+conf_item_key[2])" box></v-text-field>
+                                        <v-select v-else-if="conf_item_key[0] == 'max_sample_rate'" v-model="conf.player_settings[key][conf_item_key[0]]" :label="$t('conf.'+conf_item_key[2])" :items="sample_rates" @change="confChanged('player_settings', key, conf.player_settings[key])" box></v-select>
+                                        <v-slider v-else-if="conf_item_key[0] == 'crossfade_duration'" v-model="conf.player_settings[key][conf_item_key[0]]" :label="$t('conf.'+conf_item_key[2])" @change="confChanged('player_settings', key, conf.player_settings[key])" min=0 max=10 box thumb-label></v-slider>
+                                        <v-text-field v-else v-model="conf.player_settings[key][conf_item_key[0]]" :label="$t('conf.'+conf_item_key[2])" @change="confChanged('player_settings', key, conf.player_settings[key])" box></v-text-field>
                                   </v-list-tile>
                                   <v-list-tile v-if="!conf.player_settings[key].enabled">
-                                        <v-switch v-model="conf.player_settings[key].enabled" :label="$t('conf.'+'enabled')"></v-switch>
+                                        <v-switch v-model="conf.player_settings[key].enabled" :label="$t('conf.'+'enabled')" @change="confChanged('player_settings', key, conf.player_settings[key])"></v-switch>
                                   </v-list-tile>
                               </div>
                               <div v-if="!conf.player_settings[key].enabled">
                                   <v-list-tile>
-                                      <v-switch v-model="conf.player_settings[key].enabled" :label="$t('conf.'+'enabled')"></v-switch>
+                                      <v-switch v-model="conf.player_settings[key].enabled" :label="$t('conf.'+'enabled')" @change="confChanged('player_settings', key, conf.player_settings[key])"></v-switch>
                                   </v-list-tile>
                               </div>
                                 <v-divider></v-divider>
@@ -125,16 +125,6 @@ var Config = Vue.component('Config', {
           console.log("error", error);
         });
     },
-    saveConfig () {
-      axios
-        .post('/api/config', this.conf)
-        .then(result => {
-          console.log(result);
-        })
-        .catch(error => {
-          console.log("error", error);
-        });
-    },
     confChanged(key, subkey, newvalue) {
       console.log(key + "/" + subkey + " changed!");
       console.log(newvalue);
@@ -142,11 +132,12 @@ var Config = Vue.component('Config', {
         .post('/api/config/'+key+'/'+subkey, newvalue)
         .then(result => {
           console.log(result);
+          if (result.data.restart_required)
+            this.$toasted.show(this.$t('conf.conf_saved'));
         })
         .catch(error => {
           console.log("error", error);
         });
-
     },
     getPlayers () {
       const api_url = '/api/players';
-- 
2.34.1