[MDL-31707] Add environmental check to 2.3 about the hash extension installed. - Moodle Tracker

XML

Word

Printable

Affected Branches:
MOODLE_20_STABLE, MOODLE_21_STABLE, MOODLE_22_STABLE, MOODLE_23_STABLE
Fixed Branches:
MOODLE_21_STABLE, MOODLE_22_STABLE
Pull from Repository:
git://github.com/stronk7/moodle.git
Pull Master Branch:
~~MDL-31707~~
Pull Master Diff URL:
https://github.com/stronk7/moodle/compare/master...MDL-31707
Testing Instructions:
Hide

Go to admin -> server -> environement

In the "Moodle version" dropdown menu pick 2.3 (note it's the unique value in master right now, but both 2.1 and 2.2 show other versions).

Verify that one line with "php_extension hash must be installed and enabled" is shown.

If you can pick other versions different from 2.3 in the drop-down menu (only in 21 and 22 stable), do it.

Verify other versions do not show the "php extension hash" requirement.
Show
Go to admin -> server -> environement In the "Moodle version" dropdown menu pick 2.3 (note it's the unique value in master right now, but both 2.1 and 2.2 show other versions). Verify that one line with "php_extension hash must be installed and enabled" is shown. If you can pick other versions different from 2.3 in the drop-down menu (only in 21 and 22 stable), do it. Verify other versions do not show the "php extension hash" requirement.

Although it's installed by default since PHP 5.1, so 99% of sites out there will have it... I think it's ok to check for it in environmental checks.

Right now we are using it in a few (non-super-critical) places... but with ~~MDL-30698~~, it will be used by forums... so better check for it @ 2.3.

Ciao

has been marked as being related by

MDL-32000 RADIUS authentification

is a QA test written for

MDL-30698 Message-ID causes mail problem when sending to more than one person (e.g. forum subscription)

Participants:: Andrew Davis, Ankit Agarwal, Daniele Cordella, Eloy Lafuente (stronk7), Sam Hemelryk

Component watchers:: David Woloszyn, Huong Nguyen, Jake Dallimore, Meirza, Michael Hawkins, Raquel Ortega, Safat Shahin, Stevani Andolo