Richard, is that fix what you had in mind for master?
** Changed in: mahara
Status: Confirmed => In Progress
** Changed in: mahara
Status: In Progress => Fix Committed
--
You received this bug notification because you are a member of Mahara
Contributors, which is subscribed to Mahara.
https://bugs.launchpad.net/bugs/826453
Title:
Code to delete session id on logout is not triggered
Status in Mahara ePortfolio:
Fix Committed
Bug description:
See https://reviews.mahara.org/#change,573
As Brian King noted, there is a bug in the $USER->logout() function
that calls table_exists on a string rather than an XMLDBTable object.
What's worse is that this code in logout() doesn't really serve any
useful purpose. It is trying to delete a record of the user's session
id when the user logs out, but because most of the time ddl.php is not
included at this point, it won't be triggered. We should either
include ddl.php explicitly, so it succeeds, or else maybe remove that
code altogether.
To manage notifications about this bug go to:
https://bugs.launchpad.net/mahara/+bug/826453/+subscriptions
_______________________________________________
Mailing list: https://launchpad.net/~mahara-contributors
Post to : mahara-contributors@lists.launchpad.net
Unsubscribe : https://launchpad.net/~mahara-contributors
More help : https://help.launchpad.net/ListHelp
