[12.0][MIG[REF] Split Fiscal Mixin Document in two class Fields and Methods #881
Conversation
mbcosta
force-pushed
the
12.0-split_fields_methods_fiscal_document
branch
from
c6bad04 to
2b4feab
Compare
| for comment_id in self.fiscal_operation_id.comment_ids: | ||
| self.comment_ids += comment_id | ||
|
|
||
| subsequent_documents = [(6, 0, {})] |
There was a problem hiding this comment.
Precisa ter um metodo _onchange_fiscal_operation_id no l10n_br_fiscal.document para chamar o super e depois colocar essa parte do código do documento subsequente.
mbcosta
force-pushed
the
12.0-split_fields_methods_fiscal_document
branch
from
2b4feab to
9ae3ca3
Compare
mbcosta
force-pushed
the
12.0-split_fields_methods_fiscal_document
branch
from
9ae3ca3 to
2ad8842
Compare
| from odoo import api, models | ||
|
|
||
|
|
||
| class FiscalDocumentMixin(models.AbstractModel): |
There was a problem hiding this comment.
Tem que mudar o nome dessa class:
| class FiscalDocumentMixin(models.AbstractModel): | |
| class FiscalDocumentMixinMethods(models.AbstractModel): |
mbcosta
force-pushed
the
12.0-split_fields_methods_fiscal_document
branch
from
2ad8842 to
de1dc62
Compare
|
Hi @renatonlima @rvalyi thanks for your review, I made the changes |
| @@ -0,0 +1,83 @@ | |||
| # Copyright (C) 2019 Renato Lima - Akretion <[email protected]> | |||
There was a problem hiding this comment.
@mbcosta Coloca seu nome e 2020
There was a problem hiding this comment.
desculpa @hendrixcosta no caso eu nao acho justificado: o Magno apenas cortou uma classe o arquivo em 2 copiando esse meu PR da linha #839 e a mando da gente. Quando alguém alterar uma porção significativa de código a gente nunca teve problema em acrescentar o @author. Porem não eh legal passar a imagem do que qualquer pessoa que alterar 2 linhas num arquivo acrescenta copyrights no projeto, pois vc qdo essas 2 linhas tem mentoria da gente ou que eh fix obvio eh discutivel e vc sabe que infelizmente aqui tem gente que ta para trolar o projeto de querer enfiar o nome dele ou da empresa dele a qualquer custo para fingir que tem algum papel de relevância (e ver o apescto comercial por tras) enquanto nao tem. Eh triste ter gente assim, mas aqui tem, entao a gente tem que ver o examplo que a gente da tb. Pensa que se vc der esse examplo, daqui 2 anos, pode ter gente que altera 5% do codigo dos seus modulos de RH e ja vai querer passar a imagem que fez o mesmo quanto vc e querer se prevalecer do trabalho, de coordenar o projeto e sei la o que... Nos modulos que vc lidera vc pode ate decidir isso, mas eh uma coisa para vc pensar... Ai por examplo o Magno ta trabalhando em migrar os modulos de CNAB ai vc vai ver que ele vai accrescentar os copryrights (que na vdd são da Akretion) e autoria dele sem o menor problema, e ja tem, mas ai eh trabalho significativo.
…Fields and Methods for allow use it in inherit class.
renatonlima
force-pushed
the
12.0-split_fields_methods_fiscal_document
branch
from
de1dc62 to
d6c6965
Compare
|
/ocabot merge nobump |
|
This PR looks fantastic, let's merge it! |
|
Congratulations, your PR was merged at 6067b13. Thanks a lot for contributing to OCA. ❤️ |
Split Fiscal Mixin Document in two class Fields and Methods for allow use it in inherit class.
cc @renatonlima @rvalyi @mileo