userprotect_check_bypass() checks the $bypass_defaults array by key. userprotect_administrator_bypass_defaults() creates a linear array (wrong), whereas a saved settings form creates an associative array (correct). Because of this bug, upon initial install, the default administrator bypass setting is effectively nothing, rather than everything as the setting form seems to indicate. Simple patch though.

CommentFileSizeAuthor
userprotect_bypass_defaults.patch333 bytescedarm
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

hunmonk’s picture

Title: userprotect_administrator_bypass_defaults() bug » userprotect_administrator_bypass_defaults() should return keyed array
Status: Needs review » Fixed

committed to 5.x-1.x-dev and 6.x-1.x-dev -- thanks!

Status: Fixed » Closed (fixed)

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