From f847d9f5465590dc6294cf33baf4e55c59c41008 Mon Sep 17 00:00:00 2001 From: mudrd8mz Date: Mon, 23 Mar 2009 09:47:45 +0000 Subject: [PATCH] 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 --- question/format/blackboard_6/format.php | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/question/format/blackboard_6/format.php b/question/format/blackboard_6/format.php index 57ad00c8462..69d0c19d9f7 100644 --- a/question/format/blackboard_6/format.php +++ b/question/format/blackboard_6/format.php @@ -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 {