MDL-18644 Bugs found by syntax checker, merged from MOODLE_19_STABLE

anovikov · Mar 23, 2009 · 0cd2553 · 0cd2553
1 parent 081a9b0
commit 0cd2553
Showing 1 changed file with 4 additions and 4 deletions.
diff --git a/question/format/blackboard_six/format.php b/question/format/blackboard_six/format.php
@@ -40,6 +40,7 @@ function check_and_create_import_dir($unique_code) {
     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 == '') {
@@ -158,10 +159,10 @@ function readdata($filename) {
         $ext = substr($this->realfilename, strpos($this->realfilename,'.'), strlen($this->realfilename)-1);      
         if ($ext=='.dat') {
             if (!is_readable($filename)) {
-                print_error('filenotreadable', 'error');	
-            }		
+                print_error('filenotreadable', 'error');    
+            }       
             return file($filename);
-        }	  
+        }     
 
         $unique_code = time();
         $temp_dir = $CFG->dataroot."/temp/bbquiz_import/".$unique_code;
@@ -184,7 +185,6 @@ function readdata($filename) {
                             } else {
                                 return $filearray;
                             }
-                            return false;        
                         }
                     }
                     else {