Completed
Push — master ( 57e9fb...792895 )
by Dieter
11:00 queued 03:24
created

InformativeBestPricingWithoutPNR13::__construct()   A

Complexity

Conditions 2
Paths 2

Size

Total Lines 8

Duplication

Lines 0
Ratio 0 %

Code Coverage

Tests 6
CRAP Score 2

Importance

Changes 0
Metric Value
dl 0
loc 8
ccs 6
cts 6
cp 1
rs 10
c 0
b 0
f 0
cc 2
nc 2
nop 1
crap 2
1
<?php
2
/**
3
 * amadeus-ws-client
4
 *
5
 * Copyright 2015 Amadeus Benelux NV
6
 *
7
 * Licensed under the Apache License, Version 2.0 (the "License");
8
 * you may not use this file except in compliance with the License.
9
 * You may obtain a copy of the License at
10
 *
11
 * http://www.apache.org/licenses/LICENSE-2.0
12
 *
13
 * Unless required by applicable law or agreed to in writing, software
14
 * distributed under the License is distributed on an "AS IS" BASIS,
15
 * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
16
 * See the License for the specific language governing permissions and
17
 * limitations under the License.
18
 *
19
 * @package Amadeus
20
 * @license https://opensource.org/licenses/Apache-2.0 Apache 2.0
21
 */
22
23
namespace Amadeus\Client\Struct\Fare;
24
25
use Amadeus\Client\RequestOptions\Fare\InformativeBestPricingWithoutPnr\CabinOptions;
26
use Amadeus\Client\RequestOptions\FareInformativeBestPricingWithoutPnrOptions;
27
use Amadeus\Client\Struct\Fare\PricePnr13\PricingOptionGroup;
28
use Amadeus\Client\Struct\Fare\PricePnr13\PricingOptionKey;
29
30
/**
31
 * InformativeBestPricingWithoutPNR13
32
 *
33
 * @package Amadeus\Client\Struct\Fare
34
 * @author Dieter Devlieghere <[email protected]>
35
 */
36
class InformativeBestPricingWithoutPNR13 extends InformativePricingWithoutPNR13
37
{
38
    /**
39
     * InformativeBestPricingWithoutPNR13 constructor.
40
     *
41
     * @param FareInformativeBestPricingWithoutPnrOptions|null $options
42
     */
43 12
    public function __construct(FareInformativeBestPricingWithoutPnrOptions $options = null)
44
    {
45 12
        parent::__construct($options);
46
47 12
        if (!empty($options->cabin)) {
48 4
            $this->loadCabin($options->cabin);
49 2
        }
50 12
    }
51
52
    /**
53
     * @param CabinOptions $cabinOptions
54
     *
55
     * @return InformativeBestPricingWithoutPNR13
56
     */
57 4
    protected function loadCabin($cabinOptions)
58
    {
59 4
        if ($cabinOptions instanceof CabinOptions) {
60 4
            $optionDetails = [];
61 4
            foreach ($cabinOptions->getOptions() as $cabinOption) {
62 4
                $optionDetails[] = [$cabinOption->getType() =>$cabinOption->getClass()];
63 2
            }
64
65 4
            $this->pricingOptionGroup = array_merge(
66 4
                $this->pricingOptionGroup,
67
                [
68 4
                    new PricingOptionGroup(
69 4
                        PricingOptionKey::OPTION_CABIN,
70 2
                        $optionDetails
0 ignored issues
show
Documentation introduced by
$optionDetails is of type array, but the function expects a string|null.

It seems like the type of the argument is not accepted by the function/method which you are calling.

In some cases, in particular if PHP’s automatic type-juggling kicks in this might be fine. In other cases, however this might be a bug.

We suggest to add an explicit type cast like in the following example:

function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
Loading history...
71 2
                    )
72 2
                ]
73 2
            );
74 2
        }
75
76 4
        return $this;
77
    }
78
}
79