[14.0][RFC] l10n_br_account: refactor method to override create in account.move.line and fiscal.document.line

[marcelsavegnago]

Este unlink está afetando os valores de alguns campos do account.move como por exemplo o ind_final, ind_pres e outros e entendo que se faz melhor não deixar criar a linha do documento fiscal ao invés de criar para depois excluir.

[OCA-git-bot]

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

[rvalyi]

@marcelsavegnago gosto da ideia do PR. Porem vc tirou o decorator @api.create_multi, isso foi voluntário??
Eh algo importante tb, para nao impedir a criação dos account.move.line por batch até nas camadas superiores dos módulos...

[marcelsavegnago]

@marcelsavegnago gosto da ideia do PR. Porem vc tirou o decorator @api.create_multi, isso foi voluntário?? Eh algo importante tb, para nao impedir a criação dos account.move.line por batch até nas camadas superiores dos módulos...

nao foi proposital nao.. errei aqui no copia e cola mesmo :D

[marcelsavegnago]

@marcelsavegnago gosto da ideia do PR. Porem vc tirou o decorator @api.create_multi, isso foi voluntário?? Eh algo importante tb, para nao impedir a criação dos account.move.line por batch até nas camadas superiores dos módulos...

resolvido

[rvalyi]

@marcelsavegnago vc chegou a ver o erro com o modulo sale_commission?
https://github.com/OCA/l10n-brazil/actions/runs/8972989735/job/24642187210?pr=3064#step:8:1675

Estranho, mas se for so isso parece ser algo possivel de contornar (primeiro teria que entender o que ele ta fazendo diferente depois do seu PR).

[marcelsavegnago]

@marcelsavegnago vc chegou a ver o erro com o modulo sale_commission? https://github.com/OCA/l10n-brazil/actions/runs/8972989735/job/24642187210?pr=3064#step:8:1675

Estranho, mas se for so isso parece ser algo possivel de contornar (primeiro teria que entender o que ele ta fazendo diferente depois do seu PR).

Organizando agora para testar local.

[rvalyi]

oba passou. Vou revisar com calma já já...

[rvalyi]

parabéns pelo fix @marcelsavegnago .
Pessoal, é sempre melhor não criar algo do que ter que detonar depois...
Eh provavel que esse unlink que o Marcel removeu era de antes do refator onde eu consegui tirar os documentos fiscais dummy e impedir a criação de linhas de documentos fiscais com o override do create. Agora já parece mais óbvio que tem que ser desse jeito. Vai até facilitar para migrar para a v16...

EDIT: o unlink tava ai por motivo, pois não foi tão simples assim como proposto neste PR. Este PR criou uma regressão com notas de varias linhas e foi solucionado aqui depois: #3077

[marcelsavegnago]

parabéns pelo fix @marcelsavegnago . Pessoal, é sempre melhor não criar algo do que ter que detonar depois... Eh provavel que esse unlink que o Marcel removeu era de antes do refator onde eu consegui tirar os documentos fiscais dummy e impedir a criação de linhas de documentos fiscais com o override do create. Agora já parece mais óbvio que tem que ser desse jeito. Vai até facilitar para migrar para a v16...

opaa.. valeeeeuuu.

[rvalyi]

/ocabot merge patch

[OCA-git-bot]

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

[OCA-git-bot]

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

[rvalyi]

IMPORTANTE: este PR criou uma regressão importante que foi analisada aqui #3037 (review)

Um fix foi proposto aqui: #3077