[4332][ADD] account_invoice_custom_portal_report #224
Conversation
| @@ -0,0 +1,13 @@ | |||
| # Copyright 2024 Quartile Limited | |||
There was a problem hiding this comment.
| # Copyright 2024 Quartile Limited | |
| # Copyright 2024 Quartile (https://www.quartile.co) |
| "version": "12.0.1.0.0", | ||
| "category": "Portal", | ||
| "license": "AGPL-3", | ||
| "author": "Quartile Limited", |
There was a problem hiding this comment.
| "author": "Quartile Limited", | |
| "author": "Quartile", |
| help="This report will be used as a template in the customer portal " | ||
| "view of the invoice.", |
There was a problem hiding this comment.
| help="This report will be used as a template in the customer portal " | |
| "view of the invoice.", | |
| help="This report template will be used in the customer portal to show the invoice.", |
| Background | ||
| ~~~~~~~~~~ | ||
|
|
||
| When we create and use a custom report for invoices and send an email to our customers with this custom report, | ||
| the customer sees the Odoo standard report template when they open the invoice in their portal view. | ||
| This module addresses this issue. |
There was a problem hiding this comment.
Is this necessary? We generally prefer to have a context/reasoning to what we do, but this one looks obvious to me.
| class CustomerPortal(CustomerPortal): | ||
| def _show_report(self, model, report_type, report_ref, download=False): | ||
| invoice_report = request.env.user.company_id.account_invoice_report | ||
| if model._name == "account.invoice" and invoice_report: |
There was a problem hiding this comment.
This should only be done for customer invoices.
@kanda999 Please check which template should show for customer credit notes.
There was a problem hiding this comment.
@yostashiro
The custom report we want to set up can also display credit note.
However, the customer may want to view a different report as credit note in the future.
@AungKoKoLin1997
Could you please add the filter for customer invoice?
| class ResCompany(models.Model): | ||
| _inherit = "res.company" | ||
|
|
||
| account_invoice_report = fields.Many2one( |
There was a problem hiding this comment.
I think that a name like account_invoice_portal_report would be better.
There was a problem hiding this comment.
Should change the name of the field since we are adding a filter.
|
|
||
| account_invoice_report = fields.Many2one( | ||
| "ir.actions.report", | ||
| help="This report will be used as a template in the customer portal " |
There was a problem hiding this comment.
| help="This report will be used as a template in the customer portal " | |
| domain="[('model', '=', 'account.invoice')]", | |
| help="This report will be used as a template in the customer portal " |
kanda999
force-pushed
the
12.0-add-account_invoice_custom_portal_report
branch
from
70ccfe5 to
795c9e6
Compare
| invoice_report = request.env.user.company_id.customer_invoice_portal_report | ||
| if ( | ||
| model._name == "account.invoice" | ||
| and invoice_report | ||
| and model.type == "out_invoice" | ||
| ): |
There was a problem hiding this comment.
For early returns.
| invoice_report = request.env.user.company_id.customer_invoice_portal_report | |
| if ( | |
| model._name == "account.invoice" | |
| and invoice_report | |
| and model.type == "out_invoice" | |
| ): | |
| if not (model._name == "account.invoice" and model.type == "out_invoice"): | |
| return super()._show_report(model, report_type, report_ref, download) | |
| invoice_report = request.env.user.company_id.customer_invoice_portal_report | |
| if invoice_report: |
4332