mirror of
https://github.com/frappe/erpnext.git
synced 2026-03-24 05:32:46 +01:00
perf: optimize company monthly sales query using date range (#48942)
* perf: optimize company monthly sales query using date range instead of DATE_FORMAT
* perf: optimize company monthly sales query using date range
(cherry picked from commit 4ede97ae2b)
This commit is contained in:
@@ -11,7 +11,7 @@ from frappe.cache_manager import clear_defaults_cache
|
|||||||
from frappe.contacts.address_and_contact import load_address_and_contact
|
from frappe.contacts.address_and_contact import load_address_and_contact
|
||||||
from frappe.custom.doctype.property_setter.property_setter import make_property_setter
|
from frappe.custom.doctype.property_setter.property_setter import make_property_setter
|
||||||
from frappe.desk.page.setup_wizard.setup_wizard import make_records
|
from frappe.desk.page.setup_wizard.setup_wizard import make_records
|
||||||
from frappe.utils import cint, formatdate, get_link_to_form, get_timestamp, today
|
from frappe.utils import add_months, cint, formatdate, get_first_day, get_link_to_form, get_timestamp, today
|
||||||
from frappe.utils.nestedset import NestedSet, rebuild_tree
|
from frappe.utils.nestedset import NestedSet, rebuild_tree
|
||||||
|
|
||||||
from erpnext.accounts.doctype.account.account import get_account_currency
|
from erpnext.accounts.doctype.account.account import get_account_currency
|
||||||
@@ -614,27 +614,29 @@ def install_country_fixtures(company, country):
|
|||||||
|
|
||||||
|
|
||||||
def update_company_current_month_sales(company):
|
def update_company_current_month_sales(company):
|
||||||
current_month_year = formatdate(today(), "MM-yyyy")
|
from_date = get_first_day(today())
|
||||||
|
to_date = get_first_day(add_months(from_date, 1))
|
||||||
|
|
||||||
results = frappe.db.sql(
|
results = frappe.db.sql(
|
||||||
f"""
|
"""
|
||||||
SELECT
|
SELECT
|
||||||
SUM(base_grand_total) AS total,
|
SUM(base_grand_total) AS total,
|
||||||
DATE_FORMAT(`posting_date`, '%m-%Y') AS month_year
|
DATE_FORMAT(posting_date, '%%m-%%Y') AS month_year
|
||||||
FROM
|
FROM
|
||||||
`tabSales Invoice`
|
`tabSales Invoice`
|
||||||
WHERE
|
WHERE
|
||||||
DATE_FORMAT(`posting_date`, '%m-%Y') = '{current_month_year}'
|
posting_date >= %s
|
||||||
|
AND posting_date < %s
|
||||||
AND docstatus = 1
|
AND docstatus = 1
|
||||||
AND company = {frappe.db.escape(company)}
|
AND company = %s
|
||||||
GROUP BY
|
GROUP BY
|
||||||
month_year
|
month_year
|
||||||
""",
|
""",
|
||||||
|
(from_date, to_date, company),
|
||||||
as_dict=True,
|
as_dict=True,
|
||||||
)
|
)
|
||||||
|
|
||||||
monthly_total = results[0]["total"] if len(results) > 0 else 0
|
monthly_total = results[0]["total"] if len(results) > 0 else 0
|
||||||
|
|
||||||
frappe.db.set_value("Company", company, "total_monthly_sales", monthly_total)
|
frappe.db.set_value("Company", company, "total_monthly_sales", monthly_total)
|
||||||
|
|
||||||
|
|
||||||
|
|||||||
Reference in New Issue
Block a user