Skip to content

Commit 7e91e5d

Browse files
authored
Merge pull request #25 from web-token/Spomky-patch-1
Method "addDefaultSources" refactored
2 parents 43e85cf + 83fc437 commit 7e91e5d

File tree

1 file changed

+24
-32
lines changed

1 file changed

+24
-32
lines changed

src/Bundle/JoseFramework/DependencyInjection/JoseFrameworkExtension.php

Lines changed: 24 additions & 32 deletions
Original file line numberDiff line numberDiff line change
@@ -122,41 +122,33 @@ private function addDefaultSources()
122122
if (class_exists(JKUSource::class) && class_exists(HttplugBundle::class)) {
123123
$this->addSource(new JKUSource());
124124
}
125-
if (class_exists(JWKSource::class)) {
126-
$this->addSource(new JWKSource());
127-
}
128-
if (class_exists(JWKUriSource::class)) {
129-
$this->addSource(new JWKUriSource());
130-
}
131-
if (class_exists(JWKSetSource::class)) {
132-
$this->addSource(new JWKSetSource());
133-
}
134-
if (class_exists(ClaimChecker::class)) {
135-
$this->addSource(new ClaimChecker());
136-
}
137-
if (class_exists(HeaderChecker::class)) {
138-
$this->addSource(new HeaderChecker());
139-
}
140-
if (class_exists(JWSBuilder::class)) {
141-
$this->addSource(new JWSBuilder());
142-
}
143-
if (class_exists(JWSSerializer::class)) {
144-
$this->addSource(new JWSSerializer());
145-
}
146-
if (class_exists(JWSVerifier::class)) {
147-
$this->addSource(new JWSVerifier());
148-
}
149-
if (class_exists(JWEBuilder::class)) {
150-
$this->addSource(new JWEBuilder());
151-
}
152-
if (class_exists(JWEDecrypter::class)) {
153-
$this->addSource(new JWEDecrypter());
154-
}
155-
if (class_exists(JWESerializer::class)) {
156-
$this->addSource(new JWESerializer());
125+
foreach ($this->getSourceClasses() as $class) {
126+
if (class_exists($class)) {
127+
$this->addSource(new $class());
128+
}
157129
}
158130
}
159131

132+
/**
133+
* @return string[]
134+
*/
135+
private function getSourceClasses(): array
136+
{
137+
return [
138+
JWKSource::class,
139+
JWKUriSource::class,
140+
JWKSetSource::class,
141+
ClaimChecker::class,
142+
HeaderChecker::class,
143+
JWSBuilder::class,
144+
JWSSerializer::class,
145+
JWSVerifier::class,
146+
JWEBuilder::class,
147+
JWEDecrypter::class,
148+
JWESerializer::class,
149+
];
150+
}
151+
160152
/**
161153
* {@inheritdoc}
162154
*/

0 commit comments

Comments
 (0)