vassili/seitime-frappe
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions 4

Merge pull request #24726 from barredterra/respect-null

Browse source 
fix: respect `null` when formatting number values
This commit is contained in:
mergify[bot] 2024-02-05 05:28:14 +00:00 • committed by GitHub
commit 77b248b180
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
1 changed files with 16 additions and 0 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

16
frappe/public/js/frappe/form/formatters.js
View file

@ -49,6 +49,10 @@ frappe.form.formatters = {
return __(frappe.form.formatters["Data"](value, df));
},
Float: function (value, docfield, options, doc) {
if (value === null) {
return "";
}
// don't allow 0 precision for Floats, hence or'ing with null
var precision =
docfield.precision ||
@ -73,12 +77,20 @@ frappe.form.formatters = {
}
},
Int: function (value, docfield, options) {
if (value === null) {
return "";
}
if (cstr(docfield.options).trim() === "File Size") {
return frappe.form.formatters.FileSize(value);
}
return frappe.form.formatters._right(value == null ? "" : cint(value), options);
},
Percent: function (value, docfield, options) {
if (value === null) {
return "";
}
const precision =
docfield.precision ||
cint(frappe.boot.sysdefaults && frappe.boot.sysdefaults.float_precision) ||
@ -105,6 +117,10 @@ frappe.form.formatters = {
</div>`;
},
Currency: function (value, docfield, options, doc) {
if (value === null) {
return "";
}
var currency = frappe.meta.get_field_currency(docfield, doc);
let precision;