Details
-
Type:
Bug
-
Status:
Closed
-
Priority:
Minor
-
Resolution: Fixed
-
Affects Version/s: 1.9.4
-
Component/s: Global search
-
Labels:None
-
Affected Branches:MOODLE_19_STABLE
-
Fixed Branches:MOODLE_19_STABLE, MOODLE_20_STABLE
Description
The scripts in the global search folder contain several instances of the following code:
$separator = (array_key_exists('WINDIR', $_SERVER)) ? ';' : ':' ;
ini_set('include_path', $CFG->dirroot.'\search'.$separator.ini_get('include_path'));
But it seems unnecessary when we should use the PATH_SEPARATOR instead:
ini_set('include_path', $CFG->dirroot.'\search'.PATH_SEPARATOR.ini_get('include_path'));
Scripts affected:
- search/add.php
- search/delete.php
- search/indexer.php
- search/indexersplash.php
- search/update.php
- search/tests/index.php
sure, much better indeed...