Skip to content

Commit a32d955

Browse files
author
lafricain79
committed
fix: treat zero and null as unlimited for max submissions
1 parent a1fde9f commit a32d955

1 file changed

Lines changed: 3 additions & 3 deletions

File tree

lib/Controller/ApiController.php

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1372,13 +1372,13 @@ public function newSubmission(int $formId, array $answers, string $shareHash = '
13721372
// Does the user have permissions to submit
13731373
// This is done right before insert so we minimize race conditions for submitting on unique-submission forms
13741374
if (!$this->formsService->canSubmit($form)) {
1375-
throw new OCSForbiddenException('Already submitted');
1375+
throw new OCSForbiddenException($this->l10n->t('Already submitted'));
13761376
}
13771377

13781378
// Check if max submissions limit is reached
13791379
$maxSubmissions = $form->getMaxSubmissions();
1380-
if ($maxSubmissions !== null && $this->submissionMapper->countSubmissions($formId) >= $maxSubmissions) {
1381-
throw new OCSForbiddenException('Maximum number of submissions reached');
1380+
if ($maxSubmissions > 0 && $this->submissionMapper->countSubmissions($formId) >= $maxSubmissions) {
1381+
throw new OCSForbiddenException($this->l10n->t('Maximum number of submissions reached'));
13821382
}
13831383
// Insert new submission
13841384
$this->submissionMapper->insert($submission);

0 commit comments

Comments
 (0)