Getting this error when logged in as admin. Seems to come up on every page refresh in the logs (admin/reports/dblog). This patch fixes it by checking if the force_password_change property isset first.

CommentFileSizeAuthor
#3 policynoticefix2.patch730 bytesjrowny
password-policy-notice-fix.patch551 bytesacbramley

Comments

erikwebb’s picture

erikwebb commented
Version: 7.x-1.0-beta2 » 7.x-1.x-dev
Assigned: Unassigned » erikwebb
Status: Active » Fixed

Committed.

http://drupalcode.org/project/password_policy.git/commit/83c7539

Status: Fixed » Closed (fixed)

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

jrowny’s picture

jrowny commented
Status: Closed (fixed) » Needs work
StatusFileSize
new policynoticefix2.patch730 bytes

Looks like the latest release re-opened this problem on line 35. Same fix resolves the issue. I think this only happens when installing from a distribution (in my case, OpenPublic) and then updating before ever saving any configurations.

erikwebb’s picture

erikwebb commented
Status: Needs work » Closed (fixed)

The latest release does not include all recent patches. It was a security release and limited in scope. I will try and release a new version with recent fixes this week.

jrowny’s picture

jrowny commented

Thanks, just wanted to let you know that this one happens on a different line, although it's the same error. I haven't looked too deeply at this module, but maybe you could use hook_user_load to ensure that this property exists to avoid the problem altogether. I feel like the patch I submitted as well as the earlier ones are kind of "band aids."

jrowny’s picture

jrowny commented

Opps, just realized I was using the 6.x code and this is 7. x you can just disregard all my posts in this issue!

erikwebb’s picture

erikwebb commented

Is this an issue on 6.x as well? If so, we need to fix it there as well.