Skip to content
This repository has been archived by the owner on Jul 6, 2020. It is now read-only.

#814 Added hook_update_N() to remove unused OSR staff and testing accounts. #975

Closed
wants to merge 9 commits into from

Conversation

proofoftom
Copy link
Contributor

@proofoftom proofoftom commented Mar 22, 2017

Also, changes the behavior of how users are added. Instead of adding CU users automatically, we now assign them the appropriate role if added later.

@kreynen kreynen modified the milestone: 2.5.0 May 23, 2017
@alexfinnarn
Copy link
Contributor

Not sure on the change Tommy added to cu_users_get_users(). Maybe should go in another function since it's not returning a user.

  // If a account name was provided then return the user's role name to
  // cu_users_get_user_role().
  if ($type === NULL && $account_name) {
    foreach ($user_array as $role_name => $users) {
      if (in_array($account_name, array_keys($users))) {
        return $role_name;
      }
    }
  }

@alexfinnarn alexfinnarn added this to the 2.6.x milestone Sep 25, 2017
@alexfinnarn alexfinnarn self-assigned this Sep 25, 2017
@kreynen kreynen modified the milestones: 2.6.x, 2.8.x Nov 9, 2017
@kreynen kreynen modified the milestones: 2.8.x, 2.9.x May 3, 2018
@alexfinnarn alexfinnarn modified the milestones: 2.9.x, 2.9.4 Jul 5, 2018
@kreynen
Copy link
Contributor

kreynen commented Jul 5, 2018

Rather than remove in a single update hook, the removal needs to be ongoing. This is now being handled in #814

@kreynen kreynen closed this Jul 5, 2018
@alexfinnarn alexfinnarn deleted the feature/814 branch August 23, 2018 17:31
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants