Configuracao::carregaChavePublica() - Code Metrics - Inspection of "Adicionado exemplo de NFC-e funcional" - mazinsw/nfe-api - Measure and Improve Code Quality continuously with Scrutinizer

Test Failed

Push — master ( f8eb03...8feb3a )

by Francimar

created 2018-12-14 14:37 UTC

Configuracao::carregaChavePublica() A

↳ Parent: Configuracao

Complexity

Conditions	2
Paths	2

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	7
CRAP Score	2

Importance

Changes

Metric	Value
dl	0
loc	10
ccs	7
cts	7
cp	1
rs	9.9332
c	0
b	0
f	0
cc	2
nc	2
nop	0
crap	2

1 Method

Rating	Name	Duplication	Size	Complexity
A	Configuracao::isOffline()	0	4	1

<?php
/**
 * MIT License
 *
 * Copyright (c) 2016 MZ Desenvolvimento de Sistemas LTDA
 *
 * @author Francimar Alves <[email protected]>
 *
 * Permission is hereby granted, free of charge, to any person obtaining a copy
 * of this software and associated documentation files (the "Software"), to deal
 * in the Software without restriction, including without limitation the rights
 * to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
 * copies of the Software, and to permit persons to whom the Software is
 * furnished to do so, subject to the following conditions:
 *
 * The above copyright notice and this permission notice shall be included in all
 * copies or substantial portions of the Software.
 *
 * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
 * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
 * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
 * AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
 * LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
 * OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE
 * SOFTWARE.
 *
 */
namespace NFe\Common;

use NFe\Entity\Emitente;
use NFe\Database\Estatico;

/**
 * Fornece informações importante para a geração e envio das notas fiscais
 */
class Configuracao

{
    /**
     * @var \NFe\Database\Banco
     */
    private $banco;
    /**
     * @var \NFe\Entity\Emitente
     */
    private $emitente;
    /**
     * @var Evento
     */
    private $evento;
    /**
     * @var Certificado
     */
    private $certificado;
    /**
     * @var string
     */
    private $token;
    /**
     * @var string
     */
    private $csc;
    /**
     * @var string
     */
    private $token_ibpt;
    /**
     * @var int
     */
    private $tempo_limite;
    /**
     * @var string
     */
    private $sincrono;
    /**
     * @var int
     */
    private $offline;

    /**
     * @param mixed $configuracao array ou instância
     */
    public function __construct($configuracao = [])
    {
        $this->fromArray($configuracao);
    }

    /**
     * Banco que fornece informações sobre items da nota como: Códigos e Taxas
     * @return \NFe\Database\Banco
     */
    public function getBanco()
    {
        return $this->banco;
    }

    /**
     * Banco que fornece informações sobre items da nota como: Códigos e Taxas
     * @param \NFe\Database\Banco $banco
     * @return self
     */
    public function setBanco($banco)
    {
        $this->banco = $banco;
        return $this;
    }

    /**
     * Emitente da nota fiscal
     * @return \NFe\Entity\Emitente
     */
    public function getEmitente()
    {
        return $this->emitente;
    }

    /**
     * Emitente da nota fiscal
     * @param \NFe\Entity\Emitente $emitente
     * @return self
     */
    public function setEmitente($emitente)
    {
        $this->emitente = $emitente;
        return $this;
    }

    /**
     * Informa a instancia que receberá os eventos do processamento das notas
     * @return Evento
     */
    public function getEvento()
    {
        return $this->evento;
    }

    /**
     * Informa a instancia que receberá os eventos do processamento das notas
     * @param Evento $evento
     * @return self
     */
    public function setEvento($evento)
    {
        $this->evento = $evento;
        return $this;
    }

    /**
     * Certificado para assinar os XMLs
     * @return Certificado
     */
    public function getCertificado()
    {
        return $this->certificado;
    }

    /**
     * Informa o certificado para assinar os XMLs
     * @param Certificado $certificado
     * @return self
     */
    public function setCertificado($certificado)
    {
        $this->certificado = $certificado;
        return $this;
    }

    /**
     * Conteúdo da chave pública ou certificado no formato PEM
     * @return string
     * @deprecated Use getCertificado()->getChavePublica
     */
    public function getChavePublica()
    {
        return $this->getCertificado()->getChavePublica();
    }

    /**
     * Conteúdo da chave pública ou certificado no formato PEM
     * @param string $chave_publica
     * @return self
     * @deprecated Use getCertificado()->setChavePublica
     */
    public function setChavePublica($chave_publica)
    {
        $this->getCertificado()->setChavePublica($chave_publica);
        return $this;
    }

    /**
     * Conteúdo da chave privada do certificado no formato PEM
     * @return string
     * @deprecated Use getCertificado()->getChavePrivada
     */
    public function getChavePrivada()
    {
        return $this->getCertificado()->getChavePrivada();
    }

    /**
     * Conteúdo da chave privada do certificado no formato PEM
     * @param string $chave_privada
     * @return self
     * @deprecated Use getCertificado()->setChavePrivada
     */
    public function setChavePrivada($chave_privada)
    {
        $this->getCertificado()->setChavePrivada($chave_privada);
        return $this;
    }

    /**
     * Informa o caminho do arquivo da chave pública ou certificado no formato
     * PEM
     * @return string
     * @deprecated Use getCertificado()->getArquivoChavePublica
     */
    public function getArquivoChavePublica()
    {
        return $this->getCertificado()->getArquivoChavePublica();
    }

    /**
     * Informa o caminho do arquivo da chave pública ou certificado no formato
     * PEM
     * @param string $arquivo_chave_publica
     * @return self
     * @deprecated Use getCertificado()->setArquivoChavePublica
     */
    public function setArquivoChavePublica($arquivo_chave_publica)

    {
        $this->getCertificado()->setArquivoChavePublica($arquivo_chave_publica);
        return $this;
    }

    /**
     * Caminho do arquivo da chave privada do certificado no formato PEM
     * @return string
     * @deprecated Use getCertificado()->getArquivoChavePrivada
     */
    public function getArquivoChavePrivada()
    {
        return $this->getCertificado()->getArquivoChavePrivada();
    }

    /**
     * Altera o caminho do arquivo da chave privada do certificado no formato PEM
     * @param string $arquivo_chave_privada
     * @return self
     * @deprecated Use getCertificado()->setArquivoChavePrivada
     */
    public function setArquivoChavePrivada($arquivo_chave_privada)

    {
        $this->getCertificado()->setArquivoChavePrivada($arquivo_chave_privada);
        return $this;
    }

    /**
     * Data de expiração do certificado em timestamp
     * @return int
     * @deprecated Use getCertificado()->getExpiracao
     */
    public function getExpiracao()
    {
        return $this->getCertificado()->getExpiracao();
    }

    /**
     * Token do CSC
     * @return string
     */
    public function getToken()
    {
        return $this->token;
    }

    /**
     * Informa o token do CSC, geralmente 000001
     * @param string $token
     * @return self
     */
    public function setToken($token)
    {
        $this->token = $token;
        return $this;
    }

    /**
     * Código do contribuinte para emissão de nota fiscal
     * @return string
     */
    public function getCSC()
    {
        return $this->csc;
    }

    /**
     * Informa o código do contribuinte para emissão de nota fiscal
     * @param string $csc
     * @return self
     */
    public function setCSC($csc)
    {
        $this->csc = $csc;
        return $this;
    }

    /**
     * Token IBPT para consulta de impostos online
     */
    public function getTokenIBPT()
    {
        return $this->token_ibpt;
    }

    public function setTokenIBPT($token_ibpt)
    {
        $this->token_ibpt = $token_ibpt;
        return $this;
    }

    /**
     * Tempo limite em segundos nas conexões com os Web services, 0 para sem tempo limite
     */
    public function getTempoLimite()
    {
        return $this->tempo_limite;
    }

    public function setTempoLimite($tempo_limite)
    {
        $this->tempo_limite = intval($tempo_limite);
        return $this;
    }

    /**
     * Informa se o processo de autorização da nota é síncrono ou assíncrono
     */
    public function getSincrono($normalize = false)
    {
        if (!$normalize) {
            return $this->sincrono;
        }
        return $this->isSincrono()?'1':'0';
    }

    /**
     * Informa se o processo de autorização da nota é síncrono ou assíncrono
     */
    public function isSincrono()
    {
        return $this->sincrono == 'Y';
    }

    public function setSincrono($sincrono)
    {
        if (is_bool($sincrono)) {
            $sincrono = $sincrono ? 'Y': 'N';
        }
        $this->sincrono = $sincrono;
        return $this;
    }

    /**
     * Informa se está operando offline
     * @return mixed offline da Configuracao
     */
    public function getOffline($normalize = false)
    {
        if (!$normalize || is_null($this->offline)) {
            return $this->offline;
        }
        return Util::toDateTime($this->getOffline());
    }

    /**
     * Informa se está operando offline
     */
    public function isOffline()
    {
        return $this->offline + 180 > time();
    }

    /**
     * Entra no modo offline e sai automaticamente após 3 minutos
     */
    public function setOffline($offline)
    {
        if (!is_null($offline) && !is_numeric($offline)) {
            $offline = strtotime($offline);
        }
        $this->offline = $offline;
class Id
{
    public $id;

    public function __construct($id)
    {
        $this->id = $id;
    }

}

class Account
{
    /** @var  Id $id */
    public $id;
}

$account_id = false;

if (starsAreRight()) {
    $account_id = new Id(42);
}

$account = new Account();
if ($account instanceof Id)
{
    $account->id = $account_id;
}
        return $this;
    }

    public function toArray($recursive = false)
    {
        $configuracao = [];
        $configuracao['banco'] = $this->getBanco();
        $configuracao['emitente'] = $this->getEmitente();
        $configuracao['evento'] = $this->getEvento();
        $configuracao['certificado'] = $this->getCertificado();
        $configuracao['token'] = $this->getToken();
        $configuracao['csc'] = $this->getCSC();
        $configuracao['token_ibpt'] = $this->getTokenIBPT();
        $configuracao['tempo_limite'] = $this->getTempoLimite();
        $configuracao['sincrono'] = $this->getSincrono();
        $configuracao['offline'] = $this->getOffline($recursive);
        return $configuracao;
    }

    public function fromArray($configuracao = [])

    {
        if ($configuracao instanceof Configuracao) {
            $configuracao = $configuracao->toArray();
        } elseif (!is_array($configuracao)) {
            return $this;
        }
        $this->setBanco(new Estatico(isset($configuracao['banco']) ? $configuracao['banco'] : []));
        $this->setEmitente(new Emitente(isset($configuracao['emitente']) ? $configuracao['emitente'] : []));
        $this->setCertificado(new Certificado(isset($configuracao['certificado']) ? $configuracao['certificado'] : []));
        if (isset($configuracao['evento'])) {
            $this->setEvento($configuracao['evento']);
        } else {
            $this->setEvento(null);
function acceptsInteger($int) { }

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

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
        }
        if (isset($configuracao['token'])) {
            $this->setToken($configuracao['token']);
        } else {
            $this->setToken(null);
        }
        if (isset($configuracao['csc'])) {
            $this->setCSC($configuracao['csc']);
        } else {
            $this->setCSC(null);
        }
        if (isset($configuracao['token_ibpt'])) {
            $this->setTokenIBPT($configuracao['token_ibpt']);
        } else {
            $this->setTokenIBPT(null);
        }
        if (!isset($configuracao['tempo_limite'])) {
            $this->setTempoLimite(4);
        } else {
            $this->setTempoLimite($configuracao['tempo_limite']);
        }
        if (!isset($configuracao['sincrono'])) {
            $this->setSincrono('Y');
        } else {
            $this->setSincrono($configuracao['sincrono']);
        }
        return $this;
    }

    /**
     * Certifica que o certificado está informado e é válido
     * @throws \Exception quando o certificado estiver expirado ou não informado
     */
    public function verificaValidadeCertificado()
    {
        if (getenv('APP_ENV') != 'testing') {
            $this->getCertificado()->requerValido();
        }
    }
}


1		<?php
2		/**
3		* MIT License
4		*
5		* Copyright (c) 2016 MZ Desenvolvimento de Sistemas LTDA
6		*
7		* @author Francimar Alves <[email protected]>
8		*
9		* Permission is hereby granted, free of charge, to any person obtaining a copy
10		* of this software and associated documentation files (the "Software"), to deal
11		* in the Software without restriction, including without limitation the rights
12		* to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
13		* copies of the Software, and to permit persons to whom the Software is
14		* furnished to do so, subject to the following conditions:
15		*
16		* The above copyright notice and this permission notice shall be included in all
17		* copies or substantial portions of the Software.
18		*
19		* THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
20		* IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
21		* FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
22		* AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
23		* LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
24		* OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE
25		* SOFTWARE.
26		*
27		*/
28		namespace NFe\Common;
29
30		use NFe\Entity\Emitente;
31		use NFe\Database\Estatico;
32
33		/**
34		* Fornece informações importante para a geração e envio das notas fiscais
35		*/
36		class Configuracao
		0 ignored issues – show Complexity introduced 2018-12-14 14:43 UTC by Report Bug Copy Issue Report This class has a complexity of 55 which exceeds the configured maximum of 50. The class complexity is the sum of the complexity of all methods. A very high value is usually an indication that your class does not follow the single reponsibility principle and does more than one job. Some resources for further reading: Single Reponsibility Principle Open/Closed Principle You can also find more detailed suggestions for refactoring in the “Code” section of your repository. Loading history...
37		{
38		/**
39		* @var \NFe\Database\Banco
40		*/
41		private $banco;
42		/**
43		* @var \NFe\Entity\Emitente
44		*/
45		private $emitente;
46		/**
47		* @var Evento
48		*/
49		private $evento;
50		/**
51		* @var Certificado
52		*/
53		private $certificado;
54		/**
55		* @var string
56		*/
57		private $token;
58		/**
59		* @var string
60		*/
61		private $csc;
62		/**
63		* @var string
64		*/
65		private $token_ibpt;
66		/**
67		* @var int
68		*/
69		private $tempo_limite;
70		/**
71		* @var string
72		*/
73		private $sincrono;
74		/**
75		* @var int
76		*/
77		private $offline;
78
79		/**
80		* @param mixed $configuracao array ou instância
81		*/
82	102	public function __construct($configuracao = [])
83		{
84	102	$this->fromArray($configuracao);
85	102	}
86
87		/**
88		* Banco que fornece informações sobre items da nota como: Códigos e Taxas
89		* @return \NFe\Database\Banco
90		*/
91	72	public function getBanco()
92		{
93	72	return $this->banco;
94		}
95
96		/**
97		* Banco que fornece informações sobre items da nota como: Códigos e Taxas
98		* @param \NFe\Database\Banco $banco
99		* @return self
100		*/
101	102	public function setBanco($banco)
102		{
103	102	$this->banco = $banco;
104	102	return $this;
105		}
106
107		/**
108		* Emitente da nota fiscal
109		* @return \NFe\Entity\Emitente
110		*/
111	47	public function getEmitente()
112		{
113	47	return $this->emitente;
114		}
115
116		/**
117		* Emitente da nota fiscal
118		* @param \NFe\Entity\Emitente $emitente
119		* @return self
120		*/
121	102	public function setEmitente($emitente)
122		{
123	102	$this->emitente = $emitente;
124	102	return $this;
125		}
126
127		/**
128		* Informa a instancia que receberá os eventos do processamento das notas
129		* @return Evento
130		*/
131	9	public function getEvento()
132		{
133	9	return $this->evento;
134		}
135
136		/**
137		* Informa a instancia que receberá os eventos do processamento das notas
138		* @param Evento $evento
139		* @return self
140		*/
141	102	public function setEvento($evento)
142		{
143	102	$this->evento = $evento;
144	102	return $this;
145		}
146
147		/**
148		* Certificado para assinar os XMLs
149		* @return Certificado
150		*/
151	105	public function getCertificado()
152		{
153	105	return $this->certificado;
154		}
155
156		/**
157		* Informa o certificado para assinar os XMLs
158		* @param Certificado $certificado
159		* @return self
160		*/
161	102	public function setCertificado($certificado)
162		{
163	102	$this->certificado = $certificado;
164	102	return $this;
165		}
166
167		/**
168		* Conteúdo da chave pública ou certificado no formato PEM
169		* @return string
170		* @deprecated Use getCertificado()->getChavePublica
171		*/
172	34	public function getChavePublica()
173		{
174	34	return $this->getCertificado()->getChavePublica();
175		}
176
177		/**
178		* Conteúdo da chave pública ou certificado no formato PEM
179		* @param string $chave_publica
180		* @return self
181		* @deprecated Use getCertificado()->setChavePublica
182		*/
183	1	public function setChavePublica($chave_publica)
184		{
185	1	$this->getCertificado()->setChavePublica($chave_publica);
186	1	return $this;
187		}
188
189		/**
190		* Conteúdo da chave privada do certificado no formato PEM
191		* @return string
192		* @deprecated Use getCertificado()->getChavePrivada
193		*/
194	34	public function getChavePrivada()
195		{
196	34	return $this->getCertificado()->getChavePrivada();
197		}
198
199		/**
200		* Conteúdo da chave privada do certificado no formato PEM
201		* @param string $chave_privada
202		* @return self
203		* @deprecated Use getCertificado()->setChavePrivada
204		*/
205	1	public function setChavePrivada($chave_privada)
206		{
207	1	$this->getCertificado()->setChavePrivada($chave_privada);
208	1	return $this;
209		}
210
211		/**
212		* Informa o caminho do arquivo da chave pública ou certificado no formato
213		* PEM
214		* @return string
215		* @deprecated Use getCertificado()->getArquivoChavePublica
216		*/
217	22	public function getArquivoChavePublica()
218		{
219	22	return $this->getCertificado()->getArquivoChavePublica();
220		}
221
222		/**
223		* Informa o caminho do arquivo da chave pública ou certificado no formato
224		* PEM
225		* @param string $arquivo_chave_publica
226		* @return self
227		* @deprecated Use getCertificado()->setArquivoChavePublica
228		*/
229	102	public function setArquivoChavePublica($arquivo_chave_publica)
		0 ignored issues – show Comprehensibility Naming introduced 2018-12-14 14:43 UTC by Report Bug Copy Issue Report The variable name `$arquivo_chave_publica` exceeds the maximum configured length of `20`. Very long variable names usually make code harder to read. It is therefore recommended not to make variable names too verbose. Loading history...
230		{
231	102	$this->getCertificado()->setArquivoChavePublica($arquivo_chave_publica);
232	102	return $this;
233		}
234
235		/**
236		* Caminho do arquivo da chave privada do certificado no formato PEM
237		* @return string
238		* @deprecated Use getCertificado()->getArquivoChavePrivada
239		*/
240	22	public function getArquivoChavePrivada()
241		{
242	22	return $this->getCertificado()->getArquivoChavePrivada();
243		}
244
245		/**
246		* Altera o caminho do arquivo da chave privada do certificado no formato PEM
247		* @param string $arquivo_chave_privada
248		* @return self
249		* @deprecated Use getCertificado()->setArquivoChavePrivada
250		*/
251	102	public function setArquivoChavePrivada($arquivo_chave_privada)
		0 ignored issues – show Comprehensibility Naming introduced 2018-12-14 14:43 UTC by Report Bug Copy Issue Report The variable name `$arquivo_chave_privada` exceeds the maximum configured length of `20`. Very long variable names usually make code harder to read. It is therefore recommended not to make variable names too verbose. Loading history...
252		{
253	102	$this->getCertificado()->setArquivoChavePrivada($arquivo_chave_privada);
254	102	return $this;
255		}
256
257		/**
258		* Data de expiração do certificado em timestamp
259		* @return int
260		* @deprecated Use getCertificado()->getExpiracao
261		*/
262	1	public function getExpiracao()
263		{
264	1	return $this->getCertificado()->getExpiracao();
265		}
266
267		/**
268		* Token do CSC
269		* @return string
270		*/
271	31	public function getToken()
272		{
273	31	return $this->token;
274		}
275
276		/**
277		* Informa o token do CSC, geralmente 000001
278		* @param string $token
279		* @return self
280		*/
281	102	public function setToken($token)
282		{
283	102	$this->token = $token;
284	102	return $this;
285		}
286
287		/**
288		* Código do contribuinte para emissão de nota fiscal
289		* @return string
290		*/
291	31	public function getCSC()
292		{
293	31	return $this->csc;
294		}
295
296		/**
297		* Informa o código do contribuinte para emissão de nota fiscal
298		* @param string $csc
299		* @return self
300		*/
301	102	public function setCSC($csc)
302		{
303	102	$this->csc = $csc;
304	102	return $this;
305		}
306
307		/**
308		* Token IBPT para consulta de impostos online
309		*/
310	37	public function getTokenIBPT()
311		{
312	37	return $this->token_ibpt;
313		}
314
315	102	public function setTokenIBPT($token_ibpt)
316		{
317	102	$this->token_ibpt = $token_ibpt;
318	102	return $this;
319		}
320
321		/**
322		* Tempo limite em segundos nas conexões com os Web services, 0 para sem tempo limite
323		*/
324	24	public function getTempoLimite()
325		{
326	24	return $this->tempo_limite;
327		}
328
329	102	public function setTempoLimite($tempo_limite)
330		{
331	102	$this->tempo_limite = intval($tempo_limite);
332	102	return $this;
333		}
334
335		/**
336		* Informa se o processo de autorização da nota é síncrono ou assíncrono
337		*/
338	8	public function getSincrono($normalize = false)
339		{
340	8	if (!$normalize) {
341	3	return $this->sincrono;
342		}
343	6	return $this->isSincrono()?'1':'0';
344		}
345
346		/**
347		* Informa se o processo de autorização da nota é síncrono ou assíncrono
348		*/
349	6	public function isSincrono()
350		{
351	6	return $this->sincrono == 'Y';
352		}
353
354	102	public function setSincrono($sincrono)
355		{
356	102	if (is_bool($sincrono)) {
357	1	$sincrono = $sincrono ? 'Y': 'N';
358		}
359	102	$this->sincrono = $sincrono;
360	102	return $this;
361		}
362
363		/**
364		* Informa se está operando offline
365		* @return mixed offline da Configuracao
366		*/
367	3	public function getOffline($normalize = false)
368		{
369	3	if (!$normalize \|\| is_null($this->offline)) {
370	3	return $this->offline;
371		}
372	1	return Util::toDateTime($this->getOffline());
373		}
374
375		/**
376		* Informa se está operando offline
377		*/
378	23	public function isOffline()
379		{
380	23	return $this->offline + 180 > time();
381		}
382
383		/**
384		* Entra no modo offline e sai automaticamente após 3 minutos
385		*/
386	4	public function setOffline($offline)
387		{
388	4	if (!is_null($offline) && !is_numeric($offline)) {
389	1	$offline = strtotime($offline);
390		}
391	4	$this->offline = $offline;
		0 ignored issues – show Documentation Bug introduced 2018-12-14 14:43 UTC by Report Bug Copy Issue Report It seems like `$offline` can also be of type `double` or `string`. However, the property `$offline` is declared as type `integer`. Maybe add an additional type check? Our type inference engine has found a suspicous assignment of a value to a property. This check raises an issue when a value that can be of a mixed type is assigned to a property that is type hinted more strictly. For example, imagine you have a variable `$accountId` that can either hold an Id object or false (if there is no account id yet). Your code now assigns that value to the `id` property of an instance of the `Account` class. This class holds a proper account, so the id value must no longer be false. Either this assignment is in error or a type check should be added for that assignment. class Id { public $id; public function __construct($id) { $this->id = $id; } } class Account { /** @var Id $id */ public $id; } $account_id = false; if (starsAreRight()) { $account_id = new Id(42); } $account = new Account(); if ($account instanceof Id) { $account->id = $account_id; } Loading history...
392	4	return $this;
393		}
394
395	2	public function toArray($recursive = false)
396		{
397	2	$configuracao = [];
398	2	$configuracao['banco'] = $this->getBanco();
399	2	$configuracao['emitente'] = $this->getEmitente();
400	2	$configuracao['evento'] = $this->getEvento();
401	2	$configuracao['certificado'] = $this->getCertificado();
402	2	$configuracao['token'] = $this->getToken();
403	2	$configuracao['csc'] = $this->getCSC();
404	2	$configuracao['token_ibpt'] = $this->getTokenIBPT();
405	2	$configuracao['tempo_limite'] = $this->getTempoLimite();
406	2	$configuracao['sincrono'] = $this->getSincrono();
407	2	$configuracao['offline'] = $this->getOffline($recursive);
408	2	return $configuracao;
409		}
410
411	102	public function fromArray($configuracao = [])
		0 ignored issues – show Complexity introduced 2017-09-12 16:32 UTC by Report Bug Copy Issue Report This operation has 24000 execution paths which exceeds the configured maximum of 200. A high number of execution paths generally suggests many nested conditional statements and make the code less readible. This can usually be fixed by splitting the method into several smaller methods. You can also find more information in the “Code” section of your repository. Loading history...
412		{
413	102	if ($configuracao instanceof Configuracao) {
414	1	$configuracao = $configuracao->toArray();
415	102	} elseif (!is_array($configuracao)) {
416	1	return $this;
417		}
418	102	$this->setBanco(new Estatico(isset($configuracao['banco']) ? $configuracao['banco'] : []));
419	102	$this->setEmitente(new Emitente(isset($configuracao['emitente']) ? $configuracao['emitente'] : []));
420	102	$this->setCertificado(new Certificado(isset($configuracao['certificado']) ? $configuracao['certificado'] : []));
421	102	if (isset($configuracao['evento'])) {
422	2	$this->setEvento($configuracao['evento']);
423		} else {
424	102	$this->setEvento(null);
		0 ignored issues – show Documentation introduced 2018-12-14 14:43 UTC by Report Bug Copy Issue Report `null` is of type `null`, but the function expects a `object<NFe\Common\Evento>`. 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...
425		}
426	102	if (isset($configuracao['token'])) {
427	1	$this->setToken($configuracao['token']);
428		} else {
429	102	$this->setToken(null);
430		}
431	102	if (isset($configuracao['csc'])) {
432	1	$this->setCSC($configuracao['csc']);
433		} else {
434	102	$this->setCSC(null);
435		}
436	102	if (isset($configuracao['token_ibpt'])) {
437	1	$this->setTokenIBPT($configuracao['token_ibpt']);
438		} else {
439	102	$this->setTokenIBPT(null);
440		}
441	102	if (!isset($configuracao['tempo_limite'])) {
442	102	$this->setTempoLimite(4);
443		} else {
444	2	$this->setTempoLimite($configuracao['tempo_limite']);
445		}
446	102	if (!isset($configuracao['sincrono'])) {
447	102	$this->setSincrono('Y');
448		} else {
449	2	$this->setSincrono($configuracao['sincrono']);
450		}
451	102	return $this;
452		}
453
454		/**
455		* Certifica que o certificado está informado e é válido
456		* @throws \Exception quando o certificado estiver expirado ou não informado
457		*/
458	34	public function verificaValidadeCertificado()
459		{
460	34	if (getenv('APP_ENV') != 'testing') {
461		$this->getCertificado()->requerValido();
462		}
463	34	}
464		}
465

mazinsw / nfe-api

Push — master ( f8eb03...8feb3a )

Configuracao::carregaChavePublica() A

Complexity

Size

Duplication

Code Coverage

Importance

1 Method

Duplication Side-by-Side

Filter issues like