Merge lp://qastaging/~cmsa/openerp.pt-br-localiz/onchange_shop_id_arguments into lp://qastaging/~openerp-brazil-core-team/openerp.pt-br-localiz/openerp.pt-br-localiz-v7.0
Proposed by
Carlos Almeida
| Status: | Superseded |
|---|---|
| Proposed branch: | lp://qastaging/~cmsa/openerp.pt-br-localiz/onchange_shop_id_arguments |
| Merge into: | lp://qastaging/~openerp-brazil-core-team/openerp.pt-br-localiz/openerp.pt-br-localiz-v7.0 |
| Diff against target: |
49 lines (+1/-16) 3 files modified
l10n_br_sale/__openerp__.py (+1/-1) l10n_br_sale/sale.py (+0/-8) l10n_br_sale/sale_view.xml (+0/-7) |
| To merge this branch: | bzr merge lp://qastaging/~cmsa/openerp.pt-br-localiz/onchange_shop_id_arguments |
| Related bugs: |
| Reviewer | Review Type | Date Requested | Status |
|---|---|---|---|
| Renato Lima - http://www.akretion.com | Disapprove | ||
|
Review via email:
|
|||
Description of the change
Removi a redefinição na view e do método, pois não faz sentido redefinir este método só para chamar o super, se o super não tem estes parâmetros todos e o método não faz nada aqui com esses parâmetros...
To post a comment you must log in.
Revision history for this message
| Renato Lima - http://www.akretion.com (renatonlima) wrote | # |
review:
Disapprove
Unmerged revisions
- 544. By Carlos Almeida
-
[FIX] BUG1190772
Não faz sentido redefinir este método só para chamar o super, se o super não tem estes parâmetros todos e o método não faz nada aqui com esses parâmetros...
Preview Diff
[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
Este método na localização existe para passar no contexto o campo fiscal_category_id onde ao chamar o super, passa pelo método account_