Interface fixed

Spomky-Labs · Jun 3, 2019 · b037d7f · b037d7f
1 parent ee44e52
commit b037d7f
Show file tree

Hide file tree

Showing 9 changed files with 34 additions and 20 deletions.
diff --git a/.php_cs.dist b/.php_cs.dist
@@ -24,6 +24,7 @@ return PhpCsFixer\Config::create()
         '@DoctrineAnnotation' => true,
         '@PHP70Migration' => true,
         '@PHP71Migration' => true,
+        'phpdoc_no_empty_return' => false,
         'strict_param' => true,
         'strict_comparison' => true,
         'array_syntax' => ['syntax' => 'short'],

diff --git a/Checker/AlgHeaderChecker.php b/Checker/AlgHeaderChecker.php
@@ -39,8 +39,10 @@ public function __construct(string $algorithm)
 
     /**
      * @param mixed $algorithm
+     *
+     * @return void
      */
-    public function checkHeader($algorithm): void
+    public function checkHeader($algorithm)
     {
         if (!\is_string($algorithm)) {
             throw new InvalidHeaderException('The value of the header "alg" is not valid', 'alg', $algorithm);

diff --git a/Checker/EncHeaderChecker.php b/Checker/EncHeaderChecker.php
@@ -39,8 +39,10 @@ public function __construct(string $algorithm)
 
     /**
      * @param mixed $algorithm
+     *
+     * @return void
      */
-    public function checkHeader($algorithm): void
+    public function checkHeader($algorithm)
     {
         if (!\is_string($algorithm)) {
             throw new InvalidHeaderException('The value of the header "enc" is not valid', 'enc', $algorithm);

diff --git a/Checker/IssuerChecker.php b/Checker/IssuerChecker.php
@@ -44,8 +44,10 @@ public function supportedClaim(): string
 
     /**
      * @param mixed $issuer
+     *
+     * @return void
      */
-    public function checkClaim($issuer): void
+    public function checkClaim($issuer)
     {
         if ($this->issuer !== $issuer) {
             throw new \Exception(sprintf('The issuer "%s" is not allowed.', $issuer));
@@ -54,8 +56,10 @@ public function checkClaim($issuer): void
 
     /**
      * @param mixed $value
+     *
+     * @return void
      */
-    public function checkHeader($value): void
+    public function checkHeader($value)
     {
         $this->checkClaim($value);
     }

diff --git a/DependencyInjection/Compiler/EncryptionSupportCompilerPass.php b/DependencyInjection/Compiler/EncryptionSupportCompilerPass.php
@@ -24,9 +24,9 @@
 final class EncryptionSupportCompilerPass implements CompilerPassInterface
 {
     /**
-     * {@inheritdoc}
+     * @return void
      */
-    public function process(ContainerBuilder $container): void
+    public function process(ContainerBuilder $container)
     {
         if (false === $container->hasDefinition(LexikJoseEncoder::class) || false === $container->getParameter('lexik_jose_bridge.encoder.encryption.enabled')) {
             return;

diff --git a/DependencyInjection/Configuration.php b/DependencyInjection/Configuration.php
@@ -76,7 +76,10 @@ public function getConfigTreeBuilder()
         return $treeBuilder;
     }
 
-    private function addEncryptionSection(NodeDefinition $node): void
+    /**
+     * @return void
+     */
+    private function addEncryptionSection(NodeDefinition $node)
     {
         $node
             ->addDefaultsIfNotSet()

diff --git a/DependencyInjection/SpomkyLabsLexikJoseExtension.php b/DependencyInjection/SpomkyLabsLexikJoseExtension.php
@@ -32,9 +32,9 @@ public function getAlias()
     }
 
     /**
-     * {@inheritdoc}
+     * @return void
      */
-    public function load(array $configs, ContainerBuilder $container): void
+    public function load(array $configs, ContainerBuilder $container)
     {
         $configuration = new Configuration();
         $config = $this->processConfiguration($configuration, $configs);
@@ -63,27 +63,27 @@ public function load(array $configs, ContainerBuilder $container): void
     }
 
     /**
-     * @param ContainerBuilder $container
+     * @return void
      */
-    public function loadServices(ContainerBuilder $container): void
+    public function loadServices(ContainerBuilder $container)
     {
         $loader = new Loader\YamlFileLoader($container, new FileLocator(__DIR__.'/../Resources/config'));
         $loader->load('services.yml');
     }
 
     /**
-     * @param ContainerBuilder $container
+     * @return void
      */
-    public function loadEncryptionServices(ContainerBuilder $container): void
+    public function loadEncryptionServices(ContainerBuilder $container)
     {
         $loader = new Loader\YamlFileLoader($container, new FileLocator(__DIR__.'/../Resources/config'));
         $loader->load('encryption_services.yml');
     }
 
     /**
-     * {@inheritdoc}
+     * @return void
      */
-    public function prepend(ContainerBuilder $container): void
+    public function prepend(ContainerBuilder $container)
     {
         $isDebug = $container->getParameter('kernel.debug');
         $bridgeConfig = current($container->getExtensionConfig($this->getAlias()));

diff --git a/Encoder/LexikJoseEncoder.php b/Encoder/LexikJoseEncoder.php
@@ -178,8 +178,10 @@ public function __construct(JWSBuilder $jwsBuilder,
      * @param int|string           $encryptionKeyIndex
      * @param string               $keyEncryptionAlgorithm
      * @param string               $contentEncryptionAlgorithm
+     *
+     * @return void
      */
-    public function enableEncryptionSupport(JWEBuilder $jweBuilder, JWEDecrypter $jweLoader, HeaderCheckerManager $encryptionHeaderCheckerManager, JWKSet $encryptionKeyset, $encryptionKeyIndex, string $keyEncryptionAlgorithm, string $contentEncryptionAlgorithm): void
+    public function enableEncryptionSupport(JWEBuilder $jweBuilder, JWEDecrypter $jweLoader, HeaderCheckerManager $encryptionHeaderCheckerManager, JWKSet $encryptionKeyset, $encryptionKeyIndex, string $keyEncryptionAlgorithm, string $contentEncryptionAlgorithm)
     {
         $this->jweBuilder = $jweBuilder;
         $this->jweLoader = $jweLoader;

diff --git a/SpomkyLabsLexikJoseBundle.php b/SpomkyLabsLexikJoseBundle.php
@@ -19,17 +19,17 @@
 final class SpomkyLabsLexikJoseBundle extends Bundle
 {
     /**
-     * {@inheritdoc}
+     * @return SpomkyLabsLexikJoseExtension
      */
-    public function getContainerExtension(): SpomkyLabsLexikJoseExtension
+    public function getContainerExtension()
     {
         return new SpomkyLabsLexikJoseExtension();
     }
 
     /**
-     * {@inheritdoc}
+     * @return void
      */
-    public function build(ContainerBuilder $container): void
+    public function build(ContainerBuilder $container)
     {
         parent::build($container);
         $container->addCompilerPass(new EncryptionSupportCompilerPass());