Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

LoginLogoutRedir fills php_error.log with "Strict" notices #3

Open
joaocc opened this issue Mar 9, 2016 · 2 comments
Open

LoginLogoutRedir fills php_error.log with "Strict" notices #3

joaocc opened this issue Mar 9, 2016 · 2 comments

Comments

@joaocc
Copy link

joaocc commented Mar 9, 2016

PHP Strict Standards:  Declaration of auth_plugin_loginlogoutredir::user_authenticated_hook() should be compatible with auth_plugin_base::user_authenticated_hook(&$user, $username, $password) in /home/mm/www/auth/loginlogoutredir/auth.php on line 0

This is happening with Moodle 3.0 and PHP 5.5.
Apart from this, everything seems to be working fine.
Would it be possible to fix the signature to prevent the warning?
Thx

@hrimhari
Copy link
Owner

hrimhari commented Jul 4, 2016

Not too high on my priority list, but will try to keep in mind. Pull requests are welcome too.

@luizolivetti
Copy link

use &$user in first parameter

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants