Reduces 1 pointless function call.
Function calls are also not "super cheap" in python.
(cherry picked from commit 2eec621e95
)
[skip ci]
Co-authored-by: Ankush Menat <ankush@frappe.io>
version-14
@@ -2329,7 +2329,7 @@ def get_version(doctype, name, limit=None, head=False, raise_err=True): | |||||
""" | """ | ||||
meta = get_meta(doctype) | meta = get_meta(doctype) | ||||
if meta.track_changes: | if meta.track_changes: | ||||
names = db.get_all( | |||||
names = get_all( | |||||
"Version", | "Version", | ||||
filters={ | filters={ | ||||
"ref_doctype": doctype, | "ref_doctype": doctype, | ||||
@@ -241,7 +241,7 @@ class AutoRepeat(Document): | |||||
def set_auto_repeat_period(self, new_doc): | def set_auto_repeat_period(self, new_doc): | ||||
mcount = month_map.get(self.frequency) | mcount = month_map.get(self.frequency) | ||||
if mcount and new_doc.meta.get_field("from_date") and new_doc.meta.get_field("to_date"): | if mcount and new_doc.meta.get_field("from_date") and new_doc.meta.get_field("to_date"): | ||||
last_ref_doc = frappe.db.get_all( | |||||
last_ref_doc = frappe.get_all( | |||||
doctype=self.reference_doctype, | doctype=self.reference_doctype, | ||||
fields=["name", "from_date", "to_date"], | fields=["name", "from_date", "to_date"], | ||||
filters=[ | filters=[ | ||||
@@ -470,7 +470,7 @@ def create_repeated_entries(data): | |||||
def get_auto_repeat_entries(date=None): | def get_auto_repeat_entries(date=None): | ||||
if not date: | if not date: | ||||
date = getdate(today()) | date = getdate(today()) | ||||
return frappe.db.get_all( | |||||
return frappe.get_all( | |||||
"Auto Repeat", filters=[["next_schedule_date", "<=", date], ["status", "=", "Active"]] | "Auto Repeat", filters=[["next_schedule_date", "<=", date], ["status", "=", "Active"]] | ||||
) | ) | ||||
@@ -504,7 +504,7 @@ def make_auto_repeat(doctype, docname, frequency="Daily", start_date=None, end_d | |||||
@frappe.whitelist() | @frappe.whitelist() | ||||
@frappe.validate_and_sanitize_search_inputs | @frappe.validate_and_sanitize_search_inputs | ||||
def get_auto_repeat_doctypes(doctype, txt, searchfield, start, page_len, filters): | def get_auto_repeat_doctypes(doctype, txt, searchfield, start, page_len, filters): | ||||
res = frappe.db.get_all( | |||||
res = frappe.get_all( | |||||
"Property Setter", | "Property Setter", | ||||
{ | { | ||||
"property": "allow_auto_repeat", | "property": "allow_auto_repeat", | ||||
@@ -514,7 +514,7 @@ def get_auto_repeat_doctypes(doctype, txt, searchfield, start, page_len, filters | |||||
) | ) | ||||
docs = [r.doc_type for r in res] | docs = [r.doc_type for r in res] | ||||
res = frappe.db.get_all( | |||||
res = frappe.get_all( | |||||
"DocType", | "DocType", | ||||
{ | { | ||||
"allow_auto_repeat": 1, | "allow_auto_repeat": 1, | ||||
@@ -228,7 +228,7 @@ class TestAutoRepeat(unittest.TestCase): | |||||
data = get_auto_repeat_entries(current_date) | data = get_auto_repeat_entries(current_date) | ||||
create_repeated_entries(data) | create_repeated_entries(data) | ||||
docnames = frappe.db.get_all( | |||||
docnames = frappe.get_all( | |||||
doc.reference_doctype, filters={"auto_repeat": doc.name}, fields=["docstatus"], limit=1 | doc.reference_doctype, filters={"auto_repeat": doc.name}, fields=["docstatus"], limit=1 | ||||
) | ) | ||||
self.assertEqual(docnames[0].docstatus, 1) | self.assertEqual(docnames[0].docstatus, 1) | ||||
@@ -336,7 +336,7 @@ def get_success_action(): | |||||
def get_link_preview_doctypes(): | def get_link_preview_doctypes(): | ||||
from frappe.utils import cint | from frappe.utils import cint | ||||
link_preview_doctypes = [d.name for d in frappe.db.get_all("DocType", {"show_preview_popup": 1})] | |||||
link_preview_doctypes = [d.name for d in frappe.get_all("DocType", {"show_preview_popup": 1})] | |||||
customizations = frappe.get_all( | customizations = frappe.get_all( | ||||
"Property Setter", fields=["doc_type", "value"], filters={"property": "show_preview_popup"} | "Property Setter", fields=["doc_type", "value"], filters={"property": "show_preview_popup"} | ||||
) | ) | ||||
@@ -134,7 +134,7 @@ def clear_doctype_cache(doctype=None): | |||||
# clear all parent doctypes | # clear all parent doctypes | ||||
for dt in frappe.db.get_all( | |||||
for dt in frappe.get_all( | |||||
"DocField", "parent", dict(fieldtype=["in", frappe.model.table_fields], options=doctype) | "DocField", "parent", dict(fieldtype=["in", frappe.model.table_fields], options=doctype) | ||||
): | ): | ||||
clear_single(dt.parent) | clear_single(dt.parent) | ||||
@@ -41,7 +41,7 @@ class TestActivityLog(unittest.TestCase): | |||||
frappe.local.form_dict = frappe._dict() | frappe.local.form_dict = frappe._dict() | ||||
def get_auth_log(self, operation="Login"): | def get_auth_log(self, operation="Login"): | ||||
names = frappe.db.get_all( | |||||
names = frappe.get_all( | |||||
"Activity Log", | "Activity Log", | ||||
filters={ | filters={ | ||||
"user": "Administrator", | "user": "Administrator", | ||||
@@ -183,7 +183,7 @@ class Exporter: | |||||
child_fields = ["name", "idx", "parent", "parentfield"] + list( | child_fields = ["name", "idx", "parent", "parentfield"] + list( | ||||
{format_column_name(df) for df in self.fields if df.parent == child_table_doctype} | {format_column_name(df) for df in self.fields if df.parent == child_table_doctype} | ||||
) | ) | ||||
data = frappe.db.get_all( | |||||
data = frappe.get_all( | |||||
child_table_doctype, | child_table_doctype, | ||||
filters={ | filters={ | ||||
"parent": ("in", parent_names), | "parent": ("in", parent_names), | ||||
@@ -50,7 +50,7 @@ class Importer: | |||||
def get_data_for_import_preview(self): | def get_data_for_import_preview(self): | ||||
out = self.import_file.get_data_for_import_preview() | out = self.import_file.get_data_for_import_preview() | ||||
out.import_log = frappe.db.get_all( | |||||
out.import_log = frappe.get_all( | |||||
"Data Import Log", | "Data Import Log", | ||||
fields=["row_indexes", "success"], | fields=["row_indexes", "success"], | ||||
filters={"data_import": self.data_import.name}, | filters={"data_import": self.data_import.name}, | ||||
@@ -90,7 +90,7 @@ class Importer: | |||||
# setup import log | # setup import log | ||||
import_log = ( | import_log = ( | ||||
frappe.db.get_all( | |||||
frappe.get_all( | |||||
"Data Import Log", | "Data Import Log", | ||||
fields=["row_indexes", "success", "log_index"], | fields=["row_indexes", "success", "log_index"], | ||||
filters={"data_import": self.data_import.name}, | filters={"data_import": self.data_import.name}, | ||||
@@ -204,7 +204,7 @@ class Importer: | |||||
# Logs are db inserted directly so will have to be fetched again | # Logs are db inserted directly so will have to be fetched again | ||||
import_log = ( | import_log = ( | ||||
frappe.db.get_all( | |||||
frappe.get_all( | |||||
"Data Import Log", | "Data Import Log", | ||||
fields=["row_indexes", "success", "log_index"], | fields=["row_indexes", "success", "log_index"], | ||||
filters={"data_import": self.data_import.name}, | filters={"data_import": self.data_import.name}, | ||||
@@ -297,7 +297,7 @@ class Importer: | |||||
return | return | ||||
import_log = ( | import_log = ( | ||||
frappe.db.get_all( | |||||
frappe.get_all( | |||||
"Data Import Log", | "Data Import Log", | ||||
fields=["row_indexes", "success"], | fields=["row_indexes", "success"], | ||||
filters={"data_import": self.data_import.name}, | filters={"data_import": self.data_import.name}, | ||||
@@ -327,7 +327,7 @@ class Importer: | |||||
if not self.data_import: | if not self.data_import: | ||||
return | return | ||||
import_log = frappe.db.get_all( | |||||
import_log = frappe.get_all( | |||||
"Data Import Log", | "Data Import Log", | ||||
fields=["row_indexes", "success", "messages", "exception", "docname"], | fields=["row_indexes", "success", "messages", "exception", "docname"], | ||||
filters={"data_import": self.data_import.name}, | filters={"data_import": self.data_import.name}, | ||||
@@ -985,7 +985,7 @@ class Column: | |||||
# find all values that dont exist | # find all values that dont exist | ||||
values = list({cstr(v) for v in self.column_values[1:] if v}) | values = list({cstr(v) for v in self.column_values[1:] if v}) | ||||
exists = [ | exists = [ | ||||
cstr(d.name) for d in frappe.db.get_all(self.df.options, filters={"name": ("in", values)}) | |||||
cstr(d.name) for d in frappe.get_all(self.df.options, filters={"name": ("in", values)}) | |||||
] | ] | ||||
not_exists = list(set(values) - set(exists)) | not_exists = list(set(values) - set(exists)) | ||||
if not_exists: | if not_exists: | ||||
@@ -67,7 +67,7 @@ class TestImporter(unittest.TestCase): | |||||
data_import.start_import() | data_import.start_import() | ||||
data_import.reload() | data_import.reload() | ||||
import_log = frappe.db.get_all( | |||||
import_log = frappe.get_all( | |||||
"Data Import Log", | "Data Import Log", | ||||
fields=["row_indexes", "success", "messages", "exception", "docname"], | fields=["row_indexes", "success", "messages", "exception", "docname"], | ||||
filters={"data_import": data_import.name}, | filters={"data_import": data_import.name}, | ||||
@@ -325,7 +325,7 @@ class DocType(Document): | |||||
"""Change the timestamp of parent DocType if the current one is a child to clear caches.""" | """Change the timestamp of parent DocType if the current one is a child to clear caches.""" | ||||
if frappe.flags.in_import: | if frappe.flags.in_import: | ||||
return | return | ||||
parent_list = frappe.db.get_all( | |||||
parent_list = frappe.get_all( | |||||
"DocField", "parent", dict(fieldtype=["in", frappe.model.table_fields], options=self.name) | "DocField", "parent", dict(fieldtype=["in", frappe.model.table_fields], options=self.name) | ||||
) | ) | ||||
for p in parent_list: | for p in parent_list: | ||||
@@ -239,7 +239,7 @@ class TestFile(FrappeTestCase): | |||||
pass | pass | ||||
def delete_test_data(self): | def delete_test_data(self): | ||||
test_file_data = frappe.db.get_all( | |||||
test_file_data = frappe.get_all( | |||||
"File", | "File", | ||||
pluck="name", | pluck="name", | ||||
filters={"is_home_folder": 0, "is_attachments_folder": 0}, | filters={"is_home_folder": 0, "is_attachments_folder": 0}, | ||||
@@ -30,7 +30,7 @@ def validate_with_regex(name, label): | |||||
def export_languages_json(): | def export_languages_json(): | ||||
"""Export list of all languages""" | """Export list of all languages""" | ||||
languages = frappe.db.get_all("Language", fields=["name", "language_name"]) | |||||
languages = frappe.get_all("Language", fields=["name", "language_name"]) | |||||
languages = [{"name": d.language_name, "code": d.name} for d in languages] | languages = [{"name": d.language_name, "code": d.name} for d in languages] | ||||
languages.sort(key=lambda a: a["code"]) | languages.sort(key=lambda a: a["code"]) | ||||
@@ -60,7 +60,7 @@ class PackageRelease(Document): | |||||
self.make_tarfile(package) | self.make_tarfile(package) | ||||
def export_modules(self): | def export_modules(self): | ||||
for m in frappe.db.get_all("Module Def", dict(package=self.package)): | |||||
for m in frappe.get_all("Module Def", dict(package=self.package)): | |||||
module = frappe.get_doc("Module Def", m.name) | module = frappe.get_doc("Module Def", m.name) | ||||
for l in module.meta.links: | for l in module.meta.links: | ||||
if l.link_doctype == "Module Def": | if l.link_doctype == "Module Def": | ||||
@@ -129,7 +129,7 @@ class ServerScript(Document): | |||||
Returns: | Returns: | ||||
list: Returns list of autocompletion items. | list: Returns list of autocompletion items. | ||||
For e.g., ["frappe.utils.cint", "frappe.db.get_all", ...] | |||||
For e.g., ["frappe.utils.cint", "frappe.get_all", ...] | |||||
""" | """ | ||||
def get_keys(obj): | def get_keys(obj): | ||||
@@ -39,7 +39,7 @@ def create_translations(translation_map, language): | |||||
# first create / update local user translations | # first create / update local user translations | ||||
for source_id, translation_dict in translation_map.items(): | for source_id, translation_dict in translation_map.items(): | ||||
translation_dict = frappe._dict(translation_dict) | translation_dict = frappe._dict(translation_dict) | ||||
existing_doc_name = frappe.db.get_all( | |||||
existing_doc_name = frappe.get_all( | |||||
"Translation", | "Translation", | ||||
{ | { | ||||
"source_text": translation_dict.source_text, | "source_text": translation_dict.source_text, | ||||
@@ -621,7 +621,7 @@ class User(Document): | |||||
if login_with_username: | if login_with_username: | ||||
or_filters.append({"username": user_name}) | or_filters.append({"username": user_name}) | ||||
users = frappe.db.get_all("User", fields=["name", "enabled"], or_filters=or_filters, limit=1) | |||||
users = frappe.get_all("User", fields=["name", "enabled"], or_filters=or_filters, limit=1) | |||||
if not users: | if not users: | ||||
return | return | ||||
@@ -27,7 +27,7 @@ class UserPermission(Document): | |||||
def validate_user_permission(self): | def validate_user_permission(self): | ||||
"""checks for duplicate user permission records""" | """checks for duplicate user permission records""" | ||||
duplicate_exists = frappe.db.get_all( | |||||
duplicate_exists = frappe.get_all( | |||||
self.doctype, | self.doctype, | ||||
filters={ | filters={ | ||||
"allow": self.allow, | "allow": self.allow, | ||||
@@ -2,7 +2,7 @@ import frappe | |||||
def execute(): | def execute(): | ||||
for web_form_name in frappe.db.get_all("Web Form", pluck="name"): | |||||
for web_form_name in frappe.get_all("Web Form", pluck="name"): | |||||
web_form = frappe.get_doc("Web Form", web_form_name) | web_form = frappe.get_doc("Web Form", web_form_name) | ||||
doctype_layout = frappe.get_doc( | doctype_layout = frappe.get_doc( | ||||
dict( | dict( | ||||
@@ -172,7 +172,7 @@ class Workspace: | |||||
if not exists and frappe.db.exists(name): | if not exists and frappe.db.exists(name): | ||||
if not frappe.db.get_value("DocType", name, "issingle"): | if not frappe.db.get_value("DocType", name, "issingle"): | ||||
exists = bool(frappe.db.get_all(name, limit=1)) | |||||
exists = bool(frappe.get_all(name, limit=1)) | |||||
else: | else: | ||||
exists = True | exists = True | ||||
self.table_counts[name] = exists | self.table_counts[name] = exists | ||||
@@ -249,7 +249,7 @@ def get_heatmap_chart_config(chart, filters, heatmap_year): | |||||
timestamp_field = f"extract(epoch from timestamp {datefield})" | timestamp_field = f"extract(epoch from timestamp {datefield})" | ||||
data = dict( | data = dict( | ||||
frappe.db.get_all( | |||||
frappe.get_all( | |||||
doctype, | doctype, | ||||
fields=[ | fields=[ | ||||
timestamp_field, | timestamp_field, | ||||
@@ -61,7 +61,7 @@ def get_desktop_icons(user=None): | |||||
blocked_doctypes = [d.get("name") for d in blocked_doctypes] | blocked_doctypes = [d.get("name") for d in blocked_doctypes] | ||||
standard_icons = frappe.db.get_all("Desktop Icon", fields=fields, filters={"standard": 1}) | |||||
standard_icons = frappe.get_all("Desktop Icon", fields=fields, filters={"standard": 1}) | |||||
standard_map = {} | standard_map = {} | ||||
for icon in standard_icons: | for icon in standard_icons: | ||||
@@ -69,7 +69,7 @@ def get_desktop_icons(user=None): | |||||
icon.blocked = 1 | icon.blocked = 1 | ||||
standard_map[icon.module_name] = icon | standard_map[icon.module_name] = icon | ||||
user_icons = frappe.db.get_all( | |||||
user_icons = frappe.get_all( | |||||
"Desktop Icon", fields=fields, filters={"standard": 0, "owner": user} | "Desktop Icon", fields=fields, filters={"standard": 0, "owner": user} | ||||
) | ) | ||||
@@ -430,7 +430,7 @@ def get_context(context): | |||||
context.user = frappe.session.user | context.user = frappe.session.user | ||||
if "System Manager" in frappe.get_roles(): | if "System Manager" in frappe.get_roles(): | ||||
context.users = frappe.db.get_all( | |||||
context.users = frappe.get_all( | |||||
"User", | "User", | ||||
filters={"user_type": "System User", "enabled": 1}, | filters={"user_type": "System User", "enabled": 1}, | ||||
fields=["name", "first_name", "last_name"], | fields=["name", "first_name", "last_name"], | ||||
@@ -443,7 +443,7 @@ def get_module_icons(user=None): | |||||
frappe.only_for("System Manager") | frappe.only_for("System Manager") | ||||
if not user: | if not user: | ||||
icons = frappe.db.get_all("Desktop Icon", fields="*", filters={"standard": 1}, order_by="idx") | |||||
icons = frappe.get_all("Desktop Icon", fields="*", filters={"standard": 1}, order_by="idx") | |||||
else: | else: | ||||
frappe.cache().hdel("desktop_icons", user) | frappe.cache().hdel("desktop_icons", user) | ||||
icons = get_user_icons(user) | icons = get_user_icons(user) | ||||
@@ -151,7 +151,7 @@ def get_notification_logs(limit=20): | |||||
@frappe.whitelist() | @frappe.whitelist() | ||||
def mark_all_as_read(): | def mark_all_as_read(): | ||||
unread_docs_list = frappe.db.get_all( | |||||
unread_docs_list = frappe.get_all( | |||||
"Notification Log", filters={"read": 0, "for_user": frappe.session.user} | "Notification Log", filters={"read": 0, "for_user": frappe.session.user} | ||||
) | ) | ||||
unread_docnames = [doc.name for doc in unread_docs_list] | unread_docnames = [doc.name for doc in unread_docs_list] | ||||
@@ -38,7 +38,7 @@ class TestNotificationLog(unittest.TestCase): | |||||
def get_last_email_queue(): | def get_last_email_queue(): | ||||
res = frappe.db.get_all("Email Queue", fields=["message"], order_by="creation desc", limit=1) | |||||
res = frappe.get_all("Email Queue", fields=["message"], order_by="creation desc", limit=1) | |||||
return res[0] | return res[0] | ||||
@@ -138,7 +138,7 @@ def add_multiple(args=None): | |||||
def close_all_assignments(doctype, name): | def close_all_assignments(doctype, name): | ||||
assignments = frappe.db.get_all( | |||||
assignments = frappe.get_all( | |||||
"ToDo", | "ToDo", | ||||
fields=["allocated_to"], | fields=["allocated_to"], | ||||
filters=dict(reference_type=doctype, reference_name=name, status=("!=", "Cancelled")), | filters=dict(reference_type=doctype, reference_name=name, status=("!=", "Cancelled")), | ||||
@@ -189,7 +189,7 @@ def clear(doctype, name): | |||||
""" | """ | ||||
Clears assignments, return False if not assigned. | Clears assignments, return False if not assigned. | ||||
""" | """ | ||||
assignments = frappe.db.get_all( | |||||
assignments = frappe.get_all( | |||||
"ToDo", fields=["allocated_to"], filters=dict(reference_type=doctype, reference_name=name) | "ToDo", fields=["allocated_to"], filters=dict(reference_type=doctype, reference_name=name) | ||||
) | ) | ||||
if not assignments: | if not assignments: | ||||
@@ -136,7 +136,7 @@ class SubmittableDocumentTree: | |||||
def get_submittable_doctypes(self) -> list[str]: | def get_submittable_doctypes(self) -> list[str]: | ||||
"""Returns list of submittable doctypes.""" | """Returns list of submittable doctypes.""" | ||||
if not self._submittable_doctypes: | if not self._submittable_doctypes: | ||||
self._submittable_doctypes = frappe.db.get_all("DocType", {"is_submittable": 1}, pluck="name") | |||||
self._submittable_doctypes = frappe.get_all("DocType", {"is_submittable": 1}, pluck="name") | |||||
return self._submittable_doctypes | return self._submittable_doctypes | ||||
@@ -288,9 +288,7 @@ def get_references_across_doctypes_by_dynamic_link_field( | |||||
for doctype, fieldname, doctype_fieldname in links: | for doctype, fieldname, doctype_fieldname in links: | ||||
try: | try: | ||||
filters = [[doctype_fieldname, "in", to_doctypes]] if to_doctypes else [] | filters = [[doctype_fieldname, "in", to_doctypes]] if to_doctypes else [] | ||||
for linked_to in frappe.db.get_all( | |||||
doctype, pluck=doctype_fieldname, filters=filters, distinct=1 | |||||
): | |||||
for linked_to in frappe.get_all(doctype, pluck=doctype_fieldname, filters=filters, distinct=1): | |||||
if linked_to: | if linked_to: | ||||
links_by_doctype[linked_to].append( | links_by_doctype[linked_to].append( | ||||
{"doctype": doctype, "fieldname": fieldname, "doctype_fieldname": doctype_fieldname} | {"doctype": doctype, "fieldname": fieldname, "doctype_fieldname": doctype_fieldname} | ||||
@@ -330,17 +328,17 @@ def get_referencing_documents( | |||||
if not link_info.get("is_child"): | if not link_info.get("is_child"): | ||||
filters.extend(parent_filters or []) | filters.extend(parent_filters or []) | ||||
return {from_table: frappe.db.get_all(from_table, filters, pluck="name")} | |||||
return {from_table: frappe.get_all(from_table, filters, pluck="name")} | |||||
filters.extend(child_filters or []) | filters.extend(child_filters or []) | ||||
res = frappe.db.get_all(from_table, filters=filters, fields=["name", "parenttype", "parent"]) | |||||
res = frappe.get_all(from_table, filters=filters, fields=["name", "parenttype", "parent"]) | |||||
documents = defaultdict(list) | documents = defaultdict(list) | ||||
for parent, rows in itertools.groupby(res, key=lambda row: row["parenttype"]): | for parent, rows in itertools.groupby(res, key=lambda row: row["parenttype"]): | ||||
if allowed_parents and parent not in allowed_parents: | if allowed_parents and parent not in allowed_parents: | ||||
continue | continue | ||||
filters = (parent_filters or []) + [["name", "in", tuple(row.parent for row in rows)]] | filters = (parent_filters or []) + [["name", "in", tuple(row.parent for row in rows)]] | ||||
documents[parent].extend(frappe.db.get_all(parent, filters=filters, pluck="name") or []) | |||||
documents[parent].extend(frappe.get_all(parent, filters=filters, pluck="name") or []) | |||||
return documents | return documents | ||||
@@ -178,7 +178,7 @@ def add_comments(doc, docinfo): | |||||
def get_milestones(doctype, name): | def get_milestones(doctype, name): | ||||
return frappe.db.get_all( | |||||
return frappe.get_all( | |||||
"Milestone", | "Milestone", | ||||
fields=["creation", "owner", "track_field", "value"], | fields=["creation", "owner", "track_field", "value"], | ||||
filters=dict(reference_type=doctype, reference_name=name), | filters=dict(reference_type=doctype, reference_name=name), | ||||
@@ -249,7 +249,7 @@ def get_comments( | |||||
def get_point_logs(doctype, docname): | def get_point_logs(doctype, docname): | ||||
return frappe.db.get_all( | |||||
return frappe.get_all( | |||||
"Energy Point Log", | "Energy Point Log", | ||||
filters={"reference_doctype": doctype, "reference_name": docname, "type": ["!=", "Review"]}, | filters={"reference_doctype": doctype, "reference_name": docname, "type": ["!=", "Review"]}, | ||||
fields=["*"], | fields=["*"], | ||||
@@ -146,7 +146,7 @@ class FormMeta(Meta): | |||||
"""embed all require files""" | """embed all require files""" | ||||
# custom script | # custom script | ||||
client_scripts = ( | client_scripts = ( | ||||
frappe.db.get_all( | |||||
frappe.get_all( | |||||
"Client Script", | "Client Script", | ||||
filters={"dt": self.name, "enabled": 1}, | filters={"dt": self.name, "enabled": 1}, | ||||
fields=["name", "script", "view"], | fields=["name", "script", "view"], | ||||
@@ -16,7 +16,7 @@ def get_leaderboards(): | |||||
@frappe.whitelist() | @frappe.whitelist() | ||||
def get_energy_point_leaderboard(date_range, company=None, field=None, limit=None): | def get_energy_point_leaderboard(date_range, company=None, field=None, limit=None): | ||||
all_users = frappe.db.get_all( | |||||
all_users = frappe.get_all( | |||||
"User", | "User", | ||||
filters={ | filters={ | ||||
"name": ["not in", ["Administrator", "Guest"]], | "name": ["not in", ["Administrator", "Guest"]], | ||||
@@ -31,7 +31,7 @@ def get_energy_point_leaderboard(date_range, company=None, field=None, limit=Non | |||||
if date_range: | if date_range: | ||||
date_range = frappe.parse_json(date_range) | date_range = frappe.parse_json(date_range) | ||||
filters.append(["creation", "between", [date_range[0], date_range[1]]]) | filters.append(["creation", "between", [date_range[0], date_range[1]]]) | ||||
energy_point_users = frappe.db.get_all( | |||||
energy_point_users = frappe.get_all( | |||||
"Energy Point Log", | "Energy Point Log", | ||||
fields=["user as name", "sum(points) as value"], | fields=["user as name", "sum(points) as value"], | ||||
filters=filters, | filters=filters, | ||||
@@ -31,7 +31,7 @@ def get_energy_points_heatmap_data(user, date): | |||||
@frappe.whitelist() | @frappe.whitelist() | ||||
def get_energy_points_percentage_chart_data(user, field): | def get_energy_points_percentage_chart_data(user, field): | ||||
result = frappe.db.get_all( | |||||
result = frappe.get_all( | |||||
"Energy Point Log", | "Energy Point Log", | ||||
filters={"user": user, "type": ["!=", "Review"]}, | filters={"user": user, "type": ["!=", "Review"]}, | ||||
group_by=field, | group_by=field, | ||||
@@ -49,7 +49,7 @@ def get_energy_points_percentage_chart_data(user, field): | |||||
@frappe.whitelist() | @frappe.whitelist() | ||||
def get_user_rank(user): | def get_user_rank(user): | ||||
month_start = datetime.today().replace(day=1) | month_start = datetime.today().replace(day=1) | ||||
monthly_rank = frappe.db.get_all( | |||||
monthly_rank = frappe.get_all( | |||||
"Energy Point Log", | "Energy Point Log", | ||||
group_by="user", | group_by="user", | ||||
filters={"creation": [">", month_start], "type": ["!=", "Review"]}, | filters={"creation": [">", month_start], "type": ["!=", "Review"]}, | ||||
@@ -58,7 +58,7 @@ def get_user_rank(user): | |||||
as_list=True, | as_list=True, | ||||
) | ) | ||||
all_time_rank = frappe.db.get_all( | |||||
all_time_rank = frappe.get_all( | |||||
"Energy Point Log", | "Energy Point Log", | ||||
group_by="user", | group_by="user", | ||||
filters={"type": ["!=", "Review"]}, | filters={"type": ["!=", "Review"]}, | ||||
@@ -846,7 +846,7 @@ def get_max_email_uid(email_account): | |||||
# get maximum uid of emails | # get maximum uid of emails | ||||
max_uid = 1 | max_uid = 1 | ||||
result = frappe.db.get_all( | |||||
result = frappe.get_all( | |||||
"Communication", | "Communication", | ||||
filters={ | filters={ | ||||
"communication_medium": "Email", | "communication_medium": "Email", | ||||
@@ -9,10 +9,10 @@ from frappe.utils import parse_addr, validate_email_address | |||||
class EmailGroup(Document): | class EmailGroup(Document): | ||||
def onload(self): | def onload(self): | ||||
singles = [d.name for d in frappe.db.get_all("DocType", "name", {"issingle": 1})] | |||||
singles = [d.name for d in frappe.get_all("DocType", "name", {"issingle": 1})] | |||||
self.get("__onload").import_types = [ | self.get("__onload").import_types = [ | ||||
{"value": d.parent, "label": f"{d.parent} ({d.label})"} | {"value": d.parent, "label": f"{d.parent} ({d.label})"} | ||||
for d in frappe.db.get_all("DocField", ("parent", "label"), {"options": "Email"}) | |||||
for d in frappe.get_all("DocField", ("parent", "label"), {"options": "Email"}) | |||||
if d.parent not in singles | if d.parent not in singles | ||||
] | ] | ||||
@@ -27,7 +27,7 @@ class EmailGroup(Document): | |||||
unsubscribed_field = "unsubscribed" if meta.get_field("unsubscribed") else None | unsubscribed_field = "unsubscribed" if meta.get_field("unsubscribed") else None | ||||
added = 0 | added = 0 | ||||
for user in frappe.db.get_all(doctype, [email_field, unsubscribed_field or "name"]): | |||||
for user in frappe.get_all(doctype, [email_field, unsubscribed_field or "name"]): | |||||
try: | try: | ||||
email = parse_addr(user.get(email_field))[1] if user.get(email_field) else None | email = parse_addr(user.get(email_field))[1] if user.get(email_field) else None | ||||
if email: | if email: | ||||
@@ -107,7 +107,7 @@ class TestNewsletterMixin: | |||||
"content_type": "Rich Text", | "content_type": "Rich Text", | ||||
"message": "Testing my news.", | "message": "Testing my news.", | ||||
} | } | ||||
similar_newsletters = frappe.db.get_all(doctype, newsletter_content, pluck="name") | |||||
similar_newsletters = frappe.get_all(doctype, newsletter_content, pluck="name") | |||||
for similar_newsletter in similar_newsletters: | for similar_newsletter in similar_newsletters: | ||||
frappe.delete_doc(doctype, similar_newsletter) | frappe.delete_doc(doctype, similar_newsletter) | ||||
@@ -180,7 +180,7 @@ class TestNewsletter(TestNewsletterMixin, FrappeTestCase): | |||||
newsletter.save = MagicMock() | newsletter.save = MagicMock() | ||||
self.assertFalse(newsletter.save.called) | self.assertFalse(newsletter.save.called) | ||||
# check if the test email is in the queue | # check if the test email is in the queue | ||||
email_queue = frappe.db.get_all( | |||||
email_queue = frappe.get_all( | |||||
"Email Queue", | "Email Queue", | ||||
filters=[ | filters=[ | ||||
["reference_doctype", "=", "Newsletter"], | ["reference_doctype", "=", "Newsletter"], | ||||
@@ -176,7 +176,7 @@ class TestWebhook(unittest.TestCase): | |||||
webhook = frappe.get_doc("Webhook", {"webhook_doctype": "User"}) | webhook = frappe.get_doc("Webhook", {"webhook_doctype": "User"}) | ||||
enqueue_webhook(user, webhook) | enqueue_webhook(user, webhook) | ||||
self.assertTrue(frappe.db.get_all("Webhook Request Log", pluck="name")) | |||||
self.assertTrue(frappe.get_all("Webhook Request Log", pluck="name")) | |||||
def test_webhook_with_array_body(self): | def test_webhook_with_array_body(self): | ||||
"""Check if array request body are supported.""" | """Check if array request body are supported.""" | ||||
@@ -666,7 +666,7 @@ def is_single(doctype): | |||||
def get_parent_dt(dt): | def get_parent_dt(dt): | ||||
parent_dt = frappe.db.get_all( | |||||
parent_dt = frappe.get_all( | |||||
"DocField", "parent", dict(fieldtype=["in", frappe.model.table_fields], options=dt), limit=1 | "DocField", "parent", dict(fieldtype=["in", frappe.model.table_fields], options=dt), limit=1 | ||||
) | ) | ||||
return parent_dt and parent_dt[0].parent or "" | return parent_dt and parent_dt[0].parent or "" | ||||
@@ -764,7 +764,7 @@ def trim_tables(doctype=None, dry_run=False, quiet=False): | |||||
if doctype: | if doctype: | ||||
filters["name"] = doctype | filters["name"] = doctype | ||||
for doctype in frappe.db.get_all("DocType", filters=filters, pluck="name"): | |||||
for doctype in frappe.get_all("DocType", filters=filters, pluck="name"): | |||||
try: | try: | ||||
dropped_columns = trim_table(doctype, dry_run=dry_run) | dropped_columns = trim_table(doctype, dry_run=dry_run) | ||||
if dropped_columns: | if dropped_columns: | ||||
@@ -237,7 +237,7 @@ def update_assignments(old: str, new: str, doctype: str) -> None: | |||||
for user in common_assignments: | for user in common_assignments: | ||||
# delete todos linked to old doc | # delete todos linked to old doc | ||||
todos = frappe.db.get_all( | |||||
todos = frappe.get_all( | |||||
"ToDo", | "ToDo", | ||||
{ | { | ||||
"owner": user, | "owner": user, | ||||
@@ -7,7 +7,7 @@ def execute(): | |||||
if not frappe.db.table_exists("Dashboard Chart"): | if not frappe.db.table_exists("Dashboard Chart"): | ||||
return | return | ||||
charts_to_modify = frappe.db.get_all( | |||||
charts_to_modify = frappe.get_all( | |||||
"Dashboard Chart", | "Dashboard Chart", | ||||
fields=["name", "filters_json", "document_type"], | fields=["name", "filters_json", "document_type"], | ||||
filters={"chart_type": ["not in", ["Report", "Custom"]]}, | filters={"chart_type": ["not in", ["Report", "Custom"]]}, | ||||
@@ -8,6 +8,6 @@ def execute(): | |||||
frappe.reload_doc("desk", "doctype", "notification_settings") | frappe.reload_doc("desk", "doctype", "notification_settings") | ||||
frappe.reload_doc("desk", "doctype", "notification_subscribed_document") | frappe.reload_doc("desk", "doctype", "notification_subscribed_document") | ||||
users = frappe.db.get_all("User", fields=["name"]) | |||||
users = frappe.get_all("User", fields=["name"]) | |||||
for user in users: | for user in users: | ||||
create_notification_settings(user.name) | create_notification_settings(user.name) |
@@ -15,9 +15,7 @@ def execute(): | |||||
def setup_incoming_email_port_in_email_domains(): | def setup_incoming_email_port_in_email_domains(): | ||||
email_domains = frappe.db.get_all( | |||||
"Email Domain", ["incoming_port", "use_imap", "use_ssl", "name"] | |||||
) | |||||
email_domains = frappe.get_all("Email Domain", ["incoming_port", "use_imap", "use_ssl", "name"]) | |||||
for domain in email_domains: | for domain in email_domains: | ||||
if not domain.incoming_port: | if not domain.incoming_port: | ||||
incoming_port = get_port(domain) | incoming_port = get_port(domain) | ||||
@@ -33,7 +31,7 @@ def setup_incoming_email_port_in_email_domains(): | |||||
def setup_incoming_email_port_in_email_accounts(): | def setup_incoming_email_port_in_email_accounts(): | ||||
email_accounts = frappe.db.get_all( | |||||
email_accounts = frappe.get_all( | |||||
"Email Account", ["incoming_port", "use_imap", "use_ssl", "name", "enable_incoming"] | "Email Account", ["incoming_port", "use_imap", "use_ssl", "name", "enable_incoming"] | ||||
) | ) | ||||
@@ -6,7 +6,7 @@ import frappe | |||||
def execute(): | def execute(): | ||||
frappe.reload_doc("website", "doctype", "website_theme_ignore_app") | frappe.reload_doc("website", "doctype", "website_theme_ignore_app") | ||||
themes = frappe.db.get_all( | |||||
themes = frappe.get_all( | |||||
"Website Theme", filters={"theme_url": ("not like", "/files/website_theme/%")} | "Website Theme", filters={"theme_url": ("not like", "/files/website_theme/%")} | ||||
) | ) | ||||
for theme in themes: | for theme in themes: | ||||
@@ -15,7 +15,7 @@ def execute(): | |||||
) | ) | ||||
users = [item.parent for item in users_with_permission] | users = [item.parent for item in users_with_permission] | ||||
charts = frappe.db.get_all("Dashboard Chart", filters={"owner": ["in", users]}) | |||||
charts = frappe.get_all("Dashboard Chart", filters={"owner": ["in", users]}) | |||||
for chart in charts: | for chart in charts: | ||||
frappe.db.set_value("Dashboard Chart", chart.name, "is_public", 1) | frappe.db.set_value("Dashboard Chart", chart.name, "is_public", 1) |
@@ -5,7 +5,7 @@ import frappe | |||||
def execute(): | def execute(): | ||||
doctypes = frappe.db.get_all("DocType", {"module": "Data Migration", "custom": 0}, pluck="name") | |||||
doctypes = frappe.get_all("DocType", {"module": "Data Migration", "custom": 0}, pluck="name") | |||||
for doctype in doctypes: | for doctype in doctypes: | ||||
frappe.delete_doc("DocType", doctype, ignore_missing=True) | frappe.delete_doc("DocType", doctype, ignore_missing=True) | ||||
@@ -17,6 +17,6 @@ def execute(): | |||||
"yellow": "Yellow", | "yellow": "Yellow", | ||||
"lightblue": "Light Blue", | "lightblue": "Light Blue", | ||||
} | } | ||||
for d in frappe.db.get_all("Kanban Board Column", fields=["name", "indicator"]): | |||||
for d in frappe.get_all("Kanban Board Column", fields=["name", "indicator"]): | |||||
color_name = indicator_map.get(d.indicator, "Gray") | color_name = indicator_map.get(d.indicator, "Gray") | ||||
frappe.db.set_value("Kanban Board Column", d.name, "indicator", color_name) | frappe.db.set_value("Kanban Board Column", d.name, "indicator", color_name) |
@@ -4,7 +4,7 @@ import frappe | |||||
def execute(): | def execute(): | ||||
frappe.reload_doctype("Web Form") | frappe.reload_doctype("Web Form") | ||||
for web_form in frappe.db.get_all("Web Form", filters={"is_multi_step_form": 1}): | |||||
for web_form in frappe.get_all("Web Form", filters={"is_multi_step_form": 1}): | |||||
web_form_fields = frappe.get_doc("Web Form", web_form.name).web_form_fields | web_form_fields = frappe.get_doc("Web Form", web_form.name).web_form_fields | ||||
for web_form_field in web_form_fields: | for web_form_field in web_form_fields: | ||||
if web_form_field.fieldtype == "Section Break" and web_form_field.idx != 1: | if web_form_field.fieldtype == "Section Break" and web_form_field.idx != 1: | ||||
@@ -9,6 +9,6 @@ def execute(): | |||||
frappe.reload_doc("website", "doctype", "web_form_list_column") | frappe.reload_doc("website", "doctype", "web_form_list_column") | ||||
frappe.reload_doctype("Web Form") | frappe.reload_doctype("Web Form") | ||||
for web_form in frappe.db.get_all("Web Form", fields=["*"]): | |||||
for web_form in frappe.get_all("Web Form", fields=["*"]): | |||||
if web_form.allow_multiple and not web_form.show_list: | if web_form.allow_multiple and not web_form.show_list: | ||||
frappe.db.set_value("Web Form", web_form.name, "show_list", True) | frappe.db.set_value("Web Form", web_form.name, "show_list", True) |
@@ -451,13 +451,13 @@ def get_doctype_roles(doctype, access_type="read"): | |||||
def get_perms_for(roles, perm_doctype="DocPerm"): | def get_perms_for(roles, perm_doctype="DocPerm"): | ||||
"""Get perms for given roles""" | """Get perms for given roles""" | ||||
filters = {"permlevel": 0, "docstatus": 0, "role": ["in", roles]} | filters = {"permlevel": 0, "docstatus": 0, "role": ["in", roles]} | ||||
return frappe.db.get_all(perm_doctype, fields=["*"], filters=filters) | |||||
return frappe.get_all(perm_doctype, fields=["*"], filters=filters) | |||||
def get_doctypes_with_custom_docperms(): | def get_doctypes_with_custom_docperms(): | ||||
"""Returns all the doctypes with Custom Docperms""" | """Returns all the doctypes with Custom Docperms""" | ||||
doctypes = frappe.db.get_all("Custom DocPerm", fields=["parent"], distinct=1) | |||||
doctypes = frappe.get_all("Custom DocPerm", fields=["parent"], distinct=1) | |||||
return [d.parent for d in doctypes] | return [d.parent for d in doctypes] | ||||
@@ -525,7 +525,7 @@ def clear_user_permissions_for_doctype(doctype, user=None): | |||||
filters = {"allow": doctype} | filters = {"allow": doctype} | ||||
if user: | if user: | ||||
filters["user"] = user | filters["user"] = user | ||||
user_permissions_for_doctype = frappe.db.get_all("User Permission", filters=filters) | |||||
user_permissions_for_doctype = frappe.get_all("User Permission", filters=filters) | |||||
for d in user_permissions_for_doctype: | for d in user_permissions_for_doctype: | ||||
frappe.delete_doc("User Permission", d.name) | frappe.delete_doc("User Permission", d.name) | ||||
@@ -13,7 +13,7 @@ from frappe.utils.weasyprint import download_pdf, get_html | |||||
class PrintFormat(Document): | class PrintFormat(Document): | ||||
def onload(self): | def onload(self): | ||||
templates = frappe.db.get_all( | |||||
templates = frappe.get_all( | |||||
"Print Format Field Template", | "Print Format Field Template", | ||||
fields=["template", "field", "name"], | fields=["template", "field", "name"], | ||||
filters={"document_type": self.doc_type}, | filters={"document_type": self.doc_type}, | ||||
@@ -27,7 +27,7 @@ class PrintFormatFieldTemplate(Document): | |||||
filters = {"document_type": self.document_type, "field": self.field} | filters = {"document_type": self.document_type, "field": self.field} | ||||
if not self.is_new(): | if not self.is_new(): | ||||
filters.update({"name": ("!=", self.name)}) | filters.update({"name": ("!=", self.name)}) | ||||
result = frappe.db.get_all("Print Format Field Template", filters=filters, limit=1) | |||||
result = frappe.get_all("Print Format Field Template", filters=filters, limit=1) | |||||
if result: | if result: | ||||
frappe.throw( | frappe.throw( | ||||
_("A template already exists for field {0} of {1}").format( | _("A template already exists for field {0} of {1}").format( | ||||
@@ -101,7 +101,7 @@ def set_permission(doctype, name, user, permission_to, value=1, everyone=0): | |||||
@frappe.whitelist() | @frappe.whitelist() | ||||
def get_users(doctype, name): | def get_users(doctype, name): | ||||
"""Get list of users with which this document is shared""" | """Get list of users with which this document is shared""" | ||||
return frappe.db.get_all( | |||||
return frappe.get_all( | |||||
"DocShare", | "DocShare", | ||||
fields=[ | fields=[ | ||||
"`name`", | "`name`", | ||||
@@ -137,7 +137,7 @@ def get_shared(doctype, user=None, rights=None): | |||||
if user != "Guest": | if user != "Guest": | ||||
or_filters += [["everyone", "=", 1]] | or_filters += [["everyone", "=", 1]] | ||||
shared_docs = frappe.db.get_all( | |||||
shared_docs = frappe.get_all( | |||||
"DocShare", fields=["share_name"], filters=filters, or_filters=or_filters | "DocShare", fields=["share_name"], filters=filters, or_filters=or_filters | ||||
) | ) | ||||
@@ -42,7 +42,7 @@ class TestReportview(unittest.TestCase): | |||||
content="test", | content="test", | ||||
seen_by=[{"user": "Administrator"}], | seen_by=[{"user": "Administrator"}], | ||||
).insert() | ).insert() | ||||
result = frappe.db.get_all( | |||||
result = frappe.get_all( | |||||
"Note", | "Note", | ||||
filters={"name": note.name}, | filters={"name": note.name}, | ||||
fields=["name", "seen_by.user as seen_by"], | fields=["name", "seen_by.user as seen_by"], | ||||
@@ -55,7 +55,7 @@ class TestReportview(unittest.TestCase): | |||||
todo = frappe.get_doc( | todo = frappe.get_doc( | ||||
doctype="ToDo", description="Test ToDo", allocated_to="Administrator" | doctype="ToDo", description="Test ToDo", allocated_to="Administrator" | ||||
).insert() | ).insert() | ||||
result = frappe.db.get_all( | |||||
result = frappe.get_all( | |||||
"ToDo", | "ToDo", | ||||
filters={"name": todo.name}, | filters={"name": todo.name}, | ||||
fields=["name", "allocated_to.email as allocated_user_email"], | fields=["name", "allocated_to.email as allocated_user_email"], | ||||
@@ -9,7 +9,7 @@ class TestSitemap(unittest.TestCase): | |||||
from frappe.test_runner import make_test_records | from frappe.test_runner import make_test_records | ||||
make_test_records("Blog Post") | make_test_records("Blog Post") | ||||
blogs = frappe.db.get_all("Blog Post", {"published": 1}, ["route"], limit=1) | |||||
blogs = frappe.get_all("Blog Post", {"published": 1}, ["route"], limit=1) | |||||
xml = get_html_for_route("sitemap.xml") | xml = get_html_for_route("sitemap.xml") | ||||
self.assertTrue("/about</loc>" in xml) | self.assertTrue("/about</loc>" in xml) | ||||
self.assertTrue("/contact</loc>" in xml) | self.assertTrue("/contact</loc>" in xml) | ||||
@@ -38,7 +38,7 @@ def create_if_not_exists(doc): | |||||
@frappe.whitelist() | @frappe.whitelist() | ||||
def create_todo_records(): | def create_todo_records(): | ||||
if frappe.db.get_all("ToDo", {"description": "this is first todo"}): | |||||
if frappe.get_all("ToDo", {"description": "this is first todo"}): | |||||
return | return | ||||
frappe.get_doc( | frappe.get_doc( | ||||
@@ -80,7 +80,7 @@ def setup_workflow(): | |||||
@frappe.whitelist() | @frappe.whitelist() | ||||
def create_contact_phone_nos_records(): | def create_contact_phone_nos_records(): | ||||
if frappe.db.get_all("Contact", {"first_name": "Test Contact"}): | |||||
if frappe.get_all("Contact", {"first_name": "Test Contact"}): | |||||
return | return | ||||
doc = frappe.new_doc("Contact") | doc = frappe.new_doc("Contact") | ||||
@@ -127,7 +127,7 @@ def create_child_doctype(name, fields): | |||||
@frappe.whitelist() | @frappe.whitelist() | ||||
def create_contact_records(): | def create_contact_records(): | ||||
if frappe.db.get_all("Contact", {"first_name": "Test Form Contact 1"}): | |||||
if frappe.get_all("Contact", {"first_name": "Test Form Contact 1"}): | |||||
return | return | ||||
insert_contact("Test Form Contact 1", "12345") | insert_contact("Test Form Contact 1", "12345") | ||||
@@ -137,7 +137,7 @@ def create_contact_records(): | |||||
@frappe.whitelist() | @frappe.whitelist() | ||||
def create_multiple_todo_records(): | def create_multiple_todo_records(): | ||||
if frappe.db.get_all("ToDo", {"description": "Multiple ToDo 1"}): | |||||
if frappe.get_all("ToDo", {"description": "Multiple ToDo 1"}): | |||||
return | return | ||||
values = [(f"100{i}", f"Multiple ToDo {i}") for i in range(1, 1002)] | values = [(f"100{i}", f"Multiple ToDo {i}") for i in range(1, 1002)] | ||||
@@ -291,7 +291,7 @@ def update_child_table(name): | |||||
@frappe.whitelist() | @frappe.whitelist() | ||||
def insert_doctype_with_child_table_record(name): | def insert_doctype_with_child_table_record(name): | ||||
if frappe.db.get_all(name, {"title": "Test Grid Search"}): | |||||
if frappe.get_all(name, {"title": "Test Grid Search"}): | |||||
return | return | ||||
def insert_child(doc, data, barcode, check, rating, duration, date): | def insert_child(doc, data, barcode, check, rating, duration, date): | ||||
@@ -33,7 +33,7 @@ class WebTemplate(Document): | |||||
def on_update(self): | def on_update(self): | ||||
"""Clear cache for all Web Pages in which this template is used""" | """Clear cache for all Web Pages in which this template is used""" | ||||
routes = frappe.db.get_all( | |||||
routes = frappe.get_all( | |||||
"Web Page", | "Web Page", | ||||
filters=[ | filters=[ | ||||
["Web Page Block", "web_template", "=", self.name], | ["Web Page Block", "web_template", "=", self.name], | ||||
@@ -33,7 +33,7 @@ def get_page_info_from_web_page_with_dynamic_routes(path): | |||||
def get_page_info_from_web_form(path): | def get_page_info_from_web_form(path): | ||||
"""Query published web forms and evaluate if the route matches""" | """Query published web forms and evaluate if the route matches""" | ||||
rules, page_info = [], {} | rules, page_info = [], {} | ||||
web_forms = frappe.db.get_all("Web Form", ["name", "route", "modified"], {"published": 1}) | |||||
web_forms = frappe.get_all("Web Form", ["name", "route", "modified"], {"published": 1}) | |||||
for d in web_forms: | for d in web_forms: | ||||
rules.append(Rule(f"/{d.route}", endpoint=d.name)) | rules.append(Rule(f"/{d.route}", endpoint=d.name)) | ||||
rules.append(Rule(f"/{d.route}/list", endpoint=d.name)) | rules.append(Rule(f"/{d.route}/list", endpoint=d.name)) | ||||
@@ -40,7 +40,7 @@ def get_email_groups(user_email): | |||||
def get_current_groups(name): | def get_current_groups(name): | ||||
# Return current group by which the mail has been sent. | # Return current group by which the mail has been sent. | ||||
return frappe.db.get_all( | |||||
return frappe.get_all( | |||||
"Newsletter Email Group", | "Newsletter Email Group", | ||||
fields=["email_group"], | fields=["email_group"], | ||||
filters={"parent": name, "parenttype": "Newsletter"}, | filters={"parent": name, "parenttype": "Newsletter"}, | ||||