[14.0][ADD] l10n_br_fiscal_certificate

[rvalyi]

Como sugerido pelo @antoniospneto aqui #2529 (comment) eu extrai o modulo l10n_br_fiscal_certificate do modulo l10n_br_fiscal.

Pois tem algumas coisas a mais chegando no modulo fiscal como

• [14.0] Fiscal Tax Benefit #2545
• [14.0] l10n_br_account, l10n_br_fiscal: Possibilidade de informar os valores dos impostos manualmente. #2559

E como já temos mais de 10k linhas de código no modulo l10n_br_fiscal é bom extrair tudo que pode ser extraído (o que não é focado em documento fiscal e impostos) antes de adicionar mais coisas no modulo. No caso tira umas 400 linhas do modulo l10n_br_fiscal.

Alem disso varios módulos podem ser instalados apenas com o modulo l10n_br_fiscal e deixando a questão da assinatura dos documentos fiscais para mais tarde. O pessoal tem sempre reclamado de dificuldade de instalar a lib erpbrasil.assinatura numa distro especifica ou se ainda o maluco ta no windows. Mas é mais motivador deixar a pessoa descobrir o projeto aos poucos sem ficar blocado logo para instalar qualquer coisa.

Eh importante tb fazer esse refator antes de migrar o modulo para as v15 e v16 pois senão os cherry-pick não iriam funcionar e iria complicar as sinergias entre v14 e v16.

[OCA-git-bot]

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

[rvalyi]

Eu tive que remover a permissão de ler o certificado do usuario do account (pois o objetivo é não existir mais a dependencia do l10n_br_fiscal_certificate).

Ai eu não sei se teria que adicionar essa permissão pro usuario de nfe ou de nfse talvez? @renatonlima o que vc acha?
Sera se não seria melhor botar um sudo() na hora de ler o certificado? dar permissão de ler o certificado com a senha não criptografada nem é algo legal hoje... (se o usuario vai lé pelo JS/RPC ele sai com o certificado A1 e a senha da empresa...)
Eu acabei de fazer isso para a NFe no l10n_br_nfe. Se acharem OK podemos fazer o mesmo nos módulos de NFSe.

[rvalyi]

pessoal dei rebase e tá pronto para review

[marcelsavegnago]

LGTM

[rvalyi]

/ocabot merge nobump

[OCA-git-bot]

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

[OCA-git-bot]

@rvalyi your merge command was aborted due to failed check(s), which you can inspect on this commit of 14.0-ocabot-merge-pr-2574-by-rvalyi-bump-nobump.

After fixing the problem, you can re-issue a merge command. Please refrain from merging manually as it will most probably make the target branch red.

[rvalyi]

/ocabot merge nobump

[OCA-git-bot]

Hey, thanks for contributing! Proceeding to merge this for you.
Prepared branch 14.0-ocabot-merge-pr-2574-by-rvalyi-bump-nobump, awaiting test results.

[marcelsavegnago]

@rvalyi acho que faltou o rebase :)

[rvalyi]

/hadouken!
/ocabot merge nobump

[OCA-git-bot]

This PR looks fantastic, let's merge it!
Prepared branch 14.0-ocabot-merge-pr-2574-by-rvalyi-bump-nobump, awaiting test results.

[OCA-git-bot]

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

[antoniospneto]

o git bot fechando ao invés de marcar como concluída denovo 👀

[rvalyi]

o git bot fechando ao invés de marcar como concluída denovo eyes

o Gitbot deve estar com ciume dos milhões da Odoo SA e eu acho que ta tentando ficar tão zuado quanto no odoo/odoo de tanta raiva (la os PRs fica closed assim mesmo).