pkp#10306 removed unnecessary request param from createNotification m…

…ethod
touhidurabir · Sep 15, 2024 · c82b9ad · c82b9ad
1 parent 2cad824
commit c82b9ad
Show file tree

Hide file tree

Showing 21 changed files with 5 additions and 29 deletions.
diff --git a/api/v1/submissions/PKPSubmissionController.php b/api/v1/submissions/PKPSubmissionController.php
@@ -1079,7 +1079,6 @@ public function versionPublication(Request $illuminateRequest): JsonResponse
         $notificationSubscriptionSettingsDao = DAORegistry::getDAO('NotificationSubscriptionSettingsDAO');
         foreach ($usersIterator as $user) {
             $notification = $notificationManager->createNotification(
-                $request,
                 $user->getId(),
                 Notification::NOTIFICATION_TYPE_SUBMISSION_NEW_VERSION,
                 $submission->getData('contextId'),

diff --git a/classes/context/SubEditorsDAO.php b/classes/context/SubEditorsDAO.php
@@ -233,7 +233,6 @@ public function assignEditors(Submission $submission, Context $context): Collect
         // Send a notification to assigned users
         foreach ($assignments as $assignment) {
             $notificationManager->createNotification(
-                Application::get()->getRequest(),
                 $assignment->userId,
                 Notification::NOTIFICATION_TYPE_SUBMISSION_SUBMITTED,
                 $submission->getData('contextId'),

diff --git a/classes/decision/DecisionType.php b/classes/decision/DecisionType.php
@@ -523,7 +523,6 @@ protected function createReviewRound(Submission $submission, int $stageId, ?int
         if ($count == 0) {
             $notificationMgr = new NotificationManager();
             $notificationMgr->createNotification(
-                Application::get()->getRequest(),
                 null,
                 Notification::NOTIFICATION_TYPE_REVIEW_ROUND_STATUS,
                 $submission->getData('contextId'),

diff --git a/classes/notification/NotificationManagerDelegate.php b/classes/notification/NotificationManagerDelegate.php
@@ -56,10 +56,10 @@ public function updateNotification(PKPRequest $request, ?array $userIds, int $as
      *
      * @copydoc PKPNotificationOperationManager::createNotification()
      */
-    public function createNotification(?PKPRequest $request = null, ?int $userId = null, ?int $notificationType = null, ?int $contextId = Application::SITE_CONTEXT_ID, ?int $assocType = null, ?int $assocId = null, int $level = Notification::NOTIFICATION_LEVEL_NORMAL, ?array $params = null): ?Notification
+    public function createNotification(?int $userId = null, ?int $notificationType = null, ?int $contextId = Application::SITE_CONTEXT_ID, ?int $assocType = null, ?int $assocId = null, int $level = Notification::NOTIFICATION_LEVEL_NORMAL, ?array $params = null): ?Notification
     {
         assert($notificationType == $this->getNotificationType() || $this->multipleTypesUpdate());
-        return parent::createNotification($request, $userId, $notificationType, $contextId, $assocType, $assocId, $level, $params);
+        return parent::createNotification($userId, $notificationType, $contextId, $assocType, $assocId, $level, $params);
     }
 
     /**

diff --git a/classes/notification/PKPNotificationOperationManager.php b/classes/notification/PKPNotificationOperationManager.php
@@ -140,7 +140,7 @@ public function getParamsForCurrentLocale(array $params): array
     /**
      * Create a new notification with the specified arguments and insert into DB
      */
-    public function createNotification(?PKPRequest $request = null, ?int $userId = null, ?int $notificationType = null, ?int $contextId = Application::SITE_CONTEXT_ID, ?int $assocType = null, ?int $assocId = null, int $level = Notification::NOTIFICATION_LEVEL_NORMAL, ?array $params = null): ?Notification
+    public function createNotification(?int $userId = null, ?int $notificationType = null, ?int $contextId = Application::SITE_CONTEXT_ID, ?int $assocType = null, ?int $assocId = null, int $level = Notification::NOTIFICATION_LEVEL_NORMAL, ?array $params = null): ?Notification
     {
         if ($userId && in_array($notificationType, $this->getUserBlockedNotifications($userId, $contextId))) {
             return null;
@@ -228,7 +228,7 @@ public function formatToGeneralNotification(PKPRequest $request, array $notifica
      */
     public function formatToInPlaceNotification(PKPRequest $request, array $notifications): array
     {
-        $formattedNotificationsData = null;
+        $formattedNotificationsData = [];
 
         $templateMgr = TemplateManager::getManager($request);
         foreach ($notifications as $notification) {

diff --git a/classes/notification/managerDelegate/AnnouncementNotificationManager.php b/classes/notification/managerDelegate/AnnouncementNotificationManager.php
@@ -96,7 +96,6 @@ public function getStyleClass($notification): string
     public function notify(User $user): ?Notification
     {
         return parent::createNotification(
-            Application::get()->getRequest(),
             $user->getId(),
             Notification::NOTIFICATION_TYPE_NEW_ANNOUNCEMENT,
             $this->_announcement->getAssocId(),

diff --git a/classes/notification/managerDelegate/EditorAssignmentNotificationManager.php b/classes/notification/managerDelegate/EditorAssignmentNotificationManager.php
@@ -91,7 +91,6 @@ public function updateNotification(PKPRequest $request, ?array $userIds, int $as
         } elseif (!$editorAssigned && !$notification) {
             // Create a notification.
             $this->createNotification(
-                $request,
                 null,
                 $this->getNotificationType(),
                 $context->getId(),

diff --git a/classes/notification/managerDelegate/EditorDecisionNotificationManager.php b/classes/notification/managerDelegate/EditorDecisionNotificationManager.php
@@ -93,7 +93,6 @@ public function updateNotification(PKPRequest $request, ?array $userIds, int $as
         // got here from the editor decision which sends its own email.
         foreach ((array) $userIds as $userId) {
             $this->createNotification(
-                $request,
                 $userId,
                 $this->getNotificationType(),
                 $request->getContext()->getId(),

diff --git a/classes/notification/managerDelegate/EditorialReportNotificationManager.php b/classes/notification/managerDelegate/EditorialReportNotificationManager.php
@@ -92,7 +92,6 @@ public function getStyleClass(Notification $notification): string
     public function notify(User $user): ?Notification
     {
         return parent::createNotification(
-            $this->_request,
             $user->getId(),
             Notification::NOTIFICATION_TYPE_EDITORIAL_REPORT,
             $this->_context->getId(),

diff --git a/classes/notification/managerDelegate/PKPApproveSubmissionNotificationManager.php b/classes/notification/managerDelegate/PKPApproveSubmissionNotificationManager.php
@@ -77,7 +77,6 @@ public function updateNotification(PKPRequest $request, ?array $userIds, int $as
             if (!$notification && $isPublished == $forPublicationState) {
                 // Create notification.
                 $this->createNotification(
-                    $request,
                     null,
                     $type,
                     $submission->getData('contextId'),

diff --git a/classes/notification/managerDelegate/PKPEditingProductionStatusNotificationManager.php b/classes/notification/managerDelegate/PKPEditingProductionStatusNotificationManager.php
@@ -131,7 +131,6 @@ public function updateNotification(PKPRequest $request, ?array $userIds, int $as
                                 if ($notificationType == Notification::NOTIFICATION_TYPE_AWAITING_REPRESENTATIONS) {
                                     // Add 'awaiting representations' notification
                                     $this->_createNotification(
-                                        $request,
                                         $submissionId,
                                         $editorStageAssignment->userId,
                                         $notificationType,
@@ -145,7 +144,6 @@ public function updateNotification(PKPRequest $request, ?array $userIds, int $as
                                 if ($notificationType == Notification::NOTIFICATION_TYPE_ASSIGN_PRODUCTIONUSER) {
                                     // Add 'assign a user' notification
                                     $this->_createNotification(
-                                        $request,
                                         $submissionId,
                                         $editorStageAssignment->userId,
                                         $notificationType,
@@ -170,7 +168,6 @@ public function updateNotification(PKPRequest $request, ?array $userIds, int $as
                             if ($notificationType == Notification::NOTIFICATION_TYPE_AWAITING_COPYEDITS) {
                                 // Add 'awaiting copyedits' notification
                                 $this->_createNotification(
-                                    $request,
                                     $submissionId,
                                     $editorStageAssignment->userId,
                                     $notificationType,
@@ -184,7 +181,6 @@ public function updateNotification(PKPRequest $request, ?array $userIds, int $as
                             if ($notificationType == Notification::NOTIFICATION_TYPE_ASSIGN_COPYEDITOR) {
                                 // Add 'assign a copyeditor' notification
                                 $this->_createNotification(
-                                    $request,
                                     $submissionId,
                                     $editorStageAssignment->userId,
                                     $notificationType,
@@ -219,7 +215,7 @@ public function _removeNotification(int $submissionId, int $userId, int $notific
     /**
      * Create a notification if none exists.
      */
-    public function _createNotification(PKPRequest $request, int $submissionId, int $userId, int $notificationType, ?int $contextId): void
+    public function _createNotification(int $submissionId, int $userId, int $notificationType, ?int $contextId): void
     {
         $notification = Notification::withAssoc(Application::ASSOC_TYPE_SUBMISSION, $submissionId)
             ->withUserId($userId)
@@ -229,7 +225,6 @@ public function _createNotification(PKPRequest $request, int $submissionId, int
         if (!$notification) {
             $notificationMgr = new NotificationManager();
             $notificationMgr->createNotification(
-                $request,
                 $userId,
                 $notificationType,
                 $contextId,

diff --git a/classes/observers/listeners/AssignEditors.php b/classes/observers/listeners/AssignEditors.php
@@ -69,7 +69,6 @@ public function handle(SubmissionSubmitted $event)
 
             // Send notification
             $notification = $notificationManager->createNotification(
-                Application::get()->getRequest(),
                 $manager->getId(),
                 Notification::NOTIFICATION_TYPE_EDITOR_ASSIGNMENT_REQUIRED,
                 $event->context->getId(),

diff --git a/classes/query/QueryDAO.php b/classes/query/QueryDAO.php
@@ -405,7 +405,6 @@ public function addQuery(int $submissionId, int $stageId, string $title, string
 
         foreach ($participantUserIds as $participantUserId) {
             $notificationMgr->createNotification(
-                Application::get()->getRequest(),
                 $participantUserId,
                 Notification::NOTIFICATION_TYPE_NEW_QUERY,
                 $contextId,

diff --git a/classes/submission/action/EditorAction.php b/classes/submission/action/EditorAction.php
@@ -106,7 +106,6 @@ public function addReviewer($request, $submission, $reviewerId, &$reviewRound, $
             // Add notification
             $notificationMgr = new NotificationManager();
             $notificationMgr->createNotification(
-                $request,
                 $reviewerId,
                 Notification::NOTIFICATION_TYPE_REVIEW_ASSIGNMENT,
                 $submission->getData('contextId'),

diff --git a/classes/submission/reviewer/form/PKPReviewerReviewStep3Form.php b/classes/submission/reviewer/form/PKPReviewerReviewStep3Form.php
@@ -188,7 +188,6 @@ public function execute(...$functionParams)
 
             // Notify editors
             $notification = $notificationMgr->createNotification(
-                Application::get()->getRequest(),
                 $userId,
                 Notification::NOTIFICATION_TYPE_REVIEWER_COMMENT,
                 $submission->getData('contextId'),

diff --git a/controllers/grid/queries/QueriesGridHandler.php b/controllers/grid/queries/QueriesGridHandler.php
@@ -657,7 +657,6 @@ public function updateQuery($args, $request)
                 $user = Repo::user()->get((int) $userId);
 
                 $notification = $notificationMgr->createNotification(
-                    $request,
                     $userId,
                     Notification::NOTIFICATION_TYPE_NEW_QUERY,
                     $request->getContext()->getId(),

diff --git a/controllers/grid/queries/QueryNotesGridHandler.php b/controllers/grid/queries/QueryNotesGridHandler.php
@@ -318,7 +318,6 @@ protected function insertedNoteNotify(Note $note): void
 
             // Notify the user of a new query.
             $notification = $notificationManager->createNotification(
-                $request,
                 $userId,
                 Notification::NOTIFICATION_TYPE_QUERY_ACTIVITY,
                 $request->getContext()->getId(),

diff --git a/controllers/grid/users/reviewer/form/EditReviewForm.php b/controllers/grid/users/reviewer/form/EditReviewForm.php
@@ -185,7 +185,6 @@ public function execute(...$functionArgs)
             $context = $request->getContext();
 
             $notification = $notificationManager->createNotification(
-                $request,
                 $reviewAssignment->getReviewerId(),
                 Notification::NOTIFICATION_TYPE_REVIEW_ASSIGNMENT_UPDATED,
                 $context->getId(),

diff --git a/controllers/grid/users/stageParticipant/form/PKPStageParticipantNotifyForm.php b/controllers/grid/users/stageParticipant/form/PKPStageParticipantNotifyForm.php
@@ -219,7 +219,6 @@ public function sendMessage(int $userId, Submission $submission, Request $reques
         // Send the email
         $notificationMgr = new NotificationManager();
         $notification = $notificationMgr->createNotification(
-            $request,
             $userId,
             Notification::NOTIFICATION_TYPE_NEW_QUERY,
             $request->getContext()->getId(),
@@ -349,7 +348,6 @@ private function _addAssignmentTaskNotification($request, $type, $userId, $submi
             $context = $request->getContext();
             $notificationMgr = new NotificationManager();
             $notificationMgr->createNotification(
-                $request,
                 $userId,
                 $type,
                 $context->getId(),

diff --git a/jobs/email/EditorialReminder.php b/jobs/email/EditorialReminder.php
@@ -142,7 +142,6 @@ public function handle(): void
 
         $notificationManager = new NotificationManager();
         $notification = $notificationManager->createNotification(
-            Application::get()->getRequest(),
             $this->editorId,
             Notification::NOTIFICATION_TYPE_EDITORIAL_REMINDER,
             $this->contextId

diff --git a/jobs/notifications/StatisticsReportMail.php b/jobs/notifications/StatisticsReportMail.php
@@ -84,7 +84,6 @@ public function handle()
 
             $notificationManager = new NotificationManager();
             $notification = $notificationManager->createNotification(
-                Application::get()->getRequest(),
                 $user->getId(),
                 Notification::NOTIFICATION_TYPE_EDITORIAL_REPORT,
                 $this->contextId