Amadeus\Client\Struct\Pnr\Split

Complexity

Total Complexity

4

Size/Duplication

Total Lines	27
Duplicated Lines	11.11 %

Coupling/Cohesion

Components	0
Dependencies	4

Test Coverage

Coverage

100%

Importance

Changes	1
Bugs	0	Features	0

1 Method

Rating	Name	Duplication	Size	Complexity
A	__construct()	3	9	4

<?php
/**
 * amadeus-ws-client
 *
 * Copyright 2015 Amadeus Benelux NV
 *
 * 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.
 *
 * @package Amadeus
 * @license https://opensource.org/licenses/Apache-2.0 Apache 2.0
 */

namespace Amadeus\Client\Struct\Pnr;

use Amadeus\Client\RequestOptions\PnrSplitOptions;
use Amadeus\Client\Struct\BaseWsMessage;
use Amadeus\Client\Struct\Pnr\Split\SplitDetails;

/**
 * Structure class for representing the PNR_Split request message
 *
 * @package Amadeus\Client\Struct\Pnr
 * @author Mike Hernas <m@hern.as>
 */
class Split extends BaseWsMessage
{
    /**
     * @var ReservationInfo
     */
    public $reservationInfo;

    /**
     * @var SplitDetails
     */
    public $splitDetails;
    
    /**
     * Split constructor.
     *
     * @param PnrSplitOptions $params
     */
    public function __construct(PnrSplitOptions $params)
    {
        if (is_string($params->recordLocator) && strlen($params->recordLocator) >= 6) {

This code seems to be duplicated across your project.

            $this->reservationInfo = new ReservationInfo($params->recordLocator);
        }
        if ($params->passengerTattoos) {

The expression $params->passengerTattoos of type array is implicitly converted to a boolean; are you sure this is intended? If so, consider using ! empty($expr) instead to make it clear that you intend to check for an array without elements.

            $this->splitDetails = new SplitDetails($params->passengerTattoos);
        }
    }
}