fix(api/v1): don't use as_dict() unless expand_links is passed (#34672)

That sets all keys to null if they don't exist, a breaking change.

Signed-off-by: Akhil Narang <me@akhilnarang.dev>
This commit is contained in:
Akhil Narang 2025-11-11 23:14:39 +05:30 committed by GitHub
parent 16f2ad6e8e
commit 0fd745ae57
No known key found for this signature in database
GPG key ID: B5690EEEBB952194

View file

@ -78,12 +78,13 @@ def read_doc(doctype: str, name: str):
doc = frappe.get_doc(doctype, name)
doc.check_permission("read")
doc.apply_fieldlevel_read_permissions()
doc_dict = doc.as_dict()
if sbool(frappe.form_dict.get("expand_links")):
doc_dict = doc.as_dict()
get_values_for_link_and_dynamic_link_fields(doc_dict)
get_values_for_table_and_multiselect_fields(doc_dict)
return doc_dict
return doc_dict
return doc
def get_values_for_link_and_dynamic_link_fields(doc_dict):