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