8000 bug: Broken formatting for Currency fields · Issue #21291 · frappe/frappe · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content
bug: Broken formatting for Currency fields #21291
Closed
@cogk

Description

@cogk

Hello,

3 days ago the formatting for Currency fields public/js/frappe/form/formatters.js:107 was changed in commit 54fabab.

However, this change had a side effect. All the Currency fields where the precision is not set are now rounded to 0 decimals, instead of falling back to sysdefaults.currency_precision.
Indeed, the precision of a docfield is a Select, and thus Currency fields with no defined precision have a value of "" (empty string). Thus using the null coalescing operator here is causing problems.

image

related to #21236

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions

      0