fix: export notifications

solspace · Dec 27, 2023 · 9cced52 · 9cced52
1 parent 84a84a7
commit 9cced52
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 5 deletions.
diff --git a/packages/plugin/src/Services/Pro/ExportNotificationsService.php b/packages/plugin/src/Services/Pro/ExportNotificationsService.php
@@ -41,10 +41,7 @@ public function getAllNamesById(): array
         ;
     }
 
-    /**
-     * @return null|ExportNotificationRecord
-     */
-    public function getById(int $id = null)
+    public function getById(int $id = null): ?ExportNotificationRecord
     {
         return ExportNotificationRecord::findOne(['id' => $id]);
     }

diff --git a/packages/plugin/src/controllers/export/NotificationsController.php b/packages/plugin/src/controllers/export/NotificationsController.php
@@ -60,7 +60,6 @@ public function actionSave(): Response
 
         $post = \Craft::$app->request->post();
         $post['recipients'] = StringHelper::extractSeparatedValues($post['recipients']);
-        $post['recipients'] = json_encode($post['recipients']);
 
         $id = \Craft::$app->request->post('notificationId');
         $record = $this->getNewOrExistingNotification($id);