Merge branch 'MDL-25637'
authorSam Hemelryk <sam@moodle.com>
Mon, 24 Jan 2011 08:20:51 +0000 (16:20 +0800)
committerSam Hemelryk <sam@moodle.com>
Mon, 24 Jan 2011 08:20:51 +0000 (16:20 +0800)
14 files changed:
auth/shibboleth/logout.php
backup/lib.php
backup/restorelib.php
mod/feedback/item/info/lib.php
mod/feedback/item/multichoice/lib.php
mod/feedback/item/multichoicerated/lib.php
mod/feedback/item/textarea/lib.php
mod/feedback/item/textfield/lib.php
mod/glossary/import.php
mod/lesson/importppt.php
mod/wiki/diff/difflib.php
mod/wiki/parser/markups/nwiki.php
question/format/blackboard_six/format.php
question/type/calculatedsimple/edit_calculatedsimple_form.php

index b451720..c5c5410 100644 (file)
@@ -12,7 +12,7 @@ require_once($CFG->dirroot."/auth/shibboleth/auth.php");
 // Find out whether host supports https
 $protocol = 'http://';
 if ( isset($_SERVER['HTTPS']) && !empty($_SERVER['HTTPS']) && $_SERVER['HTTPS'] == 'on'){
-       $protocol = 'https://';
+    $protocol = 'https://';
 }
 
 // Front channel logout
@@ -197,11 +197,12 @@ function LogoutNotification($SessionID){
 /*****************************************************************************/
 
 // Same function as in adodb, but cannot be used for file session for some reason...
-function unserializesession( $serialized_string ){
-    $variables = array( );
-    $a = preg_split( "/(\w+)\|/", $serialized_string, -1, PREG_SPLIT_NO_EMPTY | PREG_SPLIT_DELIM_CAPTURE );
-    for( $i = 0; $i < count( $a ); $i = $i+2 ) {
-            $variables[$a[$i]] = unserialize( $a[$i+1] );
+function unserializesession($serialized_string) {
+    $variables = array();
+    $a = preg_split("/(\w+)\|/", $serialized_string, -1, PREG_SPLIT_NO_EMPTY | PREG_SPLIT_DELIM_CAPTURE);
+    $counta = count($a);
+    for ($i = 0; $i < $counta; $i = $i+2) {
+            $variables[$a[$i]] = unserialize($a[$i+1]);
     }
-    return( $variables );
+    return $variables;
 }
index 865430b..3a4e5da 100644 (file)
         }
 
         // Loop through all directory entries, and construct two temporary arrays containing files and sub directories
-        while(false !== ($entry = readdir($handle))) {
+        while (false !== ($entry = readdir($handle))) {
             if (is_dir($dir. $slash .$entry) && $entry != ".." && $entry != "." && $entry != $excludeddir) {
                 $dir_subdirs[] = $dir. $slash .$entry;
             }
         }
 
         // Delete all files in the curent directory return false and halt if a file cannot be removed
-        for($i=0; $i<count($dir_files); $i++) {
+        $countdirfiles = count($dir_files);
+        for ($i=0; $i<$countdirfiles; $i++) {
             chmod($dir_files[$i], $CFG->directorypermissions);
             if (((unlink($dir_files[$i]))) == FALSE) {
                 return false;
         }
 
         // Empty sub directories and then remove the directory
-        for($i=0; $i<count($dir_subdirs); $i++) {
+        $countdirsubdirs = count($dir_subdirs);
+        for($i=0; $i<$countdirsubdirs; $i++) {
             chmod($dir_subdirs[$i], $CFG->directorypermissions);
             if (delete_dir_contents($dir_subdirs[$i]) == FALSE) {
                 return false;
index d9ad84b..654a87f 100644 (file)
                                                 $tagsarr = $info['BLOG']['#']['BLOG_TAGS']['0']['#']['BLOG_TAG'];
                                                 //Iterate over tags
                                                 $tags = array();
-                                                for($i = 0; $i < sizeof($tagsarr); $i++) {
+                                                $sizetagsarr = sizeof($tagsarr);
+                                                for ($i = 0; $i < $sizetagsarr; $i++) {
                                                     $tag_info = $tagsarr[$i];
                                                     ///traverse_xmlize($tag_info);                        //Debug
                                                     ///print_object ($GLOBALS['traverse_array']);         //Debug
index c6ce954..b6311a8 100644 (file)
@@ -128,9 +128,10 @@ class feedback_item_info extends feedback_item_base {
     function print_analysed($item, $itemnr = '', $groupid = false, $courseid = false) {
         $analysed_item = $this->get_analysed($item, $groupid, $courseid);
         $data = $analysed_item->data;
-        if(is_array($data)) {
+        if (is_array($data)) {
             echo '<tr><th colspan="2" align="left">'. $itemnr . '&nbsp;('. $item->label .') ' . $item->name .'</th></tr>';
-            for($i = 0; $i < sizeof($data); $i++) {
+            $sizeofdata = sizeof($data);
+            for ($i = 0; $i < $sizeofdata; $i++) {
                 echo '<tr><td colspan="2" valign="top" align="left">-&nbsp;&nbsp;' . str_replace("\n", '<br />', $data[$i]->show) . '</td></tr>';
             }
         }
@@ -144,11 +145,12 @@ class feedback_item_info extends feedback_item_base {
         $worksheet->write_string($rowOffset, 0, $item->label, $xlsFormats->head2);
         $worksheet->write_string($rowOffset, 1, $item->name, $xlsFormats->head2);
         $data = $analysed_item->data;
-        if(is_array($data)) {
+        if (is_array($data)) {
             // $worksheet->setFormat("<l><ro2><vo>");
             $worksheet->write_string($rowOffset, 2, $data[0]->show, $xlsFormats->value_bold);
             $rowOffset++;
-            for($i = 1; $i < sizeof($data); $i++) {
+            $sizeofdata = sizeof($data);
+            for ($i = 1; $i < $sizeofdata; $i++) {
                 // $worksheet->setFormat("<l><vo>");
                 $worksheet->write_string($rowOffset, 2, $data[$i]->show, $xlsFormats->default);
                 $rowOffset++;
index a663c97..ec7102d 100644 (file)
@@ -119,14 +119,15 @@ class feedback_item_multichoice extends feedback_item_base {
 
         $analysedAnswer = array();
         if($info->subtype == 'c') {
-            for($i = 1; $i <= sizeof($answers); $i++) {
+            $sizeofanswers = sizeof($answers);
+            for ($i = 1; $i <= $sizeofanswers; $i++) {
                 $ans = null;
                 $ans->answertext = $answers[$i-1];
                 $ans->answercount = 0;
-                foreach($values as $value) {
+                foreach ($values as $value) {
                     //ist die Antwort gleich dem index der Antworten + 1?
                     $vallist = explode(FEEDBACK_MULTICHOICE_LINE_SEP, $value->value);
-                    foreach($vallist as $val) {
+                    foreach ($vallist as $val) {
                         if ($val == $i) {
                            $ans->answercount++;
                         }
@@ -136,11 +137,12 @@ class feedback_item_multichoice extends feedback_item_base {
                 $analysedAnswer[] = $ans;
             }
         }else {
-            for($i = 1; $i <= sizeof($answers); $i++) {
+            $sizeofanswers = sizeof($answers);
+            for ($i = 1; $i <= $sizeofanswers; $i++) {
                 $ans = null;
                 $ans->answertext = $answers[$i-1];
                 $ans->answercount = 0;
-                foreach($values as $value) {
+                foreach ($values as $value) {
                     //ist die Antwort gleich dem index der Antworten + 1?
                     if ($value->value == $i) {
                         $ans->answercount++;
@@ -159,26 +161,30 @@ class feedback_item_multichoice extends feedback_item_base {
 
         $printval = '';
 
-        if(!isset($value->value)) return $printval;
+        if (!isset($value->value)) {
+            return $printval;
+        }
 
         // @list($presentation) = explode(FEEDBACK_RADIO_ADJUST_SEP, $item->presentation); //remove the adjustment-info
 
         $presentation = explode (FEEDBACK_MULTICHOICE_LINE_SEP, $info->presentation);
 
-        if($info->subtype == 'c') {
+        if ($info->subtype == 'c') {
             $vallist = array_values(explode (FEEDBACK_MULTICHOICE_LINE_SEP, $value->value));
-            for($i = 0; $i < sizeof($vallist); $i++) {
-                for($k = 0; $k < sizeof($presentation); $k++) {
-                    if($vallist[$i] == ($k + 1)) {//Die Werte beginnen bei 1, das Array aber mit 0
+            $sizeofvallist = sizeof($vallist);
+            $sizeofpresentation = sizeof($presentation);
+            for ($i = 0; $i < $sizeofvallist; $i++) {
+                for ($k = 0; $k < $sizeofpresentation; $k++) {
+                    if ($vallist[$i] == ($k + 1)) {//Die Werte beginnen bei 1, das Array aber mit 0
                         $printval .= trim($presentation[$k]) . chr(10);
                         break;
                     }
                 }
             }
-        }else {
+        } else {
             $index = 1;
             foreach($presentation as $pres){
-                if($value->value == $index){
+                if ($value->value == $index){
                     $printval = $pres;
                     break;
                 }
@@ -231,8 +237,9 @@ class feedback_item_multichoice extends feedback_item_base {
         //frage schreiben
         $worksheet->write_string($rowOffset, 0, $item->label, $xlsFormats->head2_green);
         $worksheet->write_string($rowOffset, 1, $analysed_item[1], $xlsFormats->head2_green);
-        if(is_array($data)) {
-            for($i = 0; $i < sizeof($data); $i++) {
+        if (is_array($data)) {
+            $sizeofdata = sizeof($data);
+            for ($i = 0; $i < $sizeofdata; $i++) {
                 $aData = $data[$i];
 
                 // $worksheet->setFormat("<l><f><ro2><vo><c:blue>");
@@ -244,7 +251,7 @@ class feedback_item_multichoice extends feedback_item_base {
                 $worksheet->write_number($rowOffset + 2, $i + 2, $aData->quotient, $xlsFormats->procent);
             }
         }
-        $rowOffset +=;
+        $rowOffset += 3;
         return $rowOffset;
     }
 
@@ -560,14 +567,15 @@ class feedback_item_multichoice extends feedback_item_base {
     }
 
     function item_arrayToString($value) {
-        if(!is_array($value)) {
+        if (!is_array($value)) {
             return $value;
         }
         $retval = '';
         $arrvals = array_values($value);
         $arrvals = clean_param($arrvals, PARAM_INT);  //prevent sql-injection
         $retval = $arrvals[0];
-        for($i = 1; $i < sizeof($arrvals); $i++) {
+        $sizeofarrvals = sizeof($arrvals);
+        for ($i = 1; $i < $sizeofarrvals; $i++) {
             $retval .= FEEDBACK_MULTICHOICE_LINE_SEP.$arrvals[$i];
         }
         return $retval;
index af4cf02..77a2b04 100644 (file)
@@ -120,8 +120,8 @@ class feedback_item_multichoicerated extends feedback_item_base {
         //schleife ueber den Werten und ueber die Antwortmoeglichkeiten
 
         $analysedAnswer = array();
-
-        for($i = 1; $i <= sizeof($lines); $i++) {
+        $sizeoflines = sizeof($lines);
+        for($i = 1; $i <= $sizeoflines; $i++) {
             $item_values = explode(FEEDBACK_MULTICHOICERATED_VALUE_SEP, $lines[$i-1]);
             $ans = null;
             $ans->answertext = $item_values[1];
@@ -215,7 +215,8 @@ class feedback_item_multichoicerated extends feedback_item_base {
         $worksheet->write_string($rowOffset, 1, $analysed_item[1], $xlsFormats->head2);
         if(is_array($data)) {
             $avg = 0.0;
-            for($i = 0; $i < sizeof($data); $i++) {
+            $sizeofdata = sizeof($data);
+            for($i = 0; $i < $sizeofdata; $i++) {
                 $aData = $data[$i];
 
                 // $worksheet->setFormat("<l><f><ro2><vo><c:blue>");
index 161b482..1b4cd13 100644 (file)
@@ -142,7 +142,8 @@ class feedback_item_textarea extends feedback_item_base {
                 $worksheet->write_string($rowOffset, 2, $data[0], $xlsFormats->value_bold);
             }
             $rowOffset++;
-            for($i = 1; $i < sizeof($data); $i++) {
+            $sizeofdata = sizeof($data);
+            for($i = 1; $i < $sizeofdata; $i++) {
                 // $worksheet->setFormat("<l><vo>");
                 $worksheet->write_string($rowOffset, 2, $data[$i], $xlsFormats->default);
                 $rowOffset++;
index e4cb6d2..74f9b88 100644 (file)
@@ -139,7 +139,8 @@ class feedback_item_textfield extends feedback_item_base {
             // $worksheet->setFormat("<l><ro2><vo>");
             $worksheet->write_string($rowOffset, 2, $data[0], $xlsFormats->value_bold);
             $rowOffset++;
-            for($i = 1; $i < sizeof($data); $i++) {
+            $sizeofdata = sizeof($data);
+            for($i = 1; $i < $sizeofdata; $i++) {
                 // $worksheet->setFormat("<l><vo>");
                 $worksheet->write_string($rowOffset, 2, $data[$i], $xlsFormats->default);
                 $rowOffset++;
index 5b458d4..9a20c88 100644 (file)
@@ -187,7 +187,8 @@ if ($xml = glossary_read_imported_file($result)) {
     }
 
     $xmlentries = $xml['GLOSSARY']['#']['INFO'][0]['#']['ENTRIES'][0]['#']['ENTRY'];
-    for($i = 0; $i < sizeof($xmlentries); $i++) {
+    $sizeofxmlentries = sizeof($xmlentries);
+    for($i = 0; $i < $sizeofxmlentries; $i++) {
         // Inserting the entries
         $xmlentry = $xmlentries[$i];
         unset($newentry);
@@ -247,7 +248,8 @@ if ($xml = glossary_read_imported_file($result)) {
             $importedentries++;
 
             $xmlaliases = @$xmlentry['#']['ALIASES'][0]['#']['ALIAS']; // ignore missing ALIASES
-            for($k = 0; $k < sizeof($xmlaliases); $k++) {
+            $sizeofxmlaliases = sizeof($xmlaliases);
+            for($k = 0; $k < $sizeofxmlaliases; $k++) {
             /// Importing aliases
                 $xmlalias = $xmlaliases[$k];
                 $aliasname = $xmlalias['#']['NAME'][0]['#'];
@@ -263,7 +265,8 @@ if ($xml = glossary_read_imported_file($result)) {
             if (!empty($data->catsincl)) {
                 // If the categories must be imported...
                 $xmlcats = @$xmlentry['#']['CATEGORIES'][0]['#']['CATEGORY']; // ignore missing CATEGORIES
-                for($k = 0; $k < sizeof($xmlcats); $k++) {
+                $sizeofxmlcats = sizeof($xmlcats);
+                for($k = 0; $k < $sizeofxmlcats; $k++) {
                     $xmlcat = $xmlcats[$k];
 
                     $newcat = new stdClass();
index 982ee40..243ff3d 100644 (file)
@@ -110,7 +110,8 @@ if ($data = $mform->get_data()) {
             // want to allow <b><i>(maybe more) tags but when we do that
             // the preg_match messes up.
             preg_match_all("/(<([\w]+)[^>]*>)([^<\\2>]*)(<\/\\2>)/", $string, $matches);
-            for($i = 0; $i < count($matches[1]); $i++) { // go through all of our div matches
+            $countmatches = count($matches[1]);
+            for($i = 0; $i < $countmatches; $i++) { // go through all of our div matches
 
                 $class = lesson_importppt_isolate_class($matches[1][$i]); // first step in isolating the class
 
@@ -207,4 +208,4 @@ echo $OUTPUT->heading_with_help($strimportppt, 'importppt', 'lesson');
 echo $OUTPUT->box_start('generalbox boxaligncenter');
 $mform->display();
 echo $OUTPUT->box_end();
-echo $OUTPUT->footer();
\ No newline at end of file
+echo $OUTPUT->footer();
index d7389ed..61315e6 100644 (file)
@@ -300,8 +300,9 @@ class ouwiki_line {
         $pos=0;
         while(true) {
             // Find a non-space
-            for(;$pos < strlen($data) && substr($data,$pos,1)===' ';$pos++) ;
-            if($pos==strlen($data)) {
+            $strlendata = strlen($data);
+            for(;$pos < $strlendata && substr($data,$pos,1)===' ';$pos++) ;
+            if($pos==$strlendata) {
                 // No more content
                 break;
             }
@@ -527,7 +528,8 @@ class ouwiki_changes {
         //    resulting array will be -,-,3,3,3,-,0
         $squidges=array();
         $lowest=0;
-        for($index1=count($diff);$index1>=1;$index1--) {
+        $countdiff = count($diff);
+        for($index1=$countdiff;$index1>=1;$index1--) {
             $index2=$diff[$index1];
             if($index2===0) {
                 $squidges[$index1]=$lowest;
index 29d4bef..fd1324e 100644 (file)
@@ -142,7 +142,8 @@ class nwiki_parser extends wiki_markup_parser {
         $cells = array();
         foreach($normalcells as $nc) {
             $headercells = explode("!!", $nc);
-            for($i = 0; $i < count($headercells); $i++) {
+            $countheadercells = count($headercells);
+            for($i = 0; $i < $countheadercells; $i++) {
                 $cells[] = array($type, $headercells[$i]);
                 $type = 'header';
             }
index da05ba1..300a0e7 100644 (file)
@@ -73,7 +73,8 @@ class qformat_blackboard_six extends qformat_default {
         }
 
         // Delete all files in the curent directory return false and halt if a file cannot be removed
-        for($i=0; $i<count($dir_files); $i++) {
+        $countdir_files = count($dir_files);
+        for($i=0; $i<$countdir_files; $i++) {
             chmod($dir_files[$i], $CFG->directorypermissions);
             if (((unlink($dir_files[$i]))) == FALSE) {
                 return false;
@@ -81,7 +82,8 @@ class qformat_blackboard_six extends qformat_default {
         }
 
         // Empty sub directories and then remove the directory
-        for($i=0; $i<count($dir_subdirs); $i++) {
+        $countdir_subdirs = count($dir_subdirs);
+        for($i=0; $i<$countdir_subdirs; $i++) {
             chmod($dir_subdirs[$i], $CFG->directorypermissions);
             if ($this->clean_temp_dir($dir_subdirs[$i]) == FALSE) {
                 return false;
index fe3e817..0275924 100644 (file)
@@ -168,7 +168,8 @@ class question_edit_calculatedsimple_form extends question_edit_form {
                     $calcmax = optional_param('calcmax', '', PARAM_NUMBER);
                     $oldoptions  = optional_param('defoptions', '', PARAM_RAW);
                     $newdatasetvalues = false ;
-                    for($key = 1 ; $key <= sizeof($olddef) ; $key++) {
+                    $sizeofolddef = sizeof($olddef);
+                    for($key = 1; $key <= $sizeofolddef; $key++) {
                         $def = $olddef[$key] ;
                         $this->datasetdefs[$def]= new stdClass ;
                         $this->datasetdefs[$def]->type = 1;