OpenERP Brazilian Portuguese Localization (pt_BR)

Merge lp://qastaging/~cjalmeida/openerp.pt-br-localiz/7.0-fiscalposition_split into lp://qastaging/~openerp-brazil-core-team/openerp.pt-br-localiz/openerp.pt-br-localiz-v7.0

  1. 7.0-fiscalposition_split
  2. Merge into openerp.pt-br-localiz-v7.0
Proposed by Cloves Almeida
Status: Rejected
Rejected by: Renato Lima - http://www.akretion.com
Proposed branch: lp://qastaging/~cjalmeida/openerp.pt-br-localiz/7.0-fiscalposition_split
Merge into: lp://qastaging/~openerp-brazil-core-team/openerp.pt-br-localiz/openerp.pt-br-localiz-v7.0
Diff against target: 7785 lines (+3112/-2612)
44 files modified
.bzrignore (+3/-0)
l10n_br_account/__init__.py (+1/-1)
l10n_br_account/__openerp__.py (+0/-2)
l10n_br_account/account_fiscal_position.py (+323/-0)
l10n_br_account/account_invoice.py (+20/-282)
l10n_br_account/account_invoice_view.xml (+98/-106)
l10n_br_account/demo/product_demo.xml (+104/-110)
l10n_br_account/l10n_br_account.py (+1/-3)
l10n_br_account/l10n_br_account_view.xml (+0/-2)
l10n_br_account/res_partner.py (+3/-307)
l10n_br_account/res_partner_view.xml (+0/-128)
l10n_br_data_account/__openerp__.py (+0/-1)
l10n_br_data_account/l10n_br_data_account_demo.xml (+1/-843)
l10n_br_fiscal_position/__init__.py (+26/-0)
l10n_br_fiscal_position/__openerp__.py (+63/-0)
l10n_br_fiscal_position/account_fiscal_position_rule_data.xml (+219/-218)
l10n_br_fiscal_position/account_fiscal_position_rule_demo.xml (+850/-0)
l10n_br_fiscal_position/account_fiscal_position_view.xml (+134/-0)
l10n_br_fiscal_position/account_invoice.py (+317/-0)
l10n_br_fiscal_position/account_invoice_view.xml (+63/-0)
l10n_br_fiscal_position/l10n_br_account.py (+27/-0)
l10n_br_fiscal_position/l10n_br_account_view.xml (+16/-0)
l10n_br_fiscal_position/purchase.py (+168/-0)
l10n_br_fiscal_position/purchase_view.xml (+41/-0)
l10n_br_fiscal_position/sale.py (+171/-0)
l10n_br_fiscal_position/sale_view.xml (+65/-0)
l10n_br_fiscal_position/security/ir.model.access.csv (+4/-0)
l10n_br_fiscal_position/stock.py (+168/-0)
l10n_br_fiscal_position/stock_view.xml (+68/-0)
l10n_br_fiscal_position/wizard/__init__.py (+20/-0)
l10n_br_fiscal_position/wizard/stock_return_picking.py (+65/-0)
l10n_br_purchase/__openerp__.py (+0/-1)
l10n_br_purchase/purchase.py (+2/-139)
l10n_br_purchase/purchase_view.xml (+2/-16)
l10n_br_purchase/security/ir.model.access.csv (+0/-1)
l10n_br_sale/__openerp__.py (+0/-1)
l10n_br_sale/sale.py (+1/-140)
l10n_br_sale/sale_view.xml (+3/-48)
l10n_br_sale/security/ir.model.access.csv (+0/-2)
l10n_br_sale_stock/sale_stock_view.xml (+0/-11)
l10n_br_stock/__openerp__.py (+0/-1)
l10n_br_stock/stock.py (+11/-137)
l10n_br_stock/stock_view.xml (+34/-63)
l10n_br_stock/wizard/stock_return_picking.py (+20/-49)
To merge this branch: bzr merge lp://qastaging/~cjalmeida/openerp.pt-br-localiz/7.0-fiscalposition_split
Reviewer Review Type Date Requested Status
Renato Lima - http://www.akretion.com Disapprove
Review via email: mp+172249@code.qastaging.launchpad.net

Commit message

Esta modificação é uma refatoração isolando as dependências aos módulos
"account_fiscal_position_rule*" em um módulo "l10n_br_fiscal_position". O
objetivo é permitir implementações alternativas das regras fiscais.

Esta deve ser compatível com a estrutura de dados existente, bastando um
"--init=l10n_br_fiscal_position" para se ter um sistema equivalente ao original.

----

FIX: Ajuste dos dados de demonstração em l10n_br_account/product_demo.xml
para refletir as alterações feita pela OpenERP S.A. para versão 7.0 e facilitar testes.

REF: Movendo metodos "on_change" dos objetos "account.invoice",
     "account.invoice.line", "purchase.order", "purchase.order.line",
     "sale.order", "sale.order.line", "stock.picking", "stock.picking.in"
     "stock.picking.out"

REF: Nos arquivos __openerp__.py, migrado a dependência dos módulos
     "account_fiscal_position_rule*" para o módulo "l10n_br_fiscal_position".

REF: Ajuste nas views "account_invoice_view.xml", "purchase_view.xml",
     "sale_view.xml" e "stock_view.xml", migrando os atributos "on_change"
     para o módulo "l10n_br_fiscal_position".

REF: Migrado os arquivos "account_fiscal_position_rule_data.xml",
     "account_fiscal_position_rule_demo.xml",
     "account_fiscal_position_rule_view.xml" e
     "account_fiscal_position_rule.py"
     que faziam implantavam funcionalidades e dados dependentes de
     "account_fiscal_position_rule*" para o módulo "l10n_br_fiscal_position".

REF: Ajuste em "security/ir.model.access.csv" de todos os módulos, migrando
     os registros que referenciavam o objeto "account.fiscal.position.rule"
     para o módulo "l10n_br_fiscal_position".

Description of the change

Esta modificação é uma refatoração isolando as dependências aos módulos "account_fiscal_position_rule*" em um módulo "l10n_br_fiscal_position". O objetivo é permitir implementações alternativas das regras fiscais. O merge é compatível com a estrutura de dados existente, bastando um "--init=l10n_br_fiscal_position" para se ter um sistema equivalente ao original.

Com os dados de demo não deu erro em nenhuma da operações básicas (cadastrar cliente, cadastrar produto, novo pedido de venda, ordem de entrega, criar e validar invoice), mantendo as mesmas funcionalidades quando instalado o módulo l10n_br_fiscal_position. Sem o l10n_br_fiscal_position instalado, todas as funcionalidades da localização, exceto a atribuição do fiscal_position usando as regras estão OK.

Apesar do diff ser grande, o grosso é "copy/paste". Poucas alterações "dentro" dos métodos ou views.

To post a comment you must log in.
Revision history for this message
Renato Lima - http://www.akretion.com (renatonlima) wrote :

Olá Cloves,

Particulamente eu acho interessante a ideia de isolar a dependência do account_fiscal_position_rule, eu vou analisar o merge

Revision history for this message
Renato Lima - http://www.akretion.com (renatonlima) wrote :

Olá Cloves,

Eu analisei o código do merge proposto por você, a principio achei interessante na localização não ter um dependência forte de módulos que não estão no addons, mas no seu merge existe um problema sério, existe o modulo l10n_br_fiscal_position que tem como dependência todos os módulos account_fiscal_position_rule_* o que perde a modularidade, por exemplo se alguém quiser usar o módulo de vendas, mas não quiser usar compras, ao instalar o módulo l10n_br_fiscal_position ele sempre vai instalar todos os módulos dependentes (sale, purchase, stock), seria bom modularizar esse módulo l10n_br_fiscal_position e o que seria interessante também seria tirar esses módulos do projeto core da localização, desta forma o core da localização só vai depender do addons nativo o que vai facilitar de certa forma a instalação (estamos fazendo isso com o account_payment_extension). Para extender o motor de regras de posições fiscais agora na versão 7.0 com a refatoração que fizemos nos módulos account_fiscal_position_rule_* existe o método:

_map_domain(self, cr, uid, partner, addrs, company, context=None, **kwargs):

Que pode ser estendido de forma fácil passando mais parâmetros no método (vide exemplo no módulo l10n_br_account)

Como estamos em fase de migração do projeto da localização do launchpad para o github, vamos rejeitar o seu merge aqui e fazer a migração da branch do projeto da localização, você poderia fazer as modificações de modularização e reenviar um pull request no github!?

Obrigado pela contribuição,

Renato Lima - http://www.akretion.com/pt-BR

Revision history for this message
Renato Lima - http://www.akretion.com (renatonlima) wrote :

O merge deve ser modularizado, para manter a modularidade da localização brasileira, vamos migrar do launchpad para o github e esse merge pode ser refeito no github.

Obrigado pela contribuição,

Renato Lima - http://www.akretion.com/pt-BR

review: Disapprove
Revision history for this message
Raphaël Valyi - http://www.akretion.com (rvalyi) wrote :

Ola Cloves,
eu concordo com o Renato.

talvez seja interessante entao criar varios modulos assim que o Renato
explicou para preservar a modularidade. Presevar a modularidade é algo
essencial para democratzar o OpenERP. Pois fica mais simples, menos
arriscado ou seja mais barrato, alguem que so precisa da gestao de venda
nao instalar ainda gestao de estoque ou de compra.

A ideia de nao ter a dependencia das regras fiscais no core é interessante.
Assim por examplo, alguns que tem poucas notas poderiam ate usar o OpenERP
como o emissor gratuito, preenchendo tudo na mao.

Agora, com uns 3 ou 4 modulos adicionais, isso seria uma mudança grande,
talvez a gente so faria ela na versao 8, vamos pensar.

Assim que o Renato explicou, hoje, apesar de ter a dependencia sobre as
regras fiscais, é razoavlemente facil sobrecarregar e fazer o que vc queria.

Outra coisa, com o Renato, a gente pensa que talvez a gente vai deixar os
campos automaticos como CST possivel de se preencher manualmente (hoje
readonly) e so prohibir a ediçao num modulo adicional. Isso é uma mudança
simples de fazer, provavelmente iremos fazer.

Abraço e obrigado pelo interesso no projeto.

On Sat, Aug 10, 2013 at 12:53 PM, Renato Lima - http://www.akretion.com <
<email address hidden>> wrote:

> The proposal to merge
> lp:~cjalmeida/openerp.pt-br-localiz/7.0-fiscalposition_split into
> lp:openerp.pt-br-localiz has been updated.
>
> Status: Needs review => Rejected
>
> For more details, see:
>
> https://code.launchpad.net/~cjalmeida/openerp.pt-br-localiz/7.0-fiscalposition_split/+merge/172249
> --
>
> https://code.launchpad.net/~cjalmeida/openerp.pt-br-localiz/7.0-fiscalposition_split/+merge/172249
> Your team OpenERP Brazil Core Team is requested to review the proposed
> merge of lp:~cjalmeida/openerp.pt-br-localiz/7.0-fiscalposition_split into
> lp:openerp.pt-br-localiz.
>

Unmerged revisions

549. By Cloves Almeida

Esta modificação é uma refatoração isolando as dependências aos módulos
"account_fiscal_position_rule*" em um módulo "l10n_br_fiscal_position". O
objetivo é permitir implementações alternativas das regras fiscais.

Esta deve ser compatível com a estrutura de dados existente, bastando um
"--init=l10n_br_fiscal_position" para se ter um sistema equivalente ao original.

----

FIX: Ajuste dos dados de demonstração em l10n_br_account/product_demo.xml para
     refletir as alterações feita pela OpenERP S.A. para versão 7.0 e facilitar
     testes.

REF: Movendo metodos "on_change" dos objetos "account.invoice",
     "account.invoice.line", "purchase.order", "purchase.order.line",
     "sale.order", "sale.order.line", "stock.picking", "stock.picking.in"
     "stock.picking.out"

REF: Nos arquivos __openerp__.py, migrado a dependência dos módulos
     "account_fiscal_position_rule*" para o módulo "l10n_br_fiscal_position".

REF: Ajuste nas views "account_invoice_view.xml", "purchase_view.xml",
     "sale_view.xml" e "stock_view.xml", migrando os atributos "on_change" para
     o módulo "l10n_br_fiscal_position".

REF: Migrado os arquivos "account_fiscal_position_rule_data.xml",
     "account_fiscal_position_rule_demo.xml",
     "account_fiscal_position_rule_view.xml" e "account_fiscal_position_rule.py"
     que faziam implantavam funcionalidades e dados dependentes de
     "account_fiscal_position_rule*" para o módulo "l10n_br_fiscal_position".

REF: Ajuste em "security/ir.model.access.csv" de todos os módulos, migrando os
     registros que referenciavam o objeto "account.fiscal.position.rule" para o
     módulo "l10n_br_fiscal_position".

ADD: Arquivo .bzrignore ignora arquivos do Eclipse Pydev.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.