Improved how devices are created, ntfy and presence are now treated like any other device

src/devices/audio_setup.rs

@@ -5,7 +5,7 @@ use tracing::{debug, error, trace, warn};
 
 use super::Device;
 use crate::config::MqttDeviceConfig;
-use crate::device_manager::{DeviceConfig, WrappedDevice};
+use crate::device_manager::WrappedDevice;
 use crate::devices::As;
 use crate::error::DeviceConfigError;
 use crate::event::{OnMqtt, OnPresence};
@@ -13,6 +13,7 @@ use crate::messages::{RemoteAction, RemoteMessage};
 
 #[derive(Debug, Clone, LuaDeviceConfig)]
 pub struct AudioSetupConfig {
+    identifier: String,
     #[device_config(flatten)]
     mqtt: MqttDeviceConfig,
     #[device_config(from_lua)]
@@ -21,41 +22,33 @@ pub struct AudioSetupConfig {
     speakers: WrappedDevice,
 }
 
-#[async_trait]
-impl DeviceConfig for AudioSetupConfig {
-    async fn create(&self, identifier: &str) -> Result<Box<dyn Device>, DeviceConfigError> {
-        trace!(id = identifier, "Setting up AudioSetup");
-
-        let mixer_id = self.mixer.read().await.get_id().to_owned();
-        if !As::<dyn OnOff>::is(self.mixer.read().await.as_ref()) {
-            return Err(DeviceConfigError::MissingTrait(mixer_id, "OnOff".into()));
-        }
-
-        let speakers_id = self.speakers.read().await.get_id().to_owned();
-        if !As::<dyn OnOff>::is(self.speakers.read().await.as_ref()) {
-            return Err(DeviceConfigError::MissingTrait(speakers_id, "OnOff".into()));
-        }
-
-        let device = AudioSetup {
-            identifier: identifier.into(),
-            config: self.clone(),
-        };
-
-        Ok(Box::new(device))
-    }
-}
-
-// TODO: We need a better way to store the children devices
 #[derive(Debug, LuaDevice)]
 pub struct AudioSetup {
-    identifier: String,
     #[config]
     config: AudioSetupConfig,
 }
 
+impl AudioSetup {
+    async fn create(config: AudioSetupConfig) -> Result<Self, DeviceConfigError> {
+        trace!(id = config.identifier, "Setting up AudioSetup");
+
+        let mixer_id = config.mixer.read().await.get_id().to_owned();
+        if !As::<dyn OnOff>::is(config.mixer.read().await.as_ref()) {
+            return Err(DeviceConfigError::MissingTrait(mixer_id, "OnOff".into()));
+        }
+
+        let speakers_id = config.speakers.read().await.get_id().to_owned();
+        if !As::<dyn OnOff>::is(config.speakers.read().await.as_ref()) {
+            return Err(DeviceConfigError::MissingTrait(speakers_id, "OnOff".into()));
+        }
+
+        Ok(AudioSetup { config })
+    }
+}
+
 impl Device for AudioSetup {
-    fn get_id(&self) -> &str {
-        &self.identifier
+    fn get_id(&self) -> String {
+        self.config.identifier.clone()
     }
 }
 
@@ -69,7 +62,10 @@ impl OnMqtt for AudioSetup {
         let action = match RemoteMessage::try_from(message) {
             Ok(message) => message.action(),
             Err(err) => {
-                error!(id = self.identifier, "Failed to parse message: {err}");
+                error!(
+                    id = self.config.identifier,
+                    "Failed to parse message: {err}"
+                );
                 return;
             }
         };
@@ -118,7 +114,7 @@ impl OnPresence for AudioSetup {
         ) {
             // Turn off the audio setup when we leave the house
             if !presence {
-                debug!(id = self.identifier, "Turning devices off");
+                debug!(id = self.config.identifier, "Turning devices off");
                 speakers.set_on(false).await.unwrap();
                 mixer.set_on(false).await.unwrap();
             }