Found that in two places empty delete queries are being executed while creating new content.

Attached the patch to prevent them.

CommentFileSizeAuthor
domain-empty_query-issue.patch1.66 KBmathankumarc
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

agentrickard’s picture

agentrickard
he/him
Primary language English
Location Georgia (US)
CreditAttribution: agentrickard commented
Priority: Normal » Minor
Status: Needs review » Needs work

Interesting. The $nid > 0 is supposed to catch that.

A better solution would be to pass the entire $node object instead of setting an additional parameter.

agentrickard’s picture

agentrickard
he/him
Primary language English
Location Georgia (US)
CreditAttribution: agentrickard commented

Even better, split domain_store_node_grants() into two functions, one for delete and leave the original as save only.

mathankumarc’s picture

mathankumarc CreditAttribution: mathankumarc commented

Will provide the patch shortly.