Skip to content

Conversation

@AungKoKoLin1997
Copy link
Contributor

@AungKoKoLin1997 AungKoKoLin1997 force-pushed the 16.0-add-account_report_qweb_decimal_place branch 2 times, most recently from 6667a79 to 6bfee43 Compare January 4, 2023 13:49
Copy link
Member

@yostashiro yostashiro left a comment

Choose a reason for hiding this comment

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

@@ -0,0 +1 @@
This module provides the price unit format of invoice report depends on the choice of price_decimal_places in currency.
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
This module provides the price unit format of invoice report depends on the choice of price_decimal_places in currency.
This module provides the option to adjust the price unit format (number of decimal places to apply)
of invoice report according to the configuration of price_decimal_places in currency.
This module depends on report_qweb_decimal_place module in OCA/reporting-engine.

Comment on lines 8 to 16
<xpath expr="//tbody[@class='invoice_tbody']//tr//td[3]" position="replace">
<td
t-attf-class="text-end {{ 'd-none d-md-table-cell' if report_type == 'html' else '' }}"
>
<t t-set="currency" t-value="o.currency_id" />
<t t-set="price_unit" t-value="line.price_unit" />
<t t-call="report_qweb_decimal_place.price_unit_value_format" />
</td>
</xpath>
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
<xpath expr="//tbody[@class='invoice_tbody']//tr//td[3]" position="replace">
<td
t-attf-class="text-end {{ 'd-none d-md-table-cell' if report_type == 'html' else '' }}"
>
<t t-set="currency" t-value="o.currency_id" />
<t t-set="price_unit" t-value="line.price_unit" />
<t t-call="report_qweb_decimal_place.price_unit_value_format" />
</td>
</xpath>
<xpath expr="//span[@t-field='line.price_unit']" position="replace">
<t t-set="currency" t-value="o.currency_id" />
<t t-set="price_unit" t-value="line.price_unit" />
<t t-call="report_qweb_decimal_place.price_unit_value_format" />
</xpath>

Comment on lines 8 to 16
<xpath expr="//span[@t-field='line.price_unit']" position="replace">
<t t-set="currency" t-value="o.currency_id" />
<t t-set="price_unit" t-value="line.price_unit" />
<t t-call="report_qweb_decimal_place.price_unit_value_format" />
</xpath>
Copy link
Member

Choose a reason for hiding this comment

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

Depends on this discussion: #6 (review)

@yostashiro yostashiro force-pushed the 16.0-add-account_report_qweb_decimal_place branch from 7e32050 to 66c64a2 Compare January 28, 2023 15:40
@yostashiro
Copy link
Member

Same issue as #6 (comment). Ran pre-commit locally to confirm there is no issue.

@yostashiro yostashiro merged commit 26a42cd into 16.0 Jan 28, 2023
@AungKoKoLin1997 AungKoKoLin1997 deleted the 16.0-add-account_report_qweb_decimal_place branch March 13, 2023 01:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants