Skip to content

fix: fallback tpRetPisCofins when CSLL retention is zero - #92

Merged
vitormattos merged 2 commits into
mainfrom
fix/csll-retencao-fallback
Apr 3, 2026
Merged

fix: fallback tpRetPisCofins when CSLL retention is zero#92
vitormattos merged 2 commits into
mainfrom
fix/csll-retencao-fallback

Conversation

@vitormattos

Copy link
Copy Markdown
Member

Summary

  • add fallback in invoice payload assembly to avoid invalid retention type when CSLL retention value is zero
  • keep gateway-compatible behavior for PIS/COFINS/CSLL retention combinations
  • add regression coverage for the fallback scenario

Validation

  • docker compose exec akaunting.php sh -lc 'cd /var/www/html/modules/Nfse && vendor/bin/phpunit --testsuite Unit'
  • docker compose exec akaunting.php sh -lc 'cd /var/www/html/modules/Nfse && vendor/bin/php-cs-fixer fix --dry-run --diff'

Signed-off-by: Vitor Mattos <vitor@php.rio>
Signed-off-by: Vitor Mattos <vitor@php.rio>
@vitormattos
vitormattos merged commit 360fdf4 into main Apr 3, 2026
9 checks passed
@vitormattos
vitormattos deleted the fix/csll-retencao-fallback branch April 3, 2026 16:00
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.

1 participant