Merge pull request #6 from tecnospeed/hotfix/types

Hotfix/types
tecnospeed · Mar 20, 2019 · dae6a64 · dae6a64
2 parents 4e2daab + 122cb31
commit dae6a64
Show file tree

Hide file tree

Showing 7 changed files with 52 additions and 11 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -1,6 +1,12 @@
 # Changelog
 Alterações na biblioteca
 
+## [1.4.1] - 20/03/2019
+### Corrigido
+
+Correção nos tipos float e boolean que não estavam fazendo o cast adequadamente.
+Também corrigido campos nulos que deveriam ser tratados antes de enviar para a API.
+
 ## [1.4.0] - 14/03/2019
 ### Adicionado
 

diff --git a/src/Abstracts/BuilderAbstract.php b/src/Abstracts/BuilderAbstract.php
@@ -9,7 +9,7 @@
 
 abstract class BuilderAbstract implements IBuilder
 {
-    public function toArray($excludeNull = false)
+    public function toArray($excludeNull = true)
     {
         return Extract::toArray($this, $excludeNull);
     }

diff --git a/src/Nfse/Servico/Iss.php b/src/Nfse/Servico/Iss.php
@@ -23,7 +23,7 @@ public function setAliquota($aliquota)
                 'É necessário informar um valor numérico para aliquota.'
             );
         }
-        $this->aliquota = $aliquota;
+        $this->aliquota = (float)$aliquota;
     }
 
     public function getAliquota()
@@ -63,7 +63,7 @@ public function setRetido($retido)
                 'Retido deve ser um valor booleano.'
             );
         }
-        $this->retido = $retido;
+        $this->retido = (bool)$retido;
     }
 
     public function getRetido()
@@ -93,7 +93,7 @@ public function setValor($valor)
                 'É necessário informar um valor numérico para o campo valor.'
             );
         }
-        $this->valor = $valor;
+        $this->valor = (float)$valor;
     }
 
     public function getValor()
@@ -108,7 +108,7 @@ public function setValorRetido($valorRetido)
                 'É necessário informar um valor numérico para o campo valorRetido.'
             );
         }
-        $this->valorRetido = $valorRetido;
+        $this->valorRetido = (float)$valorRetido;
     }
 
     public function getValorRetido()

diff --git a/src/Nfse/Servico/Valor.php b/src/Nfse/Servico/Valor.php
@@ -22,7 +22,7 @@ public function setBaseCalculo($baseCalculo)
                 'Base de cálculo deve ser um valor numérico.'
             );
         }
-        $this->baseCalculo = $baseCalculo;
+        $this->baseCalculo = (float)$baseCalculo;
     }
 
     public function getBaseCalculo()
@@ -37,7 +37,7 @@ public function setDeducoes($deducoes)
                 'Deduções deve ser um valor numérico.'
             );
         }
-        $this->deducoes = $deducoes;
+        $this->deducoes = (float)$deducoes;
     }
 
     public function getDeducoes()
@@ -52,7 +52,7 @@ public function setDescontoCondicionado($descontoCondicionado)
                 'Desconto condicional deve ser um valor numérico.'
             );
         }
-        $this->descontoCondicionado = $descontoCondicionado;
+        $this->descontoCondicionado = (float)$descontoCondicionado;
     }
 
     public function getDescontoCondicionado()
@@ -67,7 +67,7 @@ public function setDescontoIncondicionado($descontoIncondicionado)
                 'Desconto incondicional deve ser um valor numérico.'
             );
         }
-        $this->descontoIncondicionado = $descontoIncondicionado;
+        $this->descontoIncondicionado = (float)$descontoIncondicionado;
     }
 
     public function getDescontoIncondicionado()
@@ -82,7 +82,7 @@ public function setLiquido($liquido)
                 'Valor líquido deve ser um valor numérico.'
             );
         }
-        $this->liquido = $liquido;
+        $this->liquido = (float)$liquido;
     }
 
     public function getLiquido()
@@ -97,7 +97,7 @@ public function setServico($servico)
                 'O valor do serviço deve ser um valor numérico.'
             );
         }
-        $this->servico = $servico;
+        $this->servico = (float)$servico;
     }
 
     public function getServico()

diff --git a/tests/Nfse/PrestadorTest.php b/tests/Nfse/PrestadorTest.php
@@ -250,6 +250,7 @@ public function testValidateWithValidObject()
             'cpfCnpj' => '00.000.000/0001-91',
             'inscricaoMunicipal' => '123456',
             'razaoSocial' => 'Razao Social do Prestador',
+            'simplesNacional' => false,
             'endereco' => [
                 'logradouro' => 'Rua de Teste',
                 'numero' => '1234',
@@ -298,6 +299,7 @@ public function testSend()
         $prestador->setInscricaoMunicipal('123456');
         $prestador->setRazaoSocial('Razao Social do Prestador');
         $prestador->setEndereco($endereco);
+        $prestador->setSimplesNacional(false);
         $response = $prestador->send($configuration);
 
         $this->assertEquals(200, $response->statusCode);

diff --git a/tests/Nfse/Servico/IssTest.php b/tests/Nfse/Servico/IssTest.php
@@ -56,6 +56,21 @@ public function testInvalidValorRetido()
         $iss->setValorRetido('teste');
     }
 
+    public function testTypeConversion()
+    {
+        $iss = new Iss();
+        $iss->setAliquota('0.03');
+        $iss->setRetido('');
+        $iss->setValor('0.01');
+        $iss->setValorRetido('0.01');
+        $this->assertInternalType('double', $iss->getAliquota());
+        $this->assertInternalType('double', $iss->getAliquota());
+        $this->assertInternalType('bool', $iss->getRetido());
+        $this->assertFalse($iss->getRetido());
+        $this->assertInternalType('double', $iss->getValor());
+        $this->assertInternalType('double', $iss->getValorRetido());
+    }
+
     public function testValidCreation()
     {
         $iss = new Iss();

diff --git a/tests/Nfse/Servico/ValorTest.php b/tests/Nfse/Servico/ValorTest.php
@@ -57,6 +57,24 @@ public function testServicoWithInvalidValue()
         $valor->setServico('teste');
     }
 
+    public function testTypeConversion()
+    {
+        $valor = new Valor();
+        $valor->setBaseCalculo('0.01');
+        $valor->setDeducoes('0.02');
+        $valor->setDescontoCondicionado('0.03');
+        $valor->setDescontoIncondicionado('0.04');
+        $valor->setLiquido('0.05');
+        $valor->setServico('0.06');
+
+        $this->assertInternalType('double', $valor->getBaseCalculo());
+        $this->assertInternalType('double', $valor->getDeducoes());
+        $this->assertInternalType('double', $valor->getDescontoCondicionado());
+        $this->assertInternalType('double', $valor->getDescontoIncondicionado());
+        $this->assertInternalType('double', $valor->getLiquido());
+        $this->assertInternalType('double', $valor->getServico());
+    }
+
     public function testWithValidData()
     {
         $valor = new Valor();