Surfnet\StepupRa\RaBundle\Logger\ProcedureAwareLogger

Complexity

Total Complexity

14

Size/Duplication

Total Lines	93
Duplicated Lines	11.83 %

Coupling/Cohesion

Components	1
Dependencies	3

Importance

Changes

0

12 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	4	1
A	forProcedure()	11	11	2
A	emergency()	0	4	1
A	alert()	0	4	1
A	critical()	0	4	1
A	error()	0	4	1
A	warning()	0	4	1
A	notice()	0	4	1
A	info()	0	4	1
A	debug()	0	4	1
A	log()	0	4	1
A	enrichContext()	0	10	2

<?php

/**
 * Copyright 2014 SURFnet bv
 *
 * Licensed under the Apache License, Version 2.0 (the "License");
 * you may not use this file except in compliance with the License.
 * You may obtain a copy of the License at
 *
 *     http://www.apache.org/licenses/LICENSE-2.0
 *
 * Unless required by applicable law or agreed to in writing, software
 * distributed under the License is distributed on an "AS IS" BASIS,
 * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
 * See the License for the specific language governing permissions and
 * limitations under the License.
 */

namespace Surfnet\StepupRa\RaBundle\Logger;

use Psr\Log\LoggerInterface;
use Surfnet\StepupRa\RaBundle\Exception\InvalidArgumentException;
use Surfnet\StepupRa\RaBundle\Exception\RuntimeException;

/**
 * @SuppressWarnings(PHPMD.TooManyPublicMethods)
 */
final class ProcedureAwareLogger implements LoggerInterface
{
    /**
     * @var string|null
     */
    private $procedure;

    /**
     * @var LoggerInterface
     */
    private $logger;

    public function __construct(LoggerInterface $logger)
    {
        $this->logger = $logger;
    }

    public function forProcedure($procedure)

This method seems to be duplicated in your project.

    {
        if (!is_string($procedure)) {
            throw InvalidArgumentException::invalidType('string', 'procedure', $procedure);
        }

        $logger            = new self($this->logger);
        $logger->procedure = $procedure;

        return $logger;
    }

    public function emergency($message, array $context = [])
    {
        $this->logger->emergency($message, $this->enrichContext($context));
    }

    public function alert($message, array $context = [])
    {
        $this->logger->alert($message, $this->enrichContext($context));
    }

    public function critical($message, array $context = [])
    {
        $this->logger->critical($message, $this->enrichContext($context));
    }

    public function error($message, array $context = [])
    {
        $this->logger->error($message, $this->enrichContext($context));
    }

    public function warning($message, array $context = [])
    {
        $this->logger->warning($message, $this->enrichContext($context));
    }

    public function notice($message, array $context = [])
    {
        $this->logger->notice($message, $this->enrichContext($context));
    }

    public function info($message, array $context = [])
    {
        $this->logger->info($message, $this->enrichContext($context));
    }

    public function debug($message, array $context = [])
    {
        $this->logger->debug($message, $this->enrichContext($context));
    }

    public function log($level, $message, array $context = [])
    {
        $this->logger->log($message, $this->enrichContext($context));

$this->enrichContext($context) is of type array<string,string,{"procedure":"string"}>, but the function expects a string.

    }


    /**
     * Adds the procedure to the log context.
     *
     * @param array $context
     * @return array
     * @throws RuntimeException
     */
    private function enrichContext(array $context)
    {
        if (!$this->procedure) {
            throw new RuntimeException('Authentication logging context is unknown');
        }

        $context['procedure'] = $this->procedure;

        return $context;
    }
}