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

Comments

agentrickard’s picture

agentrickard
he/him
Primary language English
Location Georgia (US)
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)
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 commented

Will provide the patch shortly.

bluegeek9’s picture

bluegeek9 commented
Issue summary: View changes
Status: Needs work » Closed (outdated)

Drupal 7 in End of Life and no longer supported. We encourage you to upgrade to a supported version of Drupal. For more information, see https://www.drupal.org/upgrade.

//www.flaticon.com/free-icons/thank-you Thank you for your contribution! Your continued support of this project makes other volunteer contributions more sustainable.
There are multiple ways to show appreciation for the work contributed to this project, including:
  • Triaging issues and adding more context to existing issues.
  • Writing documentation or patches for this project.