Just a heads up... This module won't work when the Protect Critical Users module is installed. The specific conflict is hook_form_alter. 

/**
 * Implementation of hook_form_alter().
 */
function protect_critical_users_form_alter(&$form, $form_state, $form_id) {
  global $user;

  // Remove delete button from user 1 edit form.
  if ($form_id == 'user_profile_form' && isset($form['delete']) && isset($form['_account']['#value']->uid)) {
    if ($form['_account']['#value']->uid == 1 || $form['_account']['#value']->uid == $user->uid) {
      unset($form['delete']);
    }
  }
}

As you can see... Protect Critical Users unsets the delete button. Also might I suggest the thought of merging these modules? I'm not saying they do the same task. but the idea of users who can delete their own account while still protecting certain roles of super admin from doing so to their account.

CommentFileSizeAuthor
#6 protect_critical_users-user_delete-integration.patch1.32 KBmanuel.adan
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

sanduhrs’s picture

sanduhrs
he/him
Primary language German
Location 🇪🇺 Heidelberg, Germany, Europe
CreditAttribution: sanduhrs commented
Version: 6.x-1.2 » 6.x-1.x-dev
Category: bug » feature
Priority: Critical » Normal

I will have a look at that.
Thanks.

mrfelton’s picture

mrfelton CreditAttribution: mrfelton commented

perhaps the critical_users module should not be unsetting the delete button, but instead setting $form['delete']['#access'] = FALSE;

sanduhrs’s picture

sanduhrs
he/him
Primary language German
Location 🇪🇺 Heidelberg, Germany, Europe
CreditAttribution: sanduhrs commented
Project: User Delete » Protect Critical Users
Version: 6.x-1.x-dev » 6.x-1.1

That actually feels more like a issue for protect_critical_users module.

markus_petrux’s picture

markus_petrux CreditAttribution: markus_petrux commented
Project: Protect Critical Users » User Delete
Version: 6.x-1.1 » 6.x-1.x-dev

The life time of the Protect Critical Users (PCU) module is quite short, and it will end as soon as the core issue is fixed: #46149: Prevent account cancellation for uid 1. I do not plan to extend the module with anything else. It will simply die the day core is fixed.

So... I would suggest adding to User Delete whatever it may need to make PCU redundant.

As far as PCU is concerned, this is "won't fix" / "by design". Both modules are simply not compatible. If users need User Delete, then I would suggest asking UD for anything it may need to cover the whole picture. User Delete module will need it from D7 and on, because the core patch is about to get in pretty soon, it seems to me.

sanduhrs’s picture

sanduhrs
he/him
Primary language German
Location 🇪🇺 Heidelberg, Germany, Europe
CreditAttribution: sanduhrs commented
Status: Active » Closed (works as designed)

user_delete module will not have a D7 release either as the functinality it provides is included in D7.
Closing.

manuel.adan’s picture

manuel.adan
Location 🌌
CreditAttribution: manuel.adan commented
FileSize
protect_critical_users-user_delete-integration.patch1.32 KB

This may help both module to be used with no conflict. Note that patch is for the protect_critical_users module 6.x-1.1