[BUG] 12.0 Warnings ao criar e validar a fatura

[vanderleiromera]

Ao criar a fatura recebo esses warnings

[rvalyi]

@vanderleiromera valeu por reportar. Vc esta com o modulo l10n_br_delivery atualizado? Houve um update grande dele a semana passada #1301 ).
o campo incoterm_id deveria existir por conta dessa herança https://github.com/OCA/l10n-brazil/blob/12.0/l10n_br_delivery/models/fiscal_document_mixin.py#L15
Vc poderia verificar?

[vanderleiromera]

Vou checar e retorno

[vanderleiromera]

@rvalyi obrigado pela atenção, nesse DB não tem o modulo delivery instalado, então o warning do incoterm_id. Instalando o modulo não recebo mais os erros. A dúvida que fiquei, o modulo delivery precisa ser carregado como dependência, mesmo sem uso..Grato

[marcos-mendez]

Você deficiu o plano de contas? Em sex., 27 de ago. de 2021 às 19:33, Vanderlei P. Romera < ***@***.***> escreveu:

…

Ao criar a fatura recebo esses warnings [image: Screenshot from 2021-08-27 17-29-23] <https://user-images.githubusercontent.com/65144026/131194861-503432ad-076f-4434-9f9e-440215539463.png> — You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub <#1596>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AJEVVIG3W2OZA5BIKT3ZOILT7AHFHANCNFSM5C6MSEIA> . Triage notifications on the go with GitHub Mobile for iOS <https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675> or Android <https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub>.

[vanderleiromera]

Oh @marcos-mendez sim está definido, tudo funciona correto, mas tenho TOC com warnings, o Raphael explicou sobre a herança do incoterm_id, instalando o delivery não aparecem mais.

[marcos-mendez]

Entendi, eu tmb tenho TOC com esses warning, o foda é que o inconterm é algo da entrega, não fazia muito sentido ele pra mim ali, mas tudo bem, vamos que vamos, estou aqui para aprender Em sáb., 28 de ago. de 2021 às 19:04, Vanderlei P. Romera < ***@***.***> escreveu:

…

Oh @marcos-mendez <https://github.com/marcos-mendez> sim está definido, tudo funciona correto, mas tenho TOC com warnings, o Raphael explicou sobre a herança do incoterm_id, instalando o delivery não aparecem mais. — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#1596 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AJEVVIH2SQGESCTYN7AYZT3T7FMPHANCNFSM5C6MSEIA> . Triage notifications on the go with GitHub Mobile for iOS <https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675> or Android <https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub>.

[vanderleiromera]

Exato, agora também fiquei com essa dúvida, da dependência do módulo delivery.

[rvalyi]

@vanderleiromera @marcos-mendez eu acho que eu entendi o problema: quando o @mbcosta terminou a migraçao do modulo l10n_br_delivery, ele acabou botando o campo incoterm_id dentro do SHADOWED_FIELDS do documento fiscal a76bb93#diff-daae38396c6672c56badcaa22a10ae32a938909cd9387c12b0ec96268c9c5ea2

Isso para que seja possivel ter o incoterm_id dentro de um picking por examplo (pela herança do mixin do documento fiscal) mas tb que quando vc escreve o incoterm_id num invoice esse incoterm_id seja seja tb propagado ate o documento fiscal quando vc chamar o write, mesmo que ele tem o mesmo nome incorterm_id no invoice e no documento fiscal e que então so o mecanismo do framework do _inherits nao daria conta de fazer essa copia sem adicionar o campo no SHADOWED_FIELDS.

So que fazendo isso, quando instalar o l10n_br_account apenas ele ja tenta fazer esse write e da esse warning mesmo que nao tiver o l10n_br_delivery instalado.

Por isso, nesse PR para resolver o problema #1599, eu adicionei o campo no documento fiscal (pelo mixin) ja ao nivel do l10n_br_account e nao mais ao nivel do l10n_br_delivery como o @mbcosta tinha feito.

@vanderleiromera @marcos-mendez vcs podem verificar se resolve certinho? Valeu por reportar.

[rvalyi]

eu nao mudo minha analisa do problema, mas eu fiz um fix que eu acho melhor aqui #1600

@vanderleiromera @marcos-mendez vcs podem verificar se resolve certinho?

[marcos-mendez]

Rafa eu vou testar aqui e te falo, mas só por ler o código pra mim parece certo, obvio que é só um achismo, mas enfim, vou testar e te falo Em sáb., 28 de ago. de 2021 às 20:29, Raphaël Valyi < ***@***.***> escreveu:

…

eu nao mudo minha analisa do problema, mas eu fiz um fix que eu acho melhor aqui #1600 <#1600> @vanderleiromera <https://github.com/vanderleiromera> @marcos-mendez <https://github.com/marcos-mendez> vcs podem verificar se resolve certinho? — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#1596 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AJEVVIHKHMLJYZWBPKMHHALT7FWM3ANCNFSM5C6MSEIA> . Triage notifications on the go with GitHub Mobile for iOS <https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675> or Android <https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub>.

[vanderleiromera]

solucionado com a pr #1600