MusicService: Autostart music app on music start

When starting music on a paired device, the music service will automatically trigger the music app to open.
InfiniTimeOrg · Feb 7, 2024 · e9df27a · e9df27a
1 parent 44be356
commit e9df27a
Show file tree

Hide file tree

Showing 7 changed files with 26 additions and 3 deletions.
diff --git a/src/components/ble/MusicService.cpp b/src/components/ble/MusicService.cpp
@@ -17,6 +17,7 @@
 */
 #include "components/ble/MusicService.h"
 #include "components/ble/NimbleController.h"
+#include "systemtask/SystemTask.h"
 #include <cstring>
 
 namespace {
@@ -53,7 +54,8 @@ namespace {
   }
 }
 
-Pinetime::Controllers::MusicService::MusicService(Pinetime::Controllers::NimbleController& nimble) : nimble(nimble) {
+Pinetime::Controllers::MusicService::MusicService(Pinetime::System::SystemTask& systemTask, Pinetime::Controllers::NimbleController& nimble)
+  : systemTask {systemTask}, nimble(nimble) {
   characteristicDefinition[0] = {.uuid = &msEventCharUuid.u,
                                  .access_cb = MusicCallback,
                                  .arg = this,
@@ -152,6 +154,7 @@ int Pinetime::Controllers::MusicService::OnCommand(struct ble_gatt_access_ctxt*
       // These variables need to be updated, because the progress may not be updated immediately,
       // leading to getProgress() returning an incorrect position.
       if (playing) {
+        systemTask.PushMessage(Pinetime::System::Messages::OnMusicStarted);
         trackProgressUpdateTime = xTaskGetTickCount();
       } else {
         trackProgress +=

diff --git a/src/components/ble/MusicService.h b/src/components/ble/MusicService.h
@@ -27,12 +27,17 @@
 #undef min
 
 namespace Pinetime {
+
+  namespace System {
+    class SystemTask;
+  }
+
   namespace Controllers {
     class NimbleController;
 
     class MusicService {
     public:
-      explicit MusicService(NimbleController& nimble);
+      explicit MusicService(Pinetime::System::SystemTask& systemTask, NimbleController& nimble);
 
       void Init();
 
@@ -87,6 +92,7 @@ namespace Pinetime {
       bool repeat {false};
       bool shuffle {false};
 
+      Pinetime::System::SystemTask& systemTask;
       NimbleController& nimble;
     };
   }

diff --git a/src/components/ble/NimbleController.cpp b/src/components/ble/NimbleController.cpp
@@ -42,7 +42,7 @@ NimbleController::NimbleController(Pinetime::System::SystemTask& systemTask,
     anService {systemTask, notificationManager},
     alertNotificationClient {systemTask, notificationManager},
     currentTimeService {dateTimeController},
-    musicService {*this},
+    musicService {systemTask, *this},
     weatherService {dateTimeController},
     batteryInformationService {batteryController},
     immediateAlertService {systemTask, notificationManager},

diff --git a/src/displayapp/DisplayApp.cpp b/src/displayapp/DisplayApp.cpp
@@ -261,6 +261,15 @@ void DisplayApp::Refresh() {
       case Messages::NewNotification:
         LoadNewScreen(Apps::NotificationsPreview, DisplayApp::FullRefreshDirections::Down);
         break;
+      case Messages::MusicStarted:
+        if (state != States::Running) {
+          PushMessageToSystemTask(System::Messages::GoToRunning);
+        }
+        // Load music app if not loaded
+        if (currentApp != Apps::Music) {
+          LoadNewScreen(Apps::Music, DisplayApp::FullRefreshDirections::Up);
+        }
+        break;
       case Messages::TimerDone:
         if (state != States::Running) {
           PushMessageToSystemTask(System::Messages::GoToRunning);

diff --git a/src/displayapp/Messages.h b/src/displayapp/Messages.h
@@ -15,6 +15,7 @@ namespace Pinetime {
         ButtonLongerPressed,
         ButtonDoubleClicked,
         NewNotification,
+        MusicStarted,
         TimerDone,
         BleFirmwareUpdateStarted,
         DimScreen,

diff --git a/src/systemtask/Messages.h b/src/systemtask/Messages.h
@@ -24,6 +24,7 @@ namespace Pinetime {
       OnNewHalfHour,
       OnChargingEvent,
       OnPairing,
+      OnMusicStarted,
       SetOffAlarm,
       MeasureBatteryTimerExpired,
       BatteryPercentageUpdated,

diff --git a/src/systemtask/SystemTask.cpp b/src/systemtask/SystemTask.cpp
@@ -385,6 +385,9 @@ void SystemTask::Work() {
             nimbleController.DisableRadio();
           }
           break;
+        case Messages::OnMusicStarted:
+          displayApp.PushMessage(Pinetime::Applications::Display::Messages::MusicStarted);
+          break;
         default:
           break;
       }