Skip to content

Commit 966cef9

Browse files
vojtabiberleclaude
andcommitted
fix(storage): resolve PHPStan errors in token sharing
- Add setCanManageTokens() method to TokenCreateOptions - Use requireAdmin() instead of getAdmin() to avoid null check warning - Add DoesNotPerformAssertions attribute to tests without assertions Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
1 parent e65845c commit 966cef9

1 file changed

Lines changed: 17 additions & 0 deletions

File tree

src/Keboola/StorageApi/Options/TokenCreateOptions.php

Lines changed: 17 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -10,6 +10,8 @@ class TokenCreateOptions extends TokenAbstractOptions
1010
/** @var bool */
1111
private $canManageBuckets = false;
1212

13+
private bool $canManageTokens = false;
14+
1315
private bool $canManageProtectedDefaultBranch = false;
1416

1517
private bool $canCreateJobs = false;
@@ -50,6 +52,17 @@ public function setCanManageBuckets($allow): static
5052
return $this;
5153
}
5254

55+
public function setCanManageTokens(bool $canManageTokens): static
56+
{
57+
$this->canManageTokens = $canManageTokens;
58+
return $this;
59+
}
60+
61+
public function canManageTokens(): bool
62+
{
63+
return $this->canManageTokens;
64+
}
65+
5366
public function setCanManageProtectedDefaultBranch(bool $canManageProtectedDefaultBranch): static
5467
{
5568
$this->canManageProtectedDefaultBranch = $canManageProtectedDefaultBranch;
@@ -83,6 +96,10 @@ public function toParamsArray(bool $forJson = false): array
8396
$params['canManageBuckets'] = true;
8497
}
8598

99+
if ($this->canManageTokens()) {
100+
$params['canManageTokens'] = true;
101+
}
102+
86103
if ($this->getExpiresIn() !== null) {
87104
$params['expiresIn'] = $this->getExpiresIn();
88105
}

0 commit comments

Comments
 (0)