Dipak my report is not Proforma report my report name is Qoutation / Sale ...is it possible this code i can inherit and get state from self and put condition on it ...

pls write some more help

> Your report name is "Quotation / Order" as you mentioned in your question.

> the code i given is useful.and yes please use 'docs' variable instead of 'self' if you can see in code 'docs' variable already contains recordset. so iterate it and check your condition of state.

```

class SaleOrderReportProforma(models.AbstractModel):

_inherit = 'report.sale.report_saleproforma'

@api.multi

def _get_report_values(self, docids, data=None):

res = super(SaleOrderReportProforma, self)._get_report_values(self, docids, data=None)

docs = self.env['sale.order'].browse(docids)

for doc in docs:

if doc.state != 'sale':

raise exceptions.ValidationError('Please approve the order to print the report.')

else:

return res

```

it's not working not giving error install successfully ... where i'm wrong ?

Hello Usman,

I tested below code , its working well.

Please try below code:

class SaleOrderReport(models.AbstractModel):

_name = 'report.sale.report_saleorder'

_description = 'Sale order Report'

@api.multi

def _get_report_values(self, docids, data=None):

docs = self.env['sale.order'].browse(docids)

for doc in docs:

if doc.state != 'sale':

raise exceptions.ValidationError('Please approve the order to print the report.')

return {

'doc_ids': docs.ids,

'doc_model': 'sale.order',

'docs': docs,

'proforma': True

}

replace ...

_inherit = 'report.sale.report_saleproforma'

with this code ..

_name = 'report.sale.report_saleorder'

@Dipek I tried your code, it works but it prints empty report for me.

could you please let me know how to replace @api.multi in v14

this is last option

but any logic how i can achive my goal ?

This is the simplest option to achieve your goal.