Im getting this error after logging in with the simplenews module installed.

Warning: unserialize() expects parameter 1 to be string, array given in simplenews_user_login() (line 1019 of /var/www/vhosts/robisoncreative.com/subdomains/vitae/httpdocs/sites/all/modules/simplenews/simplenews.module).
CommentFileSizeAuthor
simplenews-hook_user_login-change.patch690 bytesJackinloadup

Comments

miro_dietiker’s picture

miro_dietiker
Location Switzerland
commented
Status: Needs review » Fixed

Note that in 6.x-2.x we added a conditional unserialize after check since some third party modules modify $user...
I hope after committing this, that core does unserialize reliably.

As i see with a quick scan this should be the case.

includes/session.inc:107:    // This is done to unserialize the data member of $user.
includes/session.inc:108:    $user->data = unserialize($user->data);

Committed to cvs.

Anyways: This (subscription) codepart needs a rewrite in D7 as we already started in 6.x-2.x.

Status: Fixed » Closed (fixed)

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