Merge pull request #35018 from frappe/mergify/bp/version-14/pr-35017

fix: incorrect OR condition causing timeout error (For more than 50 line items) (backport #35015) (backport #35017)
This commit is contained in:
rohitwaghchaure
2023-04-24 18:09:45 +05:30
committed by GitHub

View File

@@ -1441,6 +1441,7 @@ def get_next_stock_reco(kwargs):
( (
CombineDatetime(sle.posting_date, sle.posting_time) CombineDatetime(sle.posting_date, sle.posting_time)
> CombineDatetime(kwargs.get("posting_date"), kwargs.get("posting_time")) > CombineDatetime(kwargs.get("posting_date"), kwargs.get("posting_time"))
)
| ( | (
( (
CombineDatetime(sle.posting_date, sle.posting_time) CombineDatetime(sle.posting_date, sle.posting_time)
@@ -1450,7 +1451,6 @@ def get_next_stock_reco(kwargs):
) )
) )
) )
)
.orderby(CombineDatetime(sle.posting_date, sle.posting_time)) .orderby(CombineDatetime(sle.posting_date, sle.posting_time))
.orderby(sle.creation) .orderby(sle.creation)
.limit(1) .limit(1)