danmarsden / moodle-auth_mcae

replaced by https://github.com/catalyst/moodle-local_cohortauto
https://github.com/catalyst/moodle-local_cohortauto
8 stars 12 forks source link

Does not work well with moddle 2.8.3 #14

Closed fvendrell closed 8 years ago

fvendrell commented 9 years ago

I got the following error message: get_context_instance() is deprecated, please use context_xxxx::instance() instead. line 3717 of /lib/deprecatedlib.php: call to debugging() line 19 of /auth/mcae/view.php: call to get_context_instance()

When trying to replace get_context_instance(CONTEXT_SYSTEM) by context_system::instance() as advice elsewhre, I got others error messages:

Debug info: Argument 2 passed to require_capability() must be an instance of context, null given, called in [dirroot]/auth/mcae/view.php on line 22 and defined Error code: codingerror Stack trace: line 393 of /lib/setuplib.php: coding_exception thrown line 783 of /lib/accesslib.php: call to default_error_handler() line 22 of /auth/mcae/view.php: call to require_capability()

I seem that this very very very useful plugin needs an update!

danmarsden commented 8 years ago

looks like this one can be closed - the code uses context_system::instance() now.

fvendrell commented 8 years ago

Thank you :-)