Completed
Push — master ( 01435f...836936 )
by Roberto
05:28
created

EvtExpRisco::__construct()   A

Complexity

Conditions 1
Paths 1

Size

Total Lines 7
Code Lines 5

Duplication

Lines 0
Ratio 0 %

Code Coverage

Tests 0
CRAP Score 2

Importance

Changes 0
Metric Value
dl 0
loc 7
ccs 0
cts 7
cp 0
rs 9.4285
c 0
b 0
f 0
cc 1
eloc 5
nc 1
nop 3
crap 2
1
<?php
2
3
namespace NFePHP\eSocial\Factories;
4
5
/**
6
 * Class eSocial EvtExpRisco Event S-2240 constructor
7
 *
8
 * @category  NFePHP
9
 * @package   NFePHPSocial
10
 * @copyright NFePHP Copyright (c) 2017
11
 * @license   http://www.gnu.org/licenses/lgpl.txt LGPLv3+
12
 * @license   https://opensource.org/licenses/MIT MIT
13
 * @license   http://www.gnu.org/licenses/gpl.txt GPLv3+
14
 * @author    Roberto L. Machado <linux.rlm at gmail dot com>
15
 * @link      http://github.com/nfephp-org/sped-esocial for the canonical source repository
16
 */
17
18
use NFePHP\Common\Certificate;
19
use NFePHP\eSocial\Common\Factory;
20
use NFePHP\eSocial\Common\FactoryId;
21
use NFePHP\eSocial\Common\FactoryInterface;
22
use stdClass;
23
24
class EvtExpRisco extends Factory implements FactoryInterface
25
{
26
    /**
27
     * @var int
28
     */
29
    public $sequencial;
30
31
    /**
32
     * @var string
33
     */
34
    protected $evtName = 'evtExpRisco';
35
36
    /**
37
     * @var string
38
     */
39
    protected $evtAlias = 'S-2240';
40
41
    /**
42
     * Parameters patterns
43
     *
44
     * @var array
45
     */
46
    protected $parameters = [];
47
48
    /**
49
     * Constructor
50
     *
51
     * @param string $config
52
     * @param stdClass $std
53
     * @param Certificate $certificate
54
     */
55
    public function __construct(
56
        $config,
57
        stdClass $std,
58
        Certificate $certificate
59
    ) {
60
        parent::__construct($config, $std, $certificate);
61
    }
62
63
    /**
64
     * Node constructor
65
     */
66
    protected function toNode()
67
    {
68
        $ideEmpregador = $this->node->getElementsByTagName('ideEmpregador')->item(0);
69
        //o idEvento pode variar de evento para evento
70
        //então cada factory individualmente terá de construir o seu
71
        $ideEvento = $this->dom->createElement("ideEvento");
72
        $this->dom->addChild(
73
            $ideEvento,
74
            "indRetif",
75
            $this->std->indretif,
76
            true
77
        );
78
        $this->dom->addChild(
79
            $ideEvento,
80
            "nrRecibo",
81
            !empty($this->std->nrrecibo) ? $this->std->nrrecibo : null,
82
            false
83
        );
84
        $this->dom->addChild(
85
            $ideEvento,
86
            "tpAmb",
87
            $this->tpAmb,
88
            true
89
        );
90
        $this->dom->addChild(
91
            $ideEvento,
92
            "procEmi",
93
            $this->procEmi,
94
            true
95
        );
96
        $this->dom->addChild(
97
            $ideEvento,
98
            "verProc",
99
            $this->verProc,
100
            true
101
        );
102
        $this->node->insertBefore($ideEvento, $ideEmpregador);
103
        
104
        $ide = $this->dom->createElement("ideVinculo");
105
        $this->dom->addChild(
106
            $ide,
107
            "cpfTrab",
108
            $this->std->cpftrab,
109
            true
110
        );
111
        $this->dom->addChild(
112
            $ide,
113
            "nisTrab",
114
            !empty($this->std->nistrab) ? $this->std->nistrab : null,
115
            false
116
        );
117
        $this->dom->addChild(
118
            $ide,
119
            "matricula",
120
            !empty($this->std->matricula) ? $this->std->matricula : null,
121
            false
122
        );
123
        $this->node->appendChild($ide);
124
        
125
        switch ($this->std->modo) {
126
            case 'INI':
127
                $noderisco = $this->dom->createElement("iniExpRisco");
128
                $dtnode = 'dtIniCondicao';
129
                break;
130
            case 'ALT':
131
                $noderisco = $this->dom->createElement("altExpRisco");
132
                $dtnode = 'dtAltCondicao';
133
                break;
134
            case 'FIM':
135
                $noderisco = $this->dom->createElement("fimExpRisco");
136
                $dtnode = 'dtFimCondicao';
137
                break;
138
        }
139
        $this->dom->addChild(
140
            $noderisco,
141
            $dtnode,
0 ignored issues
show
Bug introduced by
The variable $dtnode does not seem to be defined for all execution paths leading up to this point.

If you define a variable conditionally, it can happen that it is not defined for all execution paths.

Let’s take a look at an example:

function myFunction($a) {
    switch ($a) {
        case 'foo':
            $x = 1;
            break;

        case 'bar':
            $x = 2;
            break;
    }

    // $x is potentially undefined here.
    echo $x;
}

In the above example, the variable $x is defined if you pass “foo” or “bar” as argument for $a. However, since the switch statement has no default case statement, if you pass any other value, the variable $x would be undefined.

Available Fixes

  1. Check for existence of the variable explicitly:

    function myFunction($a) {
        switch ($a) {
            case 'foo':
                $x = 1;
                break;
    
            case 'bar':
                $x = 2;
                break;
        }
    
        if (isset($x)) { // Make sure it's always set.
            echo $x;
        }
    }
    
  2. Define a default value for the variable:

    function myFunction($a) {
        $x = ''; // Set a default which gets overridden for certain paths.
        switch ($a) {
            case 'foo':
                $x = 1;
                break;
    
            case 'bar':
                $x = 2;
                break;
        }
    
        echo $x;
    }
    
  3. Add a value for the missing path:

    function myFunction($a) {
        switch ($a) {
            case 'foo':
                $x = 1;
                break;
    
            case 'bar':
                $x = 2;
                break;
    
            // We add support for the missing case.
            default:
                $x = '';
                break;
        }
    
        echo $x;
    }
    
Loading history...
142
            $this->std->dtcondicao,
143
            true
144
        );
145
        if (!empty($this->std->codamb)) {
146
            foreach($this->std->codamb as $cod) {
147
                $infoAmb = $this->dom->createElement("infoAmb");
148
                $this->dom->addChild(
149
                    $infoAmb,
150
                    'codAmb',
151
                    $cod,
152
                    true
153
                );
154
                $noderisco->appendChild($infoAmb);
155
            }
156
        }
157
        
158
        $info = $this->dom->createElement("infoExpRisco");
159
        $info->appendChild($noderisco);
160
        
161
        $this->node->appendChild($info);
162
        $this->eSocial->appendChild($this->node);
163
        $this->xml = $this->dom->saveXML($this->eSocial);
164
        //$this->sign();
0 ignored issues
show
Unused Code Comprehensibility introduced by
84% of this comment could be valid code. Did you maybe forget this after debugging?

Sometimes obsolete code just ends up commented out instead of removed. In this case it is better to remove the code once you have checked you do not need it.

The code might also have been commented out for debugging purposes. In this case it is vital that someone uncomments it again or your project may behave in very unexpected ways in production.

This check looks for comments that seem to be mostly valid code and reports them.

Loading history...
165
    }
166
}
167