Skip to content

Commit

Permalink
Add trailing commas
Browse files Browse the repository at this point in the history
  • Loading branch information
tvdijen committed Aug 1, 2024
1 parent dac8b8c commit 72ee9ab
Show file tree
Hide file tree
Showing 4 changed files with 10 additions and 10 deletions.
4 changes: 2 additions & 2 deletions src/Auth/Process/ExpiryWarning.php
Original file line number Diff line number Diff line change
Expand Up @@ -45,7 +45,7 @@ public function __construct(array $config, $reserved)
if (array_key_exists('warndaysbefore', $config)) {
Assert::integerish(
$config['warndaysbefore'],
'Invalid value for \'warndaysbefore\'-option to authX509::ExpiryWarning filter.'
'Invalid value for \'warndaysbefore\'-option to authX509::ExpiryWarning filter.',
);
$this->warndaysbefore = (int)$config['warndaysbefore'];
}
Expand All @@ -54,7 +54,7 @@ public function __construct(array $config, $reserved)
$this->renewurl = $config['renewurl'];
Assert::string(
$this->renewurl,
'Invalid value for \'renewurl\'-option to authX509::ExpiryWarning filter.'
'Invalid value for \'renewurl\'-option to authX509::ExpiryWarning filter.',
);
}
}
Expand Down
4 changes: 2 additions & 2 deletions src/Auth/Source/X509userCert.php
Original file line number Diff line number Diff line change
Expand Up @@ -93,7 +93,7 @@ public function __construct(array $info, array &$config)
// Verify that the authsource config exists
if (!$authSources->hasValue($this->backend)) {
throw new Error\Exception(
sprintf('Authsource [%s] not found in authsources.php', $this->backend)
sprintf('Authsource [%s] not found in authsources.php', $this->backend),
);
}

Expand Down Expand Up @@ -234,7 +234,7 @@ public function authenticate(array &$state): void
if ($ldap_cert_data === $client_cert_data) {
$attributes = array_intersect_key(
$entry->getAttributes(),
array_fill_keys(array_values($this->ldapConfig->getArray('attributes')), null)
array_fill_keys(array_values($this->ldapConfig->getArray('attributes')), null),
);
$state['Attributes'] = $attributes;
$this->authSuccesful($state);
Expand Down
2 changes: 1 addition & 1 deletion src/Controller/ExpiryWarning.php
Original file line number Diff line number Diff line change
Expand Up @@ -43,7 +43,7 @@ class ExpiryWarning
*/
public function __construct(
protected Configuration $config,
protected Session $session
protected Session $session,
) {
}

Expand Down
10 changes: 5 additions & 5 deletions tests/src/Controller/AuthX509Test.php
Original file line number Diff line number Diff line change
Expand Up @@ -39,7 +39,7 @@ protected function setUp(): void
'module.enable' => ['authx509' => true],
],
'[ARRAY]',
'simplesaml'
'simplesaml',
);

$this->session = Session::getSessionFromRequest();
Expand All @@ -53,7 +53,7 @@ public function testNoStateId(): void
{
$request = Request::create(
'/expiryWarning',
'GET'
'GET',
);

$c = new Controller\ExpiryWarning($this->config, $this->session);
Expand All @@ -73,7 +73,7 @@ public function testNoState(): void
$request = Request::create(
'/expiryWarning',
'GET',
['StateId' => 'SomeStateId']
['StateId' => 'SomeStateId'],
);

$c = new Controller\ExpiryWarning($this->config, $this->session);
Expand All @@ -99,7 +99,7 @@ public function testProceed(): void
$request = Request::create(
'/expiryWarning',
'GET',
['StateId' => 'SomeStateId', 'proceed' => 'pleaseProceed']
['StateId' => 'SomeStateId', 'proceed' => 'pleaseProceed'],
);

$c = new Controller\ExpiryWarning($this->config, $this->session);
Expand All @@ -126,7 +126,7 @@ public function testNotProceed(): void
$request = Request::create(
'/expiryWarning',
'GET',
['StateId' => 'SomeStateId', 'proceed']
['StateId' => 'SomeStateId', 'proceed'],
);

$c = new Controller\ExpiryWarning($this->config, $this->session);
Expand Down

0 comments on commit 72ee9ab

Please sign in to comment.