pkp#8887 Adapt hook signature

ipula · Dec 4, 2023 · 5679008 · 5679008
1 parent a67e500
commit 5679008
Showing 1 changed file with 11 additions and 5 deletions.
diff --git a/classes/submission/reviewAssignment/Repository.php b/classes/submission/reviewAssignment/Repository.php
@@ -90,10 +90,10 @@ public function getSchemaMap(): maps\Schema
      * Perform validation checks on data used to add or edit a review assignment.
      *
      * @param array $props A key/value array with the new data to validate
-     * @param array $allowedLocales The context's supported locales
-     * @param string $primaryLocale The context's primary locale
      *
      * @return array A key/value array with validation errors. Empty if no errors
+     *
+     * @hook ReviewAssignment::validate [&$errors, $object, $props, $allowedLocales, $primaryLocale]
      */
     public function validate(?ReviewAssignment $object, array $props, Context $context): array
     {
@@ -118,7 +118,7 @@ public function validate(?ReviewAssignment $object, array $props, Context $conte
 
         // Check if submission exists
         if (isset($props['submissionId'])) {
-            $validator->after(function($validator) use ($props) {
+            $validator->after(function ($validator) use ($props) {
                 if (!$validator->errors()->get('submissionId')) {
                     $submission = Repo::submission()->get($props['submissionId']);
                     if (!$submission) {
@@ -130,7 +130,7 @@ public function validate(?ReviewAssignment $object, array $props, Context $conte
 
         // Check if reviewer exists
         if (isset($props['reviewerId'])) {
-            $validator->after(function($validator) use ($props, $context) {
+            $validator->after(function ($validator) use ($props, $context) {
                 if (!$validator->errors()->get('reviewerId')) {
                     $reviewer = Repo::user()->get($props['reviewerId']);
                     if (!$reviewer) {
@@ -154,13 +154,15 @@ public function validate(?ReviewAssignment $object, array $props, Context $conte
             $errors = $this->schemaService->formatValidationErrors($validator->errors());
         }
 
-        Hook::call('Category::validate', [&$errors, $object, $props, $allowedLocales, $primaryLocale]);
+        Hook::run('ReviewAssignment::validate', [&$errors, $object, $props, $allowedLocales, $primaryLocale]);
 
         return $errors;
     }
 
     /**
      * Add a new review assignment
+     *
+     * @hook ReviewAssignment::add [[$reviewAssignment]]
      */
     public function add(ReviewAssignment $reviewAssignment): int
     {
@@ -174,6 +176,8 @@ public function add(ReviewAssignment $reviewAssignment): int
 
     /**
      * Edit a review assignment
+     *
+     * @hook ReviewAssignment::edit [[$newReviewAssignment, $reviewAssignment, $params]]
      */
     public function edit(ReviewAssignment $reviewAssignment, array $params)
     {
@@ -189,6 +193,8 @@ public function edit(ReviewAssignment $reviewAssignment, array $params)
 
     /**
      * Delete a review assignment
+     *
+     * @hook ReviewAssignment::delete::before [[$reviewAssignment]]
      */
     public function delete(ReviewAssignment $reviewAssignment)
     {