Skip to content

Commit

Permalink
MDL-31166 do not try to cleanup data root paths in CLI install script
Browse files Browse the repository at this point in the history
PARAM_TYPE can not be used for full OS paths. Admins have to enter correct data there, if not they get directory creation error.
  • Loading branch information
skodak committed Jan 15, 2012
1 parent f70f6e9 commit 45d88e5
Showing 1 changed file with 1 addition and 6 deletions.
7 changes: 1 addition & 6 deletions admin/cli/install.php
Original file line number Diff line number Diff line change
Expand Up @@ -337,12 +337,7 @@


//We need dataroot before lang download
$dataroot = clean_param($options['dataroot'], PARAM_PATH);
if ($dataroot !== $options['dataroot']) {
$a = (object)array('option' => 'dataroot', 'value' => $options['dataroot']);
cli_error(get_string('cliincorrectvalueerror', 'admin', $a));
}
$CFG->dataroot = $dataroot;
$CFG->dataroot = $options['dataroot'];
if ($interactive) {
cli_separator();
$i=0;
Expand Down

0 comments on commit 45d88e5

Please sign in to comment.