Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[14.0][FIX] l10n_br_nfe: remoção do warning inconsistent 'compute_sudo' #3107

Merged
merged 1 commit into from
May 31, 2024

Conversation

antoniospneto
Copy link
Contributor

Pequena correção para remover o seguinte WARNING:

2024-05-29 18:08:06,760 572 WARNING odoo odoo.modules.registry: 
res.partner: inconsistent 'compute_sudo' for computed fields: 
nfe40_CNPJ, nfe40_CPF, nfe40_choice_emit, nfe40_CEP, nfe40_fone, nfe40_IE, 
nfe40_choice_tlocal, nfe40_choice_dest, nfe40_choice_autxml, nfe40_choice_transporta 

O problema ocorre porque alguns campos computados pela função _compute_nfe_data possuem a propriedade compute_sudo=True, enquanto outros não. Para garantir consistência, todos os campos foram ajustados para compute_sudo=True.

@OCA-git-bot
Copy link
Contributor

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

@marcelsavegnago marcelsavegnago requested review from rvalyi and mbcosta May 31, 2024 12:51
@renatonlima
Copy link
Member

/ocabot merge minor

@OCA-git-bot
Copy link
Contributor

This PR looks fantastic, let's merge it!
Prepared branch 14.0-ocabot-merge-pr-3107-by-renatonlima-bump-minor, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit 40dbe5d into OCA:14.0 May 31, 2024
7 checks passed
@OCA-git-bot
Copy link
Contributor

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

@rvalyi
Copy link
Member

rvalyi commented May 31, 2024

pessoal esse tipo de coisa pode ser merge patch (precisa atualizar o banco não). Eh bom a gente começar a fazer isso com rigor para ter escalabilidade la na frente...

cc @renatonlima

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants