Completed
Push — master ( 684883...0d5cb2 )
by Frederik
06:17 queued 04:16
created

Message::getAccount()   D

Complexity

Conditions 10
Paths 15

Size

Total Lines 39
Code Lines 20

Duplication

Lines 0
Ratio 0 %

Code Coverage

Tests 9
CRAP Score 10.6008

Importance

Changes 2
Bugs 0 Features 2
Metric Value
c 2
b 0
f 2
dl 0
loc 39
ccs 9
cts 11
cp 0.8182
rs 4.8196
cc 10
eloc 20
nc 15
nop 1
crap 10.6008

How to fix   Complexity   

Long Method

Small methods make your code easier to understand, in particular if combined with a good name. Besides, if your method is small, finding a good name is usually much easier.

For example, if you find yourself adding comments to a method's body, this is usually a good sign to extract the commented part to a new method, and use the comment as a starting point when coming up with a good name for this new method.

Commonly applied refactorings include:

1
<?php
2
3
namespace Genkgo\Camt\Camt052\Decoder;
4
5
use Genkgo\Camt\Decoder\Message as BaseMessageDecoder;
6
use Genkgo\Camt\Camt052\DTO as Camt052DTO;
7
use Genkgo\Camt\DTO;
8
use \SimpleXMLElement;
9
use \DateTimeImmutable;
10
use Genkgo\Camt\Iban;
11
12
abstract class Message extends BaseMessageDecoder
13
{
14
    /**
15
     * @param DTO\Message      $message
16
     * @param SimpleXMLElement $document
17
     */
18 4
    public function addGroupHeader(DTO\Message $message, SimpleXMLElement $document)
19
    {
20 4
        $xmlGroupHeader = $this->getRootElement($document)->GrpHdr;
0 ignored issues
show
Bug introduced by
The property GrpHdr does not seem to exist in SimpleXMLElement.

An attempt at access to an undefined property has been detected. This may either be a typographical error or the property has been renamed but there are still references to its old name.

If you really want to allow access to undefined properties, you can define magic methods to allow access. See the php core documentation on Overloading.

Loading history...
21 4
        $groupHeader = new DTO\GroupHeader(
22 4
            (string)$xmlGroupHeader->MsgId,
23 4
            new DateTimeImmutable((string)$xmlGroupHeader->CreDtTm)
24 4
        );
25
26 4
        $message->setGroupHeader($groupHeader);
27 4
    }
28
29
    /**
30
     * @param DTO\Message      $message
31
     * @param SimpleXMLElement $document
32
     */
33 4
    public function addRecords(DTO\Message $message, SimpleXMLElement $document)
34 3
    {
35 4
        $reports = [];
36
37 4
        $xmlReports = $this->getRootElement($document)->Rpt;
0 ignored issues
show
Bug introduced by
The property Rpt does not seem to exist in SimpleXMLElement.

An attempt at access to an undefined property has been detected. This may either be a typographical error or the property has been renamed but there are still references to its old name.

If you really want to allow access to undefined properties, you can define magic methods to allow access. See the php core documentation on Overloading.

Loading history...
38 4
        foreach ($xmlReports as $xmlReport) {
39 4
            $report = new Camt052DTO\Report(
40 4
                (string) $xmlReport->Id,
41 4
                new DateTimeImmutable((string)$xmlReport->CreDtTm),
42 4
                $this->getAccount($xmlReport)
0 ignored issues
show
Bug introduced by
It seems like $this->getAccount($xmlReport) targeting Genkgo\Camt\Camt052\Decoder\Message::getAccount() can also be of type null; however, Genkgo\Camt\DTO\Record::__construct() does only seem to accept object<Genkgo\Camt\DTO\Account>, maybe add an additional type check?

This check looks at variables that are passed out again to other methods.

If the outgoing method call has stricter type requirements than the method itself, an issue is raised.

An additional type check may prevent trouble.

Loading history...
43 4
            );
44
45 4
            $this->recordDecoder->addBalances($report, $xmlReport);
46 4
            $this->recordDecoder->addEntries($report, $xmlReport);
47
48 4
            $reports[] = $report;
49 4
        }
50
51 4
        $message->setRecords($reports);
52 4
    }
53
54
    /**
55
     * @param SimpleXMLElement $document
56
     *
57
     * @return SimpleXMLElement
58
     */
59
    abstract public function getRootElement(SimpleXMLElement $document);
60
61
    /**
62
     * @param SimpleXMLElement $xmlRecord
63
     *
64
     * @return DTO\Account
65
     */
66 4
    protected function getAccount(SimpleXMLElement $xmlRecord)
67
    {
68 4
        if (isset($xmlRecord->Acct->Id->IBAN)) {
69 3
            return new DTO\IbanAccount(new Iban((string) $xmlRecord->Acct->Id->IBAN));
70
        }
71
72 4
        if (isset($xmlRecord->Acct->Id->BBAN)) {
73
            return new Camt052DTO\BBANAccount((string) $xmlRecord->Acct->Id->BBAN);
74 3
        }
75
76 4
        if (isset($xmlRecord->Acct->Id->UPIC)) {
77 3
            return new Camt052DTO\UPICAccount((string) $xmlRecord->Acct->Id->UPIC);
78
        }
79
80 4
        if (isset($xmlRecord->Acct->Id->PrtryAcct)) {
81 4
            return new Camt052DTO\ProprietaryAccount((string) $xmlRecord->Acct->Id->PrtryAcct->Id);
82
        }
83
84
        if (isset($xmlRecord->Acct->Id->Othr)) {
85
            $xmlOtherIdentification = $xmlRecord->Acct->Id->Othr;
86
            $otherAccount = new DTO\OtherAccount((string) $xmlOtherIdentification->Id);
87
88
            if (isset($xmlOtherIdentification->SchmeNm)) {
89
                if (isset($xmlOtherIdentification->SchmeNm->Cd)) {
90
                    $otherAccount->setSchemeName((string) $xmlOtherIdentification->SchmeNm->Cd);
91
                }
92
93
                if (isset($otherIdentification->SchmeNm->Prtry)) {
0 ignored issues
show
Bug introduced by
The variable $otherIdentification does not exist. Did you mean $xmlOtherIdentification?

This check looks for variables that are accessed but have not been defined. It raises an issue if it finds another variable that has a similar name.

The variable may have been renamed without also renaming all references.

Loading history...
94
                    $otherAccount->setSchemeName((string) $xmlOtherIdentification->SchmeNm->Prtry);
95
                }
96
            }
97
98
            if (isset($xmlOtherIdentification->Issr)) {
99
                $otherAccount->setIssuer($xmlOtherIdentification->Issr);
100
            }
101
102
            return $otherAccount;
103
        }
104
    }
105
}
106