Comment 3 for bug 824445

Revision history for this message
Mahara Bot (dev-mahara) wrote : A change has been merged

Reviewed: https://reviews.mahara.org/578
Committed: http://gitorious.org/mahara/mahara/commit/0a694de54c37f00e8c2c826f46f319a59e9bb0a8
Submitter: Francois Marier (<email address hidden>)
Branch: master

commit 0a694de54c37f00e8c2c826f46f319a59e9bb0a8
Author: Andrew Robert Nicols <email address hidden>
Date: Thu Aug 11 11:54:48 2011 +0100

    Bug #824445: Improve error reporting for missing plugins

    This set of changes attempts to handle missing plugins in a better fashion.
    Rather than throwing an uncaught error which causes a unrecoverable site
    error, which in turn means that no user can use any other plugin; this
    patch disables the problem plugin and informs administrators of the issue.

    To handle the case where core plugins, which under normal circumstances
    cannot be disabled, are missing and disabled; the plugins.tpl template is
    also changed to test for the contents of the activateform, instead of
    whether the plugin can ordinarily be disabled. This means that if the issue
    is then fixed, the plugin can once again be re-enabled, at which point the
    [ hide ] link will no longer appear.

    Change-Id: I6f9fa73ae0ac307b5f5f8f30b8205690926f54fc
    Signed-off-by: Andrew Robert Nicols <email address hidden>