`frappe.db.get_list` does check for Document.`has_permission` about frappe HOT 4 CLOSED

@cgallay this is how it works. We can't invoke a document's permission hook which requires a single document for doing SQL queries. It will make things insanely slow.

You should rewrite same logic as perm query, that will handle get_list part.

Example:

from frappe.

has_permission is documented in hooks:

https://frappeframework.com/docs/user/en/python-api/hooks#document-permissions

perm query is documented on same page: https://frappeframework.com/docs/user/en/python-api/hooks#modify-list-query

from frappe.

Perfect, thanks! I was checking at the wrong location (https://docs.erpnext.com/docs/user/manual/en/user-permissions)

from frappe.

Thanks @ankush for the reply. I'll have to overwrite the get_permission_query_condition function then.

Would be nice if the documentation about permission gets an update. There is nothing mentioning the has_permission either.

from frappe.