Merge branch 'MDL-25708' of git://github.com/stronk7/moodle
authorPetr Skoda <commits@skodak.org>
Mon, 31 Jan 2011 13:21:53 +0000 (14:21 +0100)
committerPetr Skoda <commits@skodak.org>
Mon, 31 Jan 2011 13:21:53 +0000 (14:21 +0100)
1  2 
message/lib.php
mod/resource/db/upgradelib.php

diff --combined message/lib.php
@@@ -140,7 -140,7 +140,7 @@@ function message_print_participants($co
  
      $countparticipants = count_enrolled_users($context);
      $participants = get_enrolled_users($context, '', 0, 'u.*', '', $page*MESSAGE_CONTACTS_PER_PAGE, MESSAGE_CONTACTS_PER_PAGE);
 -    
 +
      $pagingbar = new paging_bar($countparticipants, $page, MESSAGE_CONTACTS_PER_PAGE, $PAGE->url, 'page');
      echo $OUTPUT->render($pagingbar);
  
@@@ -269,24 -269,21 +269,21 @@@ function message_get_contacts($user1=nu
                   GROUP BY $userfields
                   ORDER BY u.firstname ASC";
  
-     if ($rs = $DB->get_recordset_sql($contactsql, array($user1->id, $user1->id))){
-         foreach($rs as $rd){
-             if($rd->lastaccess >= $timefrom){
-                 // they have been active recently, so are counted online
-                 $onlinecontacts[] = $rd;
-             }else{
-                 $offlinecontacts[] = $rd;
-             }
+     $rs = $DB->get_recordset_sql($contactsql, array($user1->id, $user1->id));
+     foreach ($rs as $rd) {
+         if ($rd->lastaccess >= $timefrom) {
+             // they have been active recently, so are counted online
+             $onlinecontacts[] = $rd;
+         } else {
+             $offlinecontacts[] = $rd;
+         }
  
-             if (!empty($user2) && $user2->id==$rd->id) {
-                 $user2->iscontact = true;
-             }
+         if (!empty($user2) && $user2->id==$rd->id) {
+             $user2->iscontact = true;
          }
-         unset($rd);
-         $rs->close();
      }
+     $rs->close();
  
      // get messages from anyone who isn't in our contact list and count the number
      // of messages we have from each of them
                    GROUP BY $userfields
                    ORDER BY u.firstname ASC";
  
-     if($rs = $DB->get_recordset_sql($strangersql, array($USER->id))){
-         foreach($rs as $rd){
-             $strangers[] = $rd;
-         }
-         unset($rd);
-         $rs->close();
+     $rs = $DB->get_recordset_sql($strangersql, array($USER->id));
+     foreach ($rs as $rd) {
+         $strangers[] = $rd;
      }
+     $rs->close();
  
      return array($onlinecontacts, $offlinecontacts, $strangers);
  }
@@@ -527,11 -522,7 +522,11 @@@ function message_print_search($advanced
  
      $doingsearch = false;
      if ($frm) {
 -        $doingsearch = !empty($frm->combinedsubmit) || !empty($frm->keywords) || (!empty($frm->personsubmit) and !empty($frm->name));
 +        if (confirm_sesskey()) {
 +            $doingsearch = !empty($frm->combinedsubmit) || !empty($frm->keywords) || (!empty($frm->personsubmit) and !empty($frm->name));
 +        } else {
 +            $frm = false;
 +        }
      }
  
      if (!empty($frm->combinedsearch)) {
@@@ -1572,7 -1563,7 +1567,7 @@@ function message_post_message($userfrom
          $eventdata->fullmessage      = $message;
          $eventdata->fullmessagehtml  = '';
      }
 -    
 +
      $eventdata->fullmessageformat = $format;
      $eventdata->smallmessage     = strip_tags($message);//strip just in case there are is any html that would break the popup notification
  
      if (!empty($eventdata->fullmessagehtml)) {
          $eventdata->fullmessagehtml .= "<br /><br />---------------------------------------------------------------------<br />".$emailtagline;
      }
 -    
 +
      $eventdata->timecreated     = time();
      return message_send($eventdata);
  }
@@@ -1773,7 -1764,7 +1768,7 @@@ function message_mark_messages_read($to
  */
  function message_mark_message_read($message, $timeread, $messageworkingempty=false) {
      global $DB;
 -    
 +
      $message->timeread = $timeread;
  
      $messageid = $message->id;
@@@ -42,7 -42,9 +42,9 @@@ function resource_20_migrate() 
      $withrelativelinks = array('text/html', 'text/xml', 'application/xhtml+xml', 'application/x-shockwave-flash');
      // note: pdf doc and other types may contain links too, but we do not support relative links there
  
-     if (!$candidates = $DB->get_recordset('resource_old', array('type'=>'file', 'migrated'=>0))) {
+     $candidates = $DB->get_recordset('resource_old', array('type'=>'file', 'migrated'=>0));
+     if (!$candidates->valid()) {
+         $candidates->close(); // Not going to iterate (but exit), close rs
          return;
      }
  
          $siteid = get_site()->id;
          $fs = get_file_storage();
  
 -        if (strpos($path, 'LOCALPATH') === 0) {
 +        if (empty($candidate->cmid)) {
 +            // skip borked records
 +            continue;
 +
 +        } else if (strpos($path, 'LOCALPATH') === 0) {
              // ignore not maintained local files - sorry
              continue;