MDL-18644 Both if and else end with return therefore another return afterwards is

obsolete. Also, "return true" at top of the function renders all the
following code useless
This commit is contained in:
mudrd8mz
2009-03-23 09:47:45 +00:00
parent 68a3d81779
commit f847d9f546
+1 -1
View File
@@ -40,6 +40,7 @@ class qformat_blackboard_6 extends qformat_default {
function clean_temp_dir($dir='') {
// for now we will just say everything happened okay note
// that a mess may be piling up in $CFG->dataroot/temp/bbquiz_import
// TODO return true at top of the function renders all the following code useless
return true;
if ($dir == '') {
@@ -184,7 +185,6 @@ class qformat_blackboard_6 extends qformat_default {
} else {
return $filearray;
}
return false;
}
}
else {