Permalink
Browse files

MDL-10905 iterating properly through readdir()

  • Loading branch information...
1 parent 9ec94a8 commit 9f94e60e368a717455593cffc06a7c72016b53af nicolasconnault committed Oct 15, 2007
View
@@ -43,7 +43,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
@@ -9,7 +9,7 @@
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
@@ -118,7 +118,7 @@ function delete_dir_contents ($dir,$excludeddir="") {
}
// 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;
}
@@ -195,7 +195,7 @@ function list_directories ($rootdir) {
$results = null;
$dir = opendir($rootdir);
- while ($file=readdir($dir)) {
+ while (false !== ($file=readdir($dir))) {
if ($file=="." || $file=="..") {
continue;
}
@@ -214,7 +214,7 @@ function list_directories_and_files ($rootdir) {
$results = "";
$dir = opendir($rootdir);
- while ($file=readdir($dir)) {
+ while (false !== ($file=readdir($dir))) {
if ($file=="." || $file=="..") {
continue;
}
@@ -301,7 +301,7 @@ function backup_copy_dir($from_file,$to_file) {
$status = mkdir($to_file,$CFG->directorypermissions);
}
$dir = opendir($from_file);
- while ($file=readdir($dir)) {
+ while (false !== ($file=readdir($dir))) {
if ($file=="." || $file=="..") {
continue;
}
@@ -57,7 +57,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;
}
@@ -55,7 +55,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
@@ -472,7 +472,7 @@ function hotpot_add_chain(&$hp) {
// 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)) {
$hp->references[] = "$xml_quiz->reference/$file";
}
@@ -28,7 +28,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 = '';
@@ -70,7 +70,7 @@ function ims_deploy_folder($file, $all='') {
$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
@@ -3161,7 +3161,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)) {
@@ -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]) {
@@ -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; }
Oops, something went wrong.

0 comments on commit 9f94e60

Please sign in to comment.