There is a function ideation_admin_paths_alter that does node_load just to find out type of a node. That is very wasteful (=slow) and also buggy - there are modules (such as SPS) that will implement node_load hook and call path_is_admin() which will result in calling ideation_admin_paths_alter, causing a nested loop.

CommentFileSizeAuthor
#1 2051007.patch1.11 KBmeba
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

meba’s picture

FileSize
1.11 KB

Patch attached

hefox’s picture

Issue summary: View changes
Status: Needs review » Needs work

node_load is cached and the node edit page will need to have loaded the node