Now pushed :) I opted for the get_class($e) option rather than catching the exception to reduce code duplication.
** Changed in: mahara Assignee: (unassigned) => Andrew Nicols (dobedobedoh) ** Changed in: mahara Status: New => 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/690713 Title: notify_user() gets stuck in a loop if an admin user has disabled their email Status in Mahara ePortfolio: Fix Committed Bug description: A change introduced by 4ce12125c10984b946f4061e517521cbb6322389 meant that if email_user threw an exception, the admin was sent an e-mail. If an admin user has email disabled, then mahara gets stuck in an email loop trying to send them e-mail informing them of the failure. Have fix... will push. _______________________________________________ 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