Topic: Purchase Order Report rep209

In rep209 the Tax rate is displayed twice in succession i.e Tax(5%)(5%). I dont know how to correct it.
Can someone help me here.
Image attached.
FA2.4.7

Post's attachments

rep209.png 31 kb, file has never been downloaded. 

You don't have the permssions to download the attachments of this post.

2 (edited by notrinos 01/14/2020 05:18:03 pm)

Re: Purchase Order Report rep209

Hello rafat, code block from line 179 to 182 should be replaced with only one line:

$tax_type_name = $tax_item['tax_type_name'];

Since the function get_tax_group_rates() in tax_group_db.inc has already check the suppress_tax_rates status so dont need to check again.
See the difference in the attached below

Post's attachments

rep_209_dif.png 17.6 kb, file has never been downloaded. 

You don't have the permssions to download the attachments of this post.
Phuong

Re: Purchase Order Report rep209

Thanks @notrinos
You are right.. maybe we need to commit this one.

Re: Purchase Order Report rep209

@joe: what should change and where (earlier or later) - should this be committed?

Re: Purchase Order Report rep209

This has been fixed and committed to 2.4.7 stable repo. A fixed file can be downloaded here.

/Joe

Re: Purchase Order Report rep209

The same construct is seen in

reporting/rep107.php
reporting/rep110.php
reporting/rep113.php
includes/ui/ui_view.inc

@joe: Do any of these reports or file show the same effect?

Re: Purchase Order Report rep209

Hello @apmuthu these reports do not have this problem.
These constructs are not uniform due to our lack of a codding standard.
A few sql queries return value outside of database and others don't

Phuong