fix: use CombineDatetime instead of Timestamp in QB queries
This commit is contained in:
parent
5c75894065
commit
91a398a191
@ -6,7 +6,7 @@ import frappe
|
|||||||
from frappe import _
|
from frappe import _
|
||||||
from frappe.model.document import Document
|
from frappe.model.document import Document
|
||||||
from frappe.model.naming import make_autoname, revert_series_if_last
|
from frappe.model.naming import make_autoname, revert_series_if_last
|
||||||
from frappe.query_builder.functions import CurDate, Sum, Timestamp
|
from frappe.query_builder.functions import CombineDatetime, CurDate, Sum
|
||||||
from frappe.utils import cint, flt, get_link_to_form, nowtime
|
from frappe.utils import cint, flt, get_link_to_form, nowtime
|
||||||
from frappe.utils.data import add_days
|
from frappe.utils.data import add_days
|
||||||
from frappe.utils.jinja import render_template
|
from frappe.utils.jinja import render_template
|
||||||
@ -192,7 +192,8 @@ def get_batch_qty(
|
|||||||
posting_time = nowtime()
|
posting_time = nowtime()
|
||||||
|
|
||||||
query = query.where(
|
query = query.where(
|
||||||
Timestamp(sle.posting_date, sle.posting_time) <= Timestamp(posting_date, posting_time)
|
CombineDatetime(sle.posting_date, sle.posting_time)
|
||||||
|
<= CombineDatetime(posting_date, posting_time)
|
||||||
)
|
)
|
||||||
|
|
||||||
out = query.run(as_list=True)[0][0] or 0
|
out = query.run(as_list=True)[0][0] or 0
|
||||||
|
|||||||
@ -5,7 +5,7 @@
|
|||||||
import frappe
|
import frappe
|
||||||
from frappe import _
|
from frappe import _
|
||||||
from frappe.query_builder import Field
|
from frappe.query_builder import Field
|
||||||
from frappe.query_builder.functions import Min, Timestamp
|
from frappe.query_builder.functions import CombineDatetime, Min
|
||||||
from frappe.utils import add_days, getdate, today
|
from frappe.utils import add_days, getdate, today
|
||||||
|
|
||||||
import erpnext
|
import erpnext
|
||||||
@ -75,7 +75,7 @@ def get_data(report_filters):
|
|||||||
& (sle.company == report_filters.company)
|
& (sle.company == report_filters.company)
|
||||||
& (sle.is_cancelled == 0)
|
& (sle.is_cancelled == 0)
|
||||||
)
|
)
|
||||||
.orderby(Timestamp(sle.posting_date, sle.posting_time), sle.creation)
|
.orderby(CombineDatetime(sle.posting_date, sle.posting_time), sle.creation)
|
||||||
).run(as_dict=True)
|
).run(as_dict=True)
|
||||||
|
|
||||||
for d in data:
|
for d in data:
|
||||||
|
|||||||
Loading…
x
Reference in New Issue
Block a user