Surfnet\StepupMiddlewareClientBundle\Identity\Service\AuditLogService

Complexity

Total Complexity

4

Size/Duplication

Total Lines	51
Duplicated Lines	100 %

Coupling/Cohesion

Components	1
Dependencies	4

Importance

Changes	2
Bugs	0	Features	0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	5	5	1
A	searchSecondFactorAuditLog()	20	20	3

<?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\StepupMiddlewareClientBundle\Identity\Service;

use Surfnet\StepupMiddlewareClient\Exception\AccessDeniedToResourceException;
use Surfnet\StepupMiddlewareClient\Exception\MalformedResponseException;
use Surfnet\StepupMiddlewareClient\Exception\ResourceReadException;
use Surfnet\StepupMiddlewareClient\Identity\Dto\SecondFactorAuditLogSearchQuery;
use Surfnet\StepupMiddlewareClient\Identity\Service\AuditLogService as LibraryAuditLogService;
use Surfnet\StepupMiddlewareClientBundle\Exception\InvalidResponseException;
use Surfnet\StepupMiddlewareClientBundle\Identity\Dto\AuditLog;
use Surfnet\StepupMiddlewareClientBundle\Identity\Dto\AuditLogCollection;
use Symfony\Component\Validator\Validator\ValidatorInterface;

class AuditLogService

This class seems to be duplicated in your project.

{
    /**
     * @var LibraryAuditLogService
     */
    private $service;

    /**
     * @var ValidatorInterface
     */
    private $validator;

    /**
     * @param LibraryAuditLogService $service
     * @param ValidatorInterface $validator
     */
    public function __construct(LibraryAuditLogService $service, ValidatorInterface $validator)
    {
        $this->service = $service;
        $this->validator = $validator;
    }

    /**
     * @param SecondFactorAuditLogSearchQuery $query
     * @return AuditLog

Should the return type not be AuditLog|null?

     * @throws AccessDeniedToResourceException When the consumer isn't authorised to access given resource.
     * @throws InvalidResponseException When the API responded with invalid data.
     * @throws ResourceReadException When the API doesn't respond with the resource.
     * @throws MalformedResponseException When the API doesn't respond with a proper response.
     */
    public function searchSecondFactorAuditLog(SecondFactorAuditLogSearchQuery $query)
    {
        $data = $this->service->searchSecondFactorAuditLog($query);

        if ($data === null) {
            return null;
        }

        $auditLog = AuditLog::fromData($data);
        $violations = $this->validator->validate($auditLog);

        if (count($violations) > 0) {
            throw InvalidResponseException::withViolations(
                "One or more audit log entries retrieved from the Middleware were invalid",

The string literal One or more audit log entries retrieved from the Middleware were invalid does not require double quotes, as per coding-style, please use single quotes.

                $violations
            );
        }

        return $auditLog;
    }
}