[12.0][REF] Delivery Costs( Freight, Insurance and Other Value )

[mbcosta]

Refectoring Delivery Costs( Freight, Insurance and Other Value )

Alterações:

• Questões de autoria referente ao modulo l10n_br_delivery [Q&A] What are the cases where an author, person or company, can be removed in the modules ? #1252 ( PSCs por favor validem quem e quais empresas devem constar no modulo )
• Criado Parâmetro para definir se os valores de entrega Frete, Seguro e Outros Valores devem ser informados "Por Linhas" ( caso onde as linhas podem ter valores diferentes e não se quer ratear o valor ) ou "Por Total" ( caso onde é informado apenas o valor total e será rateado os valores por linhas ) dentro do modulo l10n_br_sale
• Movido métodos de inversão de campos no caso de rateio do l10n_br_delivery para o l10n_br_sale, pois esses campos já existem ao instalar apenas o l10n_br_sale
• Apagado o arquivo que implementava os campos inversos no Documento Fiscal porque o valor deve ser definido por linha e caso se queira lá informar apenas os Valores Totais e esses valores serem rateados por linha isso deve ser implementado pelo wizard https://github.com/OCA/l10n-brazil/blob/12.0/l10n_br_fiscal/wizards/l10n_br_account_invoice_costs_ratio.py ( TODO )
• Retornado os objetos l10n_br_delivery.carrier.vehicle e l10n_br_delivery.shipment https://github.com/OCA/l10n-brazil/blob/10.0/l10n_br_delivery/models/l10n_br_delivery.py https://github.com/OCA/l10n-brazil/blob/10.0/l10n_br_delivery/views/l10n_br_delivery_view.xml ( verificar o Cabeçalho de Licença do arquivo pois mantive o que existia, já que não houve grande mudança, deveria ser alterado ? )
• Incluído Teste do caso de valores diferentes nas linhas, onde não deve ocorrer o rateio dos valores, e comentado teste onde já não é mais possível a edição dos Valores Totais no Documento Fiscal.

cc @renatonlima @rvalyi @mileo @gabrielcardoso21

[renatonlima]

@mbcosta

Eu testei e se a opção do lançamento dos valores por linha e esta dando um erro ao criar um novo pedido de venda em loop

[mbcosta]

Corrigido o erro de recursividade e foram incluídas as seguintes alterações e melhorias:

• O name_get do objeto account.incoterms retornar o "Codigo - Descrição", muitas pessoas no Brasil conhecem mais os tipos de Frete pela sigla em inglês do que pela descrição tanto em inglês quanto português, antes:

depois:

• Incluído dois campos tipo 'compute' com o Total de Peso e Volume no Pedido de Venda/sale.order em muitos casos a empresa usa essa informação para fazer a cotação do valor do frete, mesmo antes de gerar a nota fiscal

[renatonlima]

@mbcosta,

Você pode fazer o rebase desta branch?

[renatonlima]

@mbcosta,

Você pode dá uma olhada no erro do travis? https://app.travis-ci.com/github/OCA/l10n-brazil/jobs/524473845#L4366

[mbcosta]

ola @renatonlima , pelo o que entendi do erro isso está acontecendo devido a alteração que ocorre no campo price_unit da stock.move ao Confirmar o stock.picking devido aos métodos de Valorização de Estoque do core eu procurei resolver esse problema aqui #1523

[renatonlima]

@mileo,

You added "blocked" label in this PR, but it is important that you comment and explain the reason that you added this and we debate about changes of this PR.

[renatonlima]

@mbcosta please can you rebase?

[rvalyi]

almost OK, Excellent work! Please also remove the models/stock_picking.py file entirely as it is not used anywhere at all anymore.

[rvalyi]

remove l10n_br_account, this is implied by l10n_br_sale_stock already

[rvalyi]

Ola @mbcosta vc esqueceu de remover o arquivo models/stock_picking.py isso não esta usado em lugar nenhum.

[rvalyi]

Falei besteira, o override no stock_picking.py tá sendo chamado no módulo delivery. Portanto tá tudo OK.

[renatonlima]

/ocabot merge nobump

[OCA-git-bot]

@renatonlima The merge process could not start, because command git push origin 12.0-ocabot-merge-pr-1301-by-renatonlima-bump-nobump failed with output:

To https://github.com/OCA/l10n-brazil
 ! [remote rejected]     12.0-ocabot-merge-pr-1301-by-renatonlima-bump-nobump -> 12.0-ocabot-merge-pr-1301-by-renatonlima-bump-nobump (cannot lock ref 'refs/heads/12.0-ocabot-merge-pr-1301-by-renatonlima-bump-nobump': reference already exists)
error: failed to push some refs to 'https://[email protected]/OCA/l10n-brazil'

[OCA-git-bot]

On my way to merge this fine PR!
Prepared branch 12.0-ocabot-merge-pr-1301-by-renatonlima-bump-nobump, awaiting test results.

[OCA-git-bot]

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