refactor: install fixtures (#33964)

* refactor: install fixtures

* style: disable semgrep for install_defaults signature
This commit is contained in:
Raffael Meyer 2023-02-11 07:45:42 +01:00 committed by GitHub
parent 0efdc6c13a
commit 201573ab9a
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23

View File

@ -335,16 +335,11 @@ def install(country=None):
make_default_records() make_default_records()
make_records(records) make_records(records)
set_up_address_templates(default_country=country) set_up_address_templates(default_country=country)
set_more_defaults()
update_global_search_doctypes()
def set_more_defaults():
# Do more setup stuff that can be done here with no dependencies
update_selling_defaults() update_selling_defaults()
update_buying_defaults() update_buying_defaults()
add_uom_data() add_uom_data()
update_item_variant_settings() update_item_variant_settings()
update_global_search_doctypes()
def update_selling_defaults(): def update_selling_defaults():
@ -381,7 +376,7 @@ def add_uom_data():
) )
for d in uoms: for d in uoms:
if not frappe.db.exists("UOM", _(d.get("uom_name"))): if not frappe.db.exists("UOM", _(d.get("uom_name"))):
uom_doc = frappe.get_doc( frappe.get_doc(
{ {
"doctype": "UOM", "doctype": "UOM",
"uom_name": _(d.get("uom_name")), "uom_name": _(d.get("uom_name")),
@ -404,7 +399,7 @@ def add_uom_data():
if not frappe.db.exists( if not frappe.db.exists(
"UOM Conversion Factor", {"from_uom": _(d.get("from_uom")), "to_uom": _(d.get("to_uom"))} "UOM Conversion Factor", {"from_uom": _(d.get("from_uom")), "to_uom": _(d.get("to_uom"))}
): ):
uom_conversion = frappe.get_doc( frappe.get_doc(
{ {
"doctype": "UOM Conversion Factor", "doctype": "UOM Conversion Factor",
"category": _(d.get("category")), "category": _(d.get("category")),
@ -412,7 +407,7 @@ def add_uom_data():
"to_uom": _(d.get("to_uom")), "to_uom": _(d.get("to_uom")),
"value": d.get("value"), "value": d.get("value"),
} }
).insert(ignore_permissions=True) ).db_insert()
def add_market_segments(): def add_market_segments():
@ -468,7 +463,7 @@ def install_company(args):
make_records(records) make_records(records)
def install_defaults(args=None): def install_defaults(args=None): # nosemgrep
records = [ records = [
# Price Lists # Price Lists
{ {
@ -493,7 +488,7 @@ def install_defaults(args=None):
# enable default currency # enable default currency
frappe.db.set_value("Currency", args.get("currency"), "enabled", 1) frappe.db.set_value("Currency", args.get("currency"), "enabled", 1)
frappe.db.set_value("Stock Settings", None, "email_footer_address", args.get("company_name")) frappe.db.set_single_value("Stock Settings", "email_footer_address", args.get("company_name"))
set_global_defaults(args) set_global_defaults(args)
update_stock_settings() update_stock_settings()