Skip to content

Commit e97580b

Browse files
committed
fix: correctly increment order counter after user-provided order
When user provides a specific signing order, the next auto-assigned order should be the provided value + 1. Changed condition from > to >= and set currentOrder to userProvidedOrder + 1 instead of just userProvidedOrder. This ensures proper order sequencing when mixing user-provided and auto-assigned orders. Signed-off-by: Vitor Mattos <1079143+vitormattos@users.noreply.github.com>
1 parent 4a15f79 commit e97580b

1 file changed

Lines changed: 2 additions & 2 deletions

File tree

lib/Service/SequentialSigningService.php

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -54,8 +54,8 @@ public function determineSigningOrder(?int $userProvidedOrder): int {
5454
}
5555

5656
if ($userProvidedOrder !== null) {
57-
if ($userProvidedOrder > $this->currentOrder) {
58-
$this->currentOrder = $userProvidedOrder;
57+
if ($userProvidedOrder >= $this->currentOrder) {
58+
$this->currentOrder = $userProvidedOrder + 1;
5959
}
6060
return $userProvidedOrder;
6161
}

0 commit comments

Comments
 (0)