cte-zl-ifrn / moodle__auth_suap

auth_suap
GNU General Public License v3.0
1 stars 0 forks source link

incorrect pluginname used in config defintion. #4

Open danmarsden opened 2 years ago

danmarsden commented 2 years ago

the name "auth/suap" is incorrect as a location for config settings. it should be "auth_suap" see examples in core like auth_ldap here: https://github.com/moodle/moodle/blob/master/auth/ldap/settings.php#L52

calls like get_config('auth/suap') should be more like get_config('auth_suap')

danmarsden commented 2 years ago

it looks like your code is a bit inconsistent - in some cases you use "auth_suap" correctly when getting config, in others like here: https://github.com/cte-zl-ifrn/moodle__auth_suap/blob/0559d75ca027d96d84b305a3a1494d5047dc823f/redirect.php#L6

you use "auth/suap" incorrectly.