HEAD is atm missing hook_domainupdate() in domain_conf to to enable deleting settings for domains. Once a domain is deleted its settings are still in domain conf table.

CommentFileSizeAuthor
#4 domain_conf.module_domainupdate.patch1.23 KBnonsie

Comments

agentrickard’s picture

agentrickard
he/him
Primary language English
Location Georgia (US)
commented
Category: feature » task

Did I remove it accidentally?

nonsie’s picture

nonsie
she/her
Primary language Estonian
commented

I think so - I can no longer find it.

nonsie’s picture

nonsie
she/her
Primary language Estonian
commented

I don't have the tools to roll a patch atm but adding the following should do it:

/**
 * Implements hook_domainupdate()
 */
function domain_conf_domainupdate($op, $domain, $form_state = array()) {
  if ($op == 'delete' && $domain['domain_id'] > 0) {
    db_query("DELETE FROM {domain_conf} WHERE domain_id = %d", $domain['domain_id']);
    cache_clear_all('variables', 'cache');
  }
}
nonsie’s picture

nonsie
she/her
Primary language Estonian
commented
Status: Active » Needs review
StatusFileSize
new domain_conf.module_domainupdate.patch1.23 KB

Patch attached

agentrickard’s picture

agentrickard
he/him
Primary language English
Location Georgia (US)
commented
Status: Needs review » Fixed

Committed to HEAD and to 5.x.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.