-
Bug
-
Resolution: Fixed
-
Major
-
3.9.19, 3.11.12, 4.0.6, 4.1.1, 4.2
-
MOODLE_311_STABLE, MOODLE_39_STABLE, MOODLE_400_STABLE, MOODLE_401_STABLE, MOODLE_402_STABLE
-
MOODLE_311_STABLE, MOODLE_39_STABLE, MOODLE_400_STABLE, MOODLE_401_STABLE
-
MDL-77417_401 -
Notice
- This needs to be tested with PHP 8.1 (because the code-changes suggested to do the testing use something that has been deprecated in that PHP version).
- The patch is exactly the same for all branches, so it's enough to perform the testing instructions below only with master.
Before the patch
- Ensure that you are using moodle.git (without the patch).
- Apply this change to login/index.php:
diff --git a/login/index.php b/login/index.phpindex 25312086d8a..b6f915e232e 100644--- a/login/index.php+++ b/login/index.php@@ -27,6 +27,8 @@require('../config.php');require_once('lib.php');+ini_set('auto_detect_line_endings', true);+redirect_if_major_upgrade_required();- Or download 77417.diff
to your moodle root folder and apply via git:
git apply 77417.diff
- Or download 77417.diff
- Init behat. e.g. if using MDK:
mdk behat - Run the --name 'Log in as an existing admin user filling the form' scenario.
php admin/tool/behat/cli/run.php --name="Log in as an existing admin user filling the form" - Verify it fails with "Exception - Unknown error type" (and other details).
- Edit login/index.php and add the silence operator:
-ini_set('auto_detect_line_endings', true);+@ini_set('auto_detect_line_endings', true);- Or download and apply 77417_silence.diff. (You might need to run "git reset --hard" first to undo the changes applied earlier.
- Or download and apply 77417_silence.diff
- Run the --name 'Log in as an existing admin user filling the form' scenario.
- Verify it fails with "Exception - Unknown error type" (and other details).
- Reset the changes by running "git reset --hard"
After the patch
- Ensure that you are using integration.git (with the patch).
- Apply this change to login/index.php:
diff --git a/login/index.php b/login/index.phpindex 25312086d8a..b6f915e232e 100644--- a/login/index.php+++ b/login/index.php@@ -27,6 +27,8 @@require('../config.php');require_once('lib.php');+ini_set('auto_detect_line_endings', true);+redirect_if_major_upgrade_required();- Or download and apply 77417.diff.
- Or download and apply 77417.diff
- Init behat
- Run the --name 'Log in as an existing admin user filling the form' scenario.
- Verify it fails with "Exception - Unknown error type" (and other details).
- Edit login/index.php and add the silence operator:
-ini_set('auto_detect_line_endings', true);+@ini_set('auto_detect_line_endings', true);- Or download and apply 77417_silence.diff.
- Or download and apply 77417_silence.diff
- Run the --name 'Log in as an existing admin user filling the form' scenario.
- Verify it passes ok.
Notice This needs to be tested with PHP 8.1 (because the code-changes suggested to do the testing use something that has been deprecated in that PHP version). The patch is exactly the same for all branches, so it's enough to perform the testing instructions below only with master . Before the patch Ensure that you are using moodle.git (without the patch). Apply this change to login/index.php : diff --git a/login/index.php b/login/index.php index 25312086d8a..b6f915e232e 100644 --- a/login/index.php +++ b/login/index.php @@ -27,6 +27,8 @@ require('../config.php'); require_once('lib.php'); +ini_set('auto_detect_line_endings', true); + redirect_if_major_upgrade_required(); Or download 77417.diff to your moodle root folder and apply via git: git apply 77417.diff Init behat. e.g. if using MDK: mdk behat Run the --name 'Log in as an existing admin user filling the form' scenario. php admin/tool/behat/cli/run.php --name="Log in as an existing admin user filling the form" Verify it fails with "Exception - Unknown error type" (and other details). Edit login/index.php and add the silence operator: -ini_set('auto_detect_line_endings', true); +@ini_set('auto_detect_line_endings', true); Or download and apply 77417_silence.diff . (You might need to run " git reset --hard " first to undo the changes applied earlier. Run the --name 'Log in as an existing admin user filling the form' scenario. Verify it fails with "Exception - Unknown error type" (and other details). Reset the changes by running " git reset --hard " After the patch Ensure that you are using integration.git (with the patch). Apply this change to login/index.php : diff --git a/login/index.php b/login/index.php index 25312086d8a..b6f915e232e 100644 --- a/login/index.php +++ b/login/index.php @@ -27,6 +27,8 @@ require('../config.php'); require_once('lib.php'); +ini_set('auto_detect_line_endings', true); + redirect_if_major_upgrade_required(); Or download and apply 77417.diff . Init behat Run the --name 'Log in as an existing admin user filling the form' scenario. Verify it fails with "Exception - Unknown error type" (and other details). Edit login/index.php and add the silence operator: -ini_set('auto_detect_line_endings', true); +@ini_set('auto_detect_line_endings', true); Or download and apply 77417_silence.diff . Run the --name 'Log in as an existing admin user filling the form' scenario. Verify it passes ok.
Right now the behat error handler (behat_error_handler) does not know about deprecated error types (E_DEPRECATED, E_USER_DEPRECATED).
That implies that:
- Any of them lead to a failed test.
- The silence operator (@) that may be used (by libraries mainly, to hide some deprecation messages on purpose) is not observed.
So this issue is about to ensure that Behat behaves exactly the same with those two "new" error types than with the rest. Because when that handling was created (seee MDL-38041), those error types did not exist. Now they do.
Ciao 
