Skip to content

Commit db46961

Browse files
rcnsheppardptvdijen
authored andcommitted
Fix PHPdoc return tags and a code warning (#46)
* Fix PHPdoc return tags and a code warning - Added @return tags - Removed additional parameter in function call - if it was intended to be a default, or worked as one a long time ago, it would be caught by the Assert now * Revert removal and switch to getOptionalValueValidate()
1 parent 9dfd9a6 commit db46961

2 files changed

Lines changed: 5 additions & 5 deletions

File tree

src/MetaLoader.php

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -214,7 +214,7 @@ public function loadSource(array $source): void
214214
/**
215215
* @param \SimpleSAML\Metadata\SAMLParser $entity
216216
* @param array $source
217-
* @bool
217+
* @return bool
218218
*/
219219
private function processCertificates(Metadata\SAMLParser $entity, array $source): bool
220220
{
@@ -234,7 +234,7 @@ private function processCertificates(Metadata\SAMLParser $entity, array $source)
234234
/**
235235
* @param \SimpleSAML\Metadata\SAMLParser $entity
236236
* @param array $source
237-
* @bool
237+
* @return bool
238238
*/
239239
private function processBlacklist(Metadata\SAMLParser $entity, array $source): bool
240240
{
@@ -251,7 +251,7 @@ private function processBlacklist(Metadata\SAMLParser $entity, array $source): b
251251
/**
252252
* @param \SimpleSAML\Metadata\SAMLParser $entity
253253
* @param array $source
254-
* @bool
254+
* @return bool
255255
*/
256256
private function processWhitelist(Metadata\SAMLParser $entity, array $source): bool
257257
{
@@ -268,7 +268,7 @@ private function processWhitelist(Metadata\SAMLParser $entity, array $source): b
268268
/**
269269
* @param \SimpleSAML\Metadata\SAMLParser $entity
270270
* @param array $source
271-
* @bool
271+
* @return bool
272272
*/
273273
private function processAttributeWhitelist(Metadata\SAMLParser $entity, array $source): bool
274274
{

src/MetaRefresh.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -66,7 +66,7 @@ public function runRefresh(?string $crontag = null): void
6666
throw new Exception("Invalid outputDir specified.");
6767
}
6868

69-
$outputFormat = $set->getValueValidate('outputFormat', ['flatfile', 'serialize', 'pdo'], 'flatfile');
69+
$outputFormat = $set->getOptionalValueValidate('outputFormat', ['flatfile', 'serialize', 'pdo'], 'flatfile');
7070

7171
$oldMetadataSrc = MetaDataStorageSource::getSource([
7272
'type' => $outputFormat,

0 commit comments

Comments
 (0)