RecurringApplicationChargeManager::cancel()   A
last analyzed

Complexity

Conditions 1
Paths 1

Size

Total Lines 4

Duplication

Lines 0
Ratio 0 %

Importance

Changes 0
Metric Value
dl 0
loc 4
rs 10
c 0
b 0
f 0
cc 1
nc 1
nop 1
1
<?php
2
3
/*
4
 * This file is part of the slince/shopify-api-php
5
 *
6
 * (c) Slince <[email protected]>
7
 *
8
 * This source file is subject to the MIT license that is bundled
9
 * with this source code in the file LICENSE.
10
 */
11
12
namespace Slince\Shopify\Manager\RecurringApplicationCharge;
13
14
use Slince\Shopify\Common\Manager\GeneralCurdable;
15
16
class RecurringApplicationChargeManager extends GeneralCurdable implements RecurringApplicationChargeManagerInterface
17
{
18
    /**
19
     * {@inheritdoc}
20
     */
21
    public static function getServiceName()
22
    {
23
        return 'recurring_application_charges';
24
    }
25
26
    /**
27
     * {@inheritdoc}
28
     */
29
    public function getResourceName()
30
    {
31
        return 'recurring_application_charge';
32
    }
33
34
    /**
35
     * {@inheritdoc}
36
     */
37
    public function getModelClass()
38
    {
39
        return RecurringApplicationCharge::class;
40
    }
41
42
    /**
43
     * {@inheritdoc}
44
     */
45
    public function count(array $query = [])
46
    {
47
        throw new \Exception('The action is not supported');
48
    }
49
50
    /**
51
     * {@inheritdoc}
52
     */
53
    public function remove($id)
54
    {
55
        throw new \Exception('The action is not supported');
56
    }
57
58
    /**
59
     * {@inheritdoc}
60
     */
61
    public function cancel($id)
62
    {
63
        parent::remove($id);
0 ignored issues
show
Comprehensibility Bug introduced by
It seems like you call parent on a different method (remove() instead of cancel()). Are you sure this is correct? If so, you might want to change this to $this->remove().

This check looks for a call to a parent method whose name is different than the method from which it is called.

Consider the following code:

class Daddy
{
    protected function getFirstName()
    {
        return "Eidur";
    }

    protected function getSurName()
    {
        return "Gudjohnsen";
    }
}

class Son
{
    public function getFirstName()
    {
        return parent::getSurname();
    }
}

The getFirstName() method in the Son calls the wrong method in the parent class.

Loading history...
64
    }
65
66
    /**
67
     * {@inheritdoc}
68
     */
69
    public function activate($id)
70
    {
71
        $data = $this->client->post('recurring_application_charges/'.$id.'/activate', []);
72
73
        return $this->fromArray(reset($data));
74
    }
75
76
    /**
77
     * {@inheritdoc}
78
     */
79
    public function customize($id, $cappedAmount)
80
    {
81
        $data = $this->client->post('recurring_application_charges/'.$id.'/customize', [], [
82
            $this->getResourceName() => [
83
                'capped_amount' => $cappedAmount
84
            ]
85
        ]);
86
87
        return $this->fromArray(reset($data));
88
    }
89
}