Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[NEW] Res Partner Fiscal Document Relation close: #1151 #1164

Closed
wants to merge 1 commit into from

Conversation

mileo
Copy link
Member

@mileo mileo commented Mar 1, 2021

No description provided.

@marcelsavegnago
Copy link
Member

@mileo aeee.. parabens. Terminando de rodar os testes já reviso e testo.

@renatonlima
Copy link
Member

@mileo

Dá uma olhada no PR #1146 Com a unificação da edição do documento fiscal e fatura eu não acho necessário.

@mileo
Copy link
Member Author

mileo commented Mar 1, 2021

Se for possível emitir uma nf-e s/ uma invoice, sem lançamentos contábeis, isto se faz necessário.

Até por tem uma forma de acessar rapidamente as notas.

cc: @renatonlima

@mileo
Copy link
Member Author

mileo commented Mar 22, 2021

@renatonlima como o #1146 atua somente no nível do account creio que este PR ainda seja relevante, para alguns casos de uso:

  1. Somente o módulo fiscal instalado;
  2. O módulo fiscal e l10n_br_account instalados e a necessidade de se emitir um documento fiscal sem fatura;

Vou pensar como deixar esta integração mais natural e volto a comentar aqui.

@mileo
Copy link
Member Author

mileo commented Apr 19, 2021

Pessoal este PR ainda tem utilidade podem revisar/comentar por gentileza? Fiz o rebase.

@marcelsavegnago
Copy link
Member

@mileo as tags da PR estão atualizadas ?

def _compute_fiscal_document_count(self):
for partner in self:
partner.fiscal_documents_count = partner.fiscal_document_ids.search_count(
[('partner_id', '=', partner.id)]
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't think it's necessary to apply this domain, because the recordset is already a one2many field and contains only the documents which the partner_id field is the partner record in question. You could do the same with len(partner.fiscal_document_ids) or partner.fiscal_document_ids.search_count([])

@rvalyi
Copy link
Member

rvalyi commented Apr 20, 2021

pessoal, por favor deixem um tempo para a gente avaliar esse PR de novo. Eu tb nao estou convencido do que é uma boa um botao desse a partir do momento que vc manipula o documento fiscal pelo invoice (e futuramente account.move), eu acho que isso so cria mais entropia no codigo e arrisca uma ediçao pela tela do documento fiscal sem acontecer a sincronizaçao do invoice/move, o que é algo perigroso... Talvez seja algo que a KMEE deveria ter num modulo dela como temos do Akretion usability e nao algo no modulo fiscal...


@api.multi
def action_view_document(self):
action = self.env.ref('l10n_br_fiscal.document_out_action').read()[0]
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Deve abrir a action correta, porque dependendo do parceiro pode existir notas de entrada/saída e no l10n_br_account deve alterar a action para abrir a action dos documentos fiscais mas com o objeto account.invoice, porque quando o l10n_br_account é instalado todo documento fiscal é criado através do objeto account.invoice

@github-actions
Copy link

There hasn't been any activity on this pull request in the past 4 months, so it has been marked as stale and it will be closed automatically if no further activity occurs in the next 30 days.
If you want this PR to never become stale, please ask a PSC member to apply the "no stale" label.

@github-actions github-actions bot added the stale PR/Issue without recent activity, it'll be soon closed automatically. label Mar 26, 2023
@github-actions github-actions bot closed this Apr 30, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
12.0 enhancement needs fixing stale PR/Issue without recent activity, it'll be soon closed automatically. work in progress
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants