Hello,

Almost every day, I have many repeated errors with drupal 7.x-1.0-beta1

First, I have a notice message saying : Session closed for .

Next, I have this PHP error :

Warning: session_destroy() [function.session-destroy]: Trying to destroy uninitialized session in drupad_user_logout() (line 429 in /mysite.com/sites/all/modules/drupad/drupad.module).

Next again, I have this PHP error :

Notice: Undefined property: stdClass::$name in drupad_user_logout() (line 426 in /mysite.com/sites/all/modules/drupad/drupad.module).

These 3 messages fill about 20 pages at once in the message logs.

CommentFileSizeAuthor
#1 d7_warning_upon_session_destroy.patch710 bytesjchatard

Comments

jchatard’s picture

jchatard commented
Assigned: Unassigned » jchatard
Status: Active » Needs review
StatusFileSize
new d7_warning_upon_session_destroy.patch710 bytes

Hi anavarre,

This must be a dumb error from me.

Can you try the patch attached? Or simply delete the function drupad_user_logout() in drupad.module.

This function has no need to exist and overloads what Drupal core already does.

Thanks,
Jérémy

anavarre’s picture

anavarre
Primary language French
Location 🇪🇺
commented

Thank you for the quick fix. I've just deleted the drupad_user_logout() function in the drupal.module file

I'll let you know if everything goes right from now on in a few days.

anavarre’s picture

anavarre
Primary language French
Location 🇪🇺
commented
Status: Needs review » Reviewed & tested by the community

Hello again,

It's been 4 days and everything works as expected. I guess the patch worked then.

Thank you

jchatard’s picture

jchatard commented
Status: Reviewed & tested by the community » Fixed

This is committed and will be available in the next release of the module.

BenK’s picture

BenK commented

Keeping track of this thread

Status: Fixed » Closed (fixed)

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

roam2345’s picture

roam2345 commented

getting a bucket load of these logs in my sites..... hmmm, disabling app till this is fixed.

roam2345’s picture

roam2345 commented

please roll a release soon, as this is fixed but yet unavailable..

Thanks.

jchatard’s picture

jchatard commented

Yep sorry.

It's coming now as 7.x-1.0

Jérémy

roam2345’s picture

roam2345 commented

Thanks Jérémy

The app rocks when it works, thanks for the info.