[14.0][FIX] l10n_br_fiscal: FCP-ST tax domain and view adjustment

[DiegoParadeda]

Este PR corrige dois pontos do FCP-ST:

1. Remove campo legado de valor de FCP-ST, que antes ficava na seção do FCP próprio (note que o campo novo já existe na nova seção)
2. Utiliza regras de NCM, CEST, NBM no domínio do tax definition (semelhante ao ICMS-ST)

[OCA-git-bot]

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

[rvalyi]

LGTM

[rvalyi]

@DiegoParadeda @mileo @ygcarvalh eu gostaria porem de chamar a atenção de vcs num ponto. Eu até aprovei o PR da FCP ST e realmente eu acho que tem que fazer parte do modulo l10n_br_fiscal assim como o suporte que temos pros outros impostos do regime normal.

POREM, o PR adicionou umas 500 linhas no modulo l10n_br_fiscal. Eu espero que vcs entendem que a gente não pode simplesmente encher o modulo l10n_br_fiscal de novos PR com 500 linhas a vontade sem por outro lado fazer um trabalho de limpar a casa. Porque hoje o modulo l10n_br_fiscal bate no teto de mais de 20 000 linhas de codigo Python e XML, se eu contabilizar o XML dos data bate até em 30 000 linhas. Nenhum outro modulo da OCA tem uma obesidade morbida assim.

Nisso eu até prototipei de passar a tesoura no l10n_br_fiscal (na v16 mas validando isso na v14 para checar que não quebra nada) nesse outro PR #3010 Eu espero que vcs entendem que a contra partida de aceitar essas novas funcionalidades de imposto é tb de passar a tesoura em outras coisas para deixar as coisas sobre controle. Por examplo modulos como l10n_br_sale, l10n_br_purchase, l10n_br_sale_stock, l10n_br_purchase_stock, l10n_br_repair, l10n_br_contract etc (uns 15 modulos talvez) não precisam saber como a gente vai lidar com os eventos de NFe ou de outro documento fiscal. Na hora da migração mesma coisa: fatiar facilita muito as coisas: facilita de dizer esse "modulo central aqui é maduro, pode confiar, vai ser facil migrar etc" enquanto se o scope fica infinito sempre vai ter coisas que vai impactar sempre e nunca estabiliza... Eh a mesma logica do que agora o Marcel extraindo as funcionalidades de faturas de retenção para não engordar o l10n_br_account.

cc @marcelsavegnago @renatonlima @mbcosta @antoniospneto @felipemotter

[mileo]

/ocabot merge minor

[OCA-git-bot]

What a great day to merge this nice PR. Let's do it!
Prepared branch 14.0-ocabot-merge-pr-3011-by-mileo-bump-minor, awaiting test results.

[OCA-git-bot]

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