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

fix: round seconds to minutes in Duration (#34020)

Browse source
This commit is contained in:
Raffael Meyer 2025-09-17 16:15:40 +02:00 • committed by GitHub
parent 391f08e63b
commit 2955e65525
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
2 changed files with 19 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

18
cypress/integration/utils.js
View file

@ -49,6 +49,24 @@ context("Utils", () => {
seconds: 0,
});
});
run_util("seconds_to_duration", 60 * 60, { hide_seconds: 1 }).then((duration) => {
expect(duration).to.deep.equal({
days: 0,
hours: 1,
minutes: 0,
seconds: 0,
});
});
run_util("seconds_to_duration", 15 * 60, { hide_seconds: 1 }).then((duration) => {
expect(duration).to.deep.equal({
days: 0,
hours: 0,
minutes: 15,
seconds: 0,
});
});
});
it("should parse days, hours, minutes and seconds", () => {

4
frappe/public/js/frappe/utils/utils.js
View file

@ -1144,8 +1144,6 @@ Object.assign(frappe.utils, {
seconds_to_duration(seconds, duration_options) {
const floor = seconds > 0 ? Math.floor : Math.ceil;
const round_base_60 = (seconds) => floor(seconds / 60 + (seconds > 0 ? 0.5 : -0.5));
const total_duration = {
days: floor(seconds / 86400), // 60 * 60 * 24
hours: floor((seconds % 86400) / 3600),
@ -1159,7 +1157,7 @@ Object.assign(frappe.utils, {
}
if (duration_options && duration_options.hide_seconds) {
total_duration.minutes += round_base_60(total_duration.seconds);
total_duration.minutes += Math.round(total_duration.seconds / 60);
total_duration.seconds = 0;
}