I get a notice in the _password_policy_constraint_validate() function when $account->roles is not defined. Patch to follow.

CommentFileSizeAuthor
#1 password_policy-roles_notice-1800280-1.patch623 bytesbdlangton
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

bdlangton’s picture

bdlangton CreditAttribution: bdlangton commented
Status: Active » Needs review
FileSize
password_policy-roles_notice-1800280-1.patch623 bytes

Here is the patch.

erikwebb’s picture

erikwebb CreditAttribution: erikwebb commented

Simple enough patch, but when would $account->roles not be defined?

bdlangton’s picture

bdlangton CreditAttribution: bdlangton commented

Good question. It actually may never happen in a standard Drupal install, it might just be something with my site that was causing the problem.

erikwebb’s picture

erikwebb CreditAttribution: erikwebb commented
Title: Notice: Undefined property: stdClass::$roles in _password_policy_constraint_validate() » Notice when $account->roles is not set
Status: Needs review » Fixed

http://drupalcode.org/project/password_policy.git/commit/1274c95

Status: Fixed » Closed (fixed)

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