[14.0] l10n_br_fiscal: Alteração do nome do método fiscal

[antoniospneto]

Na migração do módulo l10n_br_fiscal para a versão 16.0 o método _update_taxes foi renomeado para _update_fiscal_taxes

Esotu trazendo essa alteração para a 14.0 para diminuir o diff e também para evitar os conflitos nos fututos ports de commits e módulos que ainda não foram migrados.

[OCA-git-bot]

Hi @marcelsavegnago, @renatonlima, @rvalyi, some modules you are maintaining are being modified, check this out!

[rvalyi]

so para esclarecer eu mudei na v15 para _update_fiscal_taxes porque no módulo account nativo foi criado um metodo _update_taxes que passou a conflitar com o _update_taxes da localização.

Se a gente fazer backport ou nao disso eu diria que cabe a decisão a quem participa dos desenvolvimentos pesados e que usa a v14: para a gente tudo bem fazer o backport.

@marcelsavegnago vcs preferem fazer o backport ou para vcs é chato esse nivel de mudança na v14?

[antoniospneto]

@rvalyi eu tbm vou propor uma melhoria que vai ajudar bastante no desempenho do calculo dos impostos, que ai vai depender dessa alteração, é algo que vi estudando a migração do account para a 16, mas é bem interessante já aplicar na 14.0 daqui a pouco vou abrir a PR.

[rvalyi]

o impacto seria se vc tivesse modulos de customização que teria que adaptar, na localização apenas é tranquilo

[marcelsavegnago]

o impacto seria se vc tivesse modulos de customização que teria que adaptar, na localização apenas é tranquilo

sim... foi o que imaginei

@kaynnan se lembrar de algo relacionado me dá um toque

[marcelsavegnago]

o impacto seria se vc tivesse modulos de customização que teria que adaptar, na localização apenas é tranquilo

obrigado pelo toque

[rvalyi]

eh que tem backports de mudanças totalmente transparente como backport do lint, tem mudanças que alteram módulos mais periféricos e tem mudanças assim nas coisas mais centrais que é opcional fazer backport. Eu sou até a favor do backport para diminuir o diff tb, mas se a Engenere ou a Escodoo tb quiserem blocar esse tipo de backport eu acho legit tb.

[marcelsavegnago]

eh que tem backports de mudanças totalmente transparente como backport do lint, tem mudanças que alteram módulos mais periféricos e tem mudanças assim nas coisas mais centrais que é opcional fazer backport. Eu sou até a favor do backport para diminuir o diff tb, mas se a Engenere ou a Escodoo tb quiserem blocar esse tipo de backport eu acho legit tb.

por mim melhor diminuir o diff tbm

[rvalyi]

/ocabot merge minor (poderia ser merge patch mas como eh algo um pouco central e impactante vou botar minor para sinalizar)

[OCA-git-bot]

On my way to merge this fine PR! Prepared branch 14.0-ocabot-merge-pr-3264-by-rvalyi-bump-minor, awaiting test results.

[OCA-git-bot]

Congratulations, your PR was merged at 534f6e5a31418d73300971b2186219d4bb5f4eba. Thanks a lot for contributing to OCA. ❤️