Skip to content

Criar tagTransporta sem CPF ou CNPJ - #1366

Merged
robmachado merged 2 commits into
nfephp-org:masterfrom
guicalabria:patch-7
Sep 11, 2026
Merged

robmachado merged 2 commits into
nfephp-org:masterfrom
guicalabria:patch-7

Conversation

@guicalabria

Copy link
Copy Markdown
Contributor

Eu sei que você já comentou que a biblioteca não deve validar os dados, porém nesse caso para gerar a tagtransporta sem CNPJ ou CPF não tem como sem receber o erro
Warning: Undefined property: stdClass::$CNPJ in Parser.php on line 1441

Então vejo que nesse caso o correto seria adicionar ?? null; nas linhas do CPF/CNPJ

Eu sei que você já comentou que a biblioteca não deve validar os dados, porém nesse caso para gerar a tagtransporta sem CNPJ ou CPF não tem como sem receber o erro
<b>Warning</b>:  Undefined property: stdClass::$CNPJ in <b>Parser.php</b> on line <b>1441</b><br />
Então vejo que nesse caso o correto seria adicionar ?? null; nas linhas do CPF/CNPJ
@codecov

codecov Bot commented Aug 20, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 94.30%. Comparing base (92d26bf) to head (a3e6e7b).

Additional details and impacted files
@@            Coverage Diff            @@
##             master    #1366   +/-   ##
=========================================
  Coverage     94.30%   94.30%           
  Complexity     1715     1715           
=========================================
  Files            55       55           
  Lines         12271    12271           
=========================================
  Hits          11572    11572           
  Misses          699      699           

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@robmachado
robmachado merged commit 7f79886 into nfephp-org:master Sep 11, 2026
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants