Merge pull request #43930 from lbrandh/fix_timestrings

fix: use get_datetime_as_string with correct time format
This commit is contained in:
ruthra kumar
2024-12-13 12:07:56 +05:30
committed by GitHub
2 changed files with 3 additions and 4 deletions

View File

@@ -356,7 +356,7 @@ var calculate_end_time = function (frm, cdt, cdn) {
if (child.hours) {
d.add(child.hours, "hours");
frm._setting_hours = true;
frappe.model.set_value(cdt, cdn, "to_time", d.format(frappe.defaultDatetimeFormat)).then(() => {
frappe.model.set_value(cdt, cdn, "to_time", frappe.datetime.get_datetime_as_string(d)).then(() => {
frm._setting_hours = false;
});
}

View File

@@ -89,7 +89,7 @@ erpnext.timesheet.control_timer = function (frm, dialog, row, timestamp = 0) {
let d = moment(row.from_time);
if (row.expected_hours) {
d.add(row.expected_hours, "hours");
row.to_time = d.format(frappe.defaultDatetimeFormat);
row.to_time = frappe.datetime.get_datetime_as_string(d);
}
frm.refresh_field("time_logs");
frm.save();
@@ -117,8 +117,7 @@ erpnext.timesheet.control_timer = function (frm, dialog, row, timestamp = 0) {
grid_row.doc.project = args.project;
grid_row.doc.task = args.task;
grid_row.doc.expected_hours = args.expected_hours;
grid_row.doc.hours = currentIncrement / 3600;
grid_row.doc.to_time = frappe.datetime.now_datetime();
grid_row.doc.to_time = frappe.datetime.get_datetime_as_string();
grid_row.refresh();
frm.dirty();
frm.save();