Skip to content

Commit 15fbb33

Browse files
committed
#11390 Allow nullable reviewer recommendation at save for later
#11390 Allow nullable reviewer recommendation at save for later
1 parent 6e009ce commit 15fbb33

File tree

2 files changed

+5
-2
lines changed

2 files changed

+5
-2
lines changed

classes/submission/reviewer/form/PKPReviewerReviewStep3Form.php

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -285,7 +285,10 @@ public function saveForLater()
285285

286286
// Persist the updated review assignment.
287287
Repo::reviewAssignment()->edit($reviewAssignment, [
288-
'reviewerRecommendationId' => (int) $this->getData('reviewerRecommendationId'), // save the recommendation to the review assignment
288+
// save the recommendation to the review assignment
289+
'reviewerRecommendationId' => (int)$this->getData('reviewerRecommendationId') === 0
290+
? null
291+
: (int)$this->getData('reviewerRecommendationId'),
289292
]);
290293

291294
return true;

js/controllers/form/reviewer/ReviewerReviewStep3FormHandler.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -66,7 +66,7 @@
6666
prototype.updateRecommendationRequired_ = function(submitButton, event) {
6767

6868
var $formElement = this.getHtmlElement(),
69-
$recommendationElement = $formElement.find('[id^="recommendation"]');
69+
$recommendationElement = $formElement.find('[id^="reviewerRecommendationId"]');
7070
if ($recommendationElement.length) {
7171
if (submitButton.id.includes('submitFormButton-')) {
7272
$recommendationElement.attr('required', '1');

0 commit comments

Comments
 (0)