Try using get_mut() to get mutable reference.

Author: sermuns

src/bluetooth.rs

@@ -44,10 +44,6 @@ impl Device {
         Ok(())
     }
 
-    pub fn toggle_favorite(&mut self) -> () {
-        self.is_favorite = !self.is_favorite;
-    }
-
     // https://specifications.freedesktop.org/icon-naming-spec/icon-naming-spec-latest.html
     pub fn get_icon(name: &str) -> Option<String> {
         match name {

src/handler.rs

@@ -550,10 +550,25 @@ pub async fn handle_key_events(
                                     if let Some(selected_controller) =
                                         app.controller_state.selected()
                                     {
-                                        let controller = &app.controllers[selected_controller];
+                                        let controller = app
+                                            .controllers
+                                            .get_mut(selected_controller)
+                                            .expect("Selected controller should be valid");
+
                                         if let Some(index) = app.paired_devices_state.selected() {
-                                            let device = &controller.paired_devices[index];
-                                            device.toggle_favorite();
+                                            match controller.paired_devices.get_mut(index) {
+                                                Some(device) => {
+                                                    device.is_favorite = !device.is_favorite;
+                                                }
+                                                None => {
+                                                    Notification::send(
+                                                        "Selected device should be valid"
+                                                            .to_string(),
+                                                        NotificationLevel::Error,
+                                                        sender.clone(),
+                                                    )?;
+                                                }
+                                            }
                                         }
                                     }
                                 }