From b3bbeffdc0c65fe9d12dd51d92f8991772d00f56 Mon Sep 17 00:00:00 2001 From: Raj Barshikar <145206238+RajBarshikar@users.noreply.github.com> Date: Mon, 2 Feb 2026 14:37:59 +0530 Subject: [PATCH] fix: pass name string to get_doc in desktop_icon (#36292) Previously, `get_doc` was receiving the entire `s` dictionary object instead of the document name string `s.name`. This caused a `DoesNotExistError` and `SessionBootFailed` when loading the Desk. This change explicitly passes `s.name` to `get_doc` to ensure the Desktop Icon is retrieved correctly. --- frappe/desk/doctype/desktop_icon/desktop_icon.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/frappe/desk/doctype/desktop_icon/desktop_icon.py b/frappe/desk/doctype/desktop_icon/desktop_icon.py index 47f7f290a0..a825b391c0 100644 --- a/frappe/desk/doctype/desktop_icon/desktop_icon.py +++ b/frappe/desk/doctype/desktop_icon/desktop_icon.py @@ -201,7 +201,7 @@ def get_desktop_icons(user=None, bootinfo=None): permitted_parent_labels = set() if bootinfo: for s in user_icons: - icon = frappe.get_doc("Desktop Icon", s) + icon = frappe.get_doc("Desktop Icon", s.name) if icon.is_permitted(bootinfo): permitted_icons.append(s)