Surfnet\Stepup\Configuration\Entity\RaLocation

Complexity

Total Complexity

10

Size/Duplication

Total Lines	115
Duplicated Lines	9.57 %

Coupling/Cohesion

Components	4
Dependencies	1

Importance

Changes	3
Bugs	0	Features	2

10 Methods

Rating	Name	Duplication	Size	Complexity
A	create()	0	8	1
A	__construct()	11	11	1
A	rename()	0	4	1
A	relocate()	0	4	1
A	changeContactInformation()	0	4	1
A	hasRaLocationId()	0	4	1
A	getRaLocationId()	0	4	1
A	getRaLocationName()	0	4	1
A	getLocation()	0	4	1
A	getContactInformation()	0	4	1

<?php

/**
 * Copyright 2016 SURFnet B.V.
 *
 * 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\Stepup\Configuration\Entity;

use Surfnet\Stepup\Configuration\Value\ContactInformation;
use Surfnet\Stepup\Configuration\Value\Location;
use Surfnet\Stepup\Configuration\Value\RaLocationId;
use Surfnet\Stepup\Configuration\Value\RaLocationName;

class RaLocation

Since you have declared the constructor as private, maybe you should also declare the class as final.

{
    /**
     * @var RaLocationId
     */
    private $raLocationId;

    /**
     * @var RaLocationName
     */
    private $raLocationName;

    /**
     * @var Location
     */
    private $location;

    /**
     * @var ContactInformation
     */
    private $contactInformation;

    /**
     * @param RaLocationId $raLocationId
     * @param RaLocationName $raLocationName
     * @param Location $location
     * @param ContactInformation $contactInformation
     * @return RaLocation
     */
    public static function create(
        RaLocationId $raLocationId,
        RaLocationName $raLocationName,
        Location $location,
        ContactInformation $contactInformation
    ) {
        return new self($raLocationId, $raLocationName, $location, $contactInformation);
    }

    private function __construct(

This method seems to be duplicated in your project.

        RaLocationId $raLocationId,
        RaLocationName $raLocationName,
        Location $location,
        ContactInformation $contactInformation
    ) {
        $this->raLocationId       = $raLocationId;
        $this->raLocationName     = $raLocationName;
        $this->location           = $location;
        $this->contactInformation = $contactInformation;
    }

    /**
     * @param RaLocationName $raLocationName
     */
    public function rename(RaLocationName $raLocationName)
    {
        $this->raLocationName = $raLocationName;
    }

    /**
     * @param Location $location
     */
    public function relocate(Location $location)
    {
        $this->location = $location;
    }

    /**
     * @param ContactInformation $contactInformation
     */
    public function changeContactInformation(ContactInformation $contactInformation)
    {
        $this->contactInformation = $contactInformation;
    }

    /**
     * @param RaLocationId $raLocationId
     * @return bool
     */
    public function hasRaLocationId(RaLocationId $raLocationId)
    {
        return $this->raLocationId->equals($raLocationId);
    }

    /**
     * @return RaLocationId
     */
    public function getRaLocationId()
    {
        return $this->raLocationId;
    }

    /**
     * @return RaLocationName
     */
    public function getRaLocationName()
    {
        return $this->raLocationName;
    }

    /**
     * @return Location
     */
    public function getLocation()
    {
        return $this->location;
    }

    /**
     * @return ContactInformation
     */
    public function getContactInformation()
    {
        return $this->contactInformation;
    }
}