MDL-10905 iterating correctly with readdir (as shown in php doc)

This commit is contained in:
nicolasconnault 2007-10-15 05:18:58 +00:00
parent 546526ed42
commit ed818bbdbf
13 changed files with 19 additions and 19 deletions

View File

@ -54,7 +54,7 @@ function delete_subdirectories($rootdir) {
$dir = opendir($rootdir);
while ($file = readdir($dir)) {
while (false !== ($file = readdir($dir))) {
if ($file != '.' and $file != '..') {
$fullfile = $rootdir .'/'. $file;
if (filetype($fullfile) == 'dir') {

View File

@ -9,7 +9,7 @@ require_once($CFG->dirroot.'/backup/bb/xsl_emulate_xslt.inc');
function get_subdirs($directory){
$opendirectory = opendir( $directory );
while($filename = readdir($opendirectory)) {
while(false !== ($filename = readdir($opendirectory))) {
if (is_dir($directory.$filename) and $filename != ".." and $filename != "."){
$subdirs[] = $filename;
}

View File

@ -125,7 +125,7 @@
}
// Loop through all directory entries, and construct two temporary arrays containing files and sub directories
while($entry = readdir($handle)) {
while(false !== ($entry = readdir($handle))) {
if (is_dir($dir. $slash .$entry) && $entry != ".." && $entry != "." && $entry != $excludeddir) {
$dir_subdirs[] = $dir. $slash .$entry;
}
@ -202,7 +202,7 @@
$results = null;
$dir = opendir($rootdir);
while ($file=readdir($dir)) {
while (false !== ($file=readdir($dir))) {
if ($file=="." || $file=="..") {
continue;
}
@ -221,7 +221,7 @@
$results = "";
$dir = opendir($rootdir);
while ($file=readdir($dir)) {
while (false !== ($file=readdir($dir))) {
if ($file=="." || $file=="..") {
continue;
}
@ -308,7 +308,7 @@
$status = check_dir_exists($to_file, true, true);
}
$dir = opendir($from_file);
while ($file=readdir($dir)) {
while (false !== ($file=readdir($dir))) {
if ($file=="." || $file=="..") {
continue;
}

View File

@ -62,7 +62,7 @@ function assignment_upgrade($oldversion) {
notify("Moving location of assignment files...");
$basedir = opendir("$CFG->dataroot");
while ($dir = readdir($basedir)) {
while (false !== ($dir = readdir($basedir))) {
if ($dir == "." || $dir == ".." || $dir == "users") {
continue;
}

View File

@ -60,7 +60,7 @@ function assignment_upgrade($oldversion) {
notify("Moving location of assignment files...");
$basedir = opendir("$CFG->dataroot");
while ($dir = readdir($basedir)) {
while (false !== ($dir = readdir($basedir))) {
if ($dir == "." || $dir == ".." || $dir == "users") {
continue;
}

View File

@ -472,7 +472,7 @@ function hotpot_add_chain(&$hotpot) {
// get list of hotpot files in this folder
if ($dh = @opendir($xml_quiz->filepath)) {
while ($file = @readdir($dh)) {
while (false !== ($file = @readdir($dh))) {
if (preg_match('/\.(jbc|jcl|jcw|jmt|jmx|jqz|htm|html)$/', $file)) {
$hotpot->references[] = "$xml_quiz->reference/$file";
}

View File

@ -30,7 +30,7 @@
/// Loops though dir building a list of all relevent entries. Ignores files.
/// Asks for deploy if admin user AND no serialized file found.
while (false != ($filename = readdir($repository_dir))) {
while (false !== ($filename = readdir($repository_dir))) {
if ($filename != '.' && $filename != '..' && is_dir("$CFG->repository/$directory/$filename")) {
unset($item);
$item->type = '';

View File

@ -7,7 +7,7 @@
// Moodle - Modular Object-Oriented Dynamic Learning Environment //
// http://moodle.com //
// //
// Copyright (C) 1999 onwards Martin Dougiamas http://dougiamas.com //
// Copyright (C) 1999 onwards Martin Dougiamas http://dougiamas.com //
// (C) 2001-3001 Eloy Lafuente (stronk7) http://contiento.com //
// //
// This program is free software; you can redistribute it and/or modify //
@ -68,7 +68,7 @@
$dirpath = "$CFG->repository/$file";
$dir = opendir($dirpath);
while (false != ($filename = readdir($dir))) {
while (false !== ($filename = readdir($dir))) {
if ($filename != '.' && $filename != '..') {
$path = $dirpath.'/'.$filename;
if (is_dir($path) && file_exists("$path/imsmanifest.xml")) {

View File

@ -3189,7 +3189,7 @@ function ewiki_eventually_initialize(&$id, &$data, &$action) {
$path=EWIKI_INIT_PAGES;
if (!empty($path)) {
if ($dh = @opendir($path=EWIKI_INIT_PAGES)) {
while ($filename = readdir($dh)) {
while (false !== ($filename = readdir($dh))) {
#### MOODLE CHANGE TO SOLVE BUG #3830. Original doesn't support dots in names.
//Orig->if (preg_match('/^(['.EWIKI_CHARS_U.']+['.EWIKI_CHARS_L.']+\w*)+/', $filename)) {
if ($filename == clean_filename($filename) && !is_dir($path.'/'.$filename)) {

View File

@ -79,7 +79,7 @@ function ewiki_initialization_wizard($id, &$data, &$action) {
else {
ewiki_database("INIT", array());
if ($dh = @opendir($path=EWIKI_INIT_PAGES)) {
while ($filename = readdir($dh)) {
while (false !== ($filename = readdir($dh))) {
if (preg_match('/^(['.EWIKI_CHARS_U.']+['.EWIKI_CHARS_L.']+\w*)+/', $filename)) {
$found = ewiki_database("FIND", array($filename));
if (! $found[$filename]) {
@ -119,4 +119,4 @@ function ewiki_initialization_wizard($id, &$data, &$action) {
}
?>
?>

View File

@ -141,7 +141,7 @@ function ewiki_init_spages($dirs, $idprep="") {
#-- read in one directory
$dh = opendir($dir);
while ($fn = readdir($dh)) {
while (false !== ($fn = readdir($dh))) {
#-- skip over . and ..
if ($fn[0] == ".") { continue; }
@ -179,4 +179,4 @@ function ewiki_init_spages($dirs, $idprep="") {
?>
?>

View File

@ -60,7 +60,7 @@ class qformat_blackboard_6 extends qformat_default {
}
// Loop through all directory entries, and construct two temporary arrays containing files and sub directories
while($entry = readdir($handle)) {
while(false !== ($entry = readdir($handle))) {
if (is_dir($dir. $slash .$entry) && $entry != ".." && $entry != ".") {
$dir_subdirs[] = $dir. $slash .$entry;
}

View File

@ -434,7 +434,7 @@ class qformat_coursetestmanager extends qformat_default {
function fulldelete($location) {
if (is_dir($location)) {
$currdir = opendir($location);
while ($file = readdir($currdir)) {
while (false !== ($file = readdir($currdir))) {
if ($file <> ".." && $file <> ".") {
$fullfile = $location."/".$file;
if (is_dir($fullfile)) {