Was examining queries executed during creating a node and saw workflow_node_current_state query get called three times during inserting a node. As we know the sid the entire time, imo that should be 0 times.

Patch for d6 as that's what I am on, no idea the state of d7

CommentFileSizeAuthor
#1 workflow-node-current-state-1893810-1.patch3.33 KBhefox
workflow_get_current_sid.patch2.94 KBhefox

Comments

hefox’s picture

hefox commented
StatusFileSize
new workflow-node-current-state-1893810-1.patch3.33 KB

Missed a spot

johnv’s picture

johnv
Primary language Dutch
commented
Status: Needs review » Closed (won't fix)

Closing this issue, since D6 is not maintained anymore. In D7, the code has been changed drastically, and a cache is added to the function.
BTW, it is not uncommon to have 3 DB-accesss, since 1 insert consists of 3 pages: validate new node, submit new node, show created node.
Please reopen if you think this can be optimized in D7.
Thanks anyway.