When the module is enabled but inactive hook_user_insert still fires (and WOD errors out) on user registration.

This is an annoyance when moving to a dev environment and needing to leave the module on (due to feature dependencies) and could actually be a problem during deployment when you need to enable and configure without yet enabling.

Just a quick if wrapper.

CommentFileSizeAuthor
#1 simplesamlphp_auth-inactive-insert-1586262-1.patch3.53 KBdoublejosh

Comments

doublejosh’s picture

doublejosh commented
StatusFileSize
new simplesamlphp_auth-inactive-insert-1586262-1.patch3.53 KB

Cloned your 2.0 repo and made the small change, but the diff patch it gave me seems wack-a-doodle.

Just added this...

function simplesamlphp_auth_user_insert(&$edit, $account, $category = NULL) {
  if (!_simplesamlphp_auth_isEnabled()) {
    // Exit without initializing.
    return;
  }
  else {
  // ALL THAT JAZZ
  }
}
geekwisdom’s picture

geekwisdom commented
Status: Active » Fixed

Thanks again Josh. Corrected in 7.x-2.x branch. See: http://drupalcode.org/project/simplesamlphp_auth.git/commit/e88a135acc6d...

Status: Fixed » Closed (fixed)

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