Another PHP 5.3 E_NOTICE issue but it's quick to fix this one. The code shouldn't assume $account->force_password_change is set, and should explicitly check this. I've attached a really simple patch.

CommentFileSizeAuthor
#1 login_destination_1307448_notice.patch688 bytesBerdir
login_destination_force_password_change.patch792 byteschriscohen
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Berdir’s picture

Berdir
Primary language German
Location Switzerland
CreditAttribution: Berdir commented
Version: 6.x-2.12 » 6.x-2.x-dev
Status: Active » Reviewed & tested by the community
FileSize
login_destination_1307448_notice.patch688 bytes

Has nothing to do with PHP 5.3, this is just an E_NOTICE. Fix is correct though, re-uploading the same patch with the correct path.

rsvelko’s picture

rsvelko CreditAttribution: rsvelko as a volunteer commented
Issue summary: View changes

saw this simple one and although late, commiting now better than never.
PS. using isset() + check if true instead of !empty. That way is better/safer.

  • rsvelko committed 94479f6 on 6.x-2.x authored by chriscohen 
    Issue #1307478 by chriscohen, rsvelko: Notice: Undefined property:...
rsvelko’s picture

rsvelko CreditAttribution: rsvelko as a volunteer commented
Assigned: Unassigned » rsvelko
Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

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