Merge branch 'MDL-25616_recent_messages' of git://github.com/andyjdavis/moodle
authorEloy Lafuente (stronk7) <stronk7@moodle.org>
Tue, 15 Feb 2011 18:26:21 +0000 (19:26 +0100)
committerEloy Lafuente (stronk7) <stronk7@moodle.org>
Tue, 15 Feb 2011 18:26:21 +0000 (19:26 +0100)
Conflicts fixed:
message/index.php
message/lib.php

1  2 
message/index.php
message/lib.php

@@@ -163,12 -161,9 +168,12 @@@ if ($currentuser && !empty($user2) && h
                  print_error('invalidsesskey');
              }
  
-             $messageid = message_post_message($user1, $user2, $data->message, FORMAT_MOODLE, 'direct');
+             $messageid = message_post_message($user1, $user2, $data->message, FORMAT_MOODLE);
              if (!empty($messageid)) {
-                 redirect($CFG->wwwroot . '/message/index.php?usergroup='.$usergroup.'&id='.$user2->id);
 +                //including the id of the user sending the message in the logged URL so the URL works for admins
 +                //note message ID may be misleading as the message may potentially get a different ID when moved from message to message_read
 +                add_to_log(SITEID, 'message', 'write', 'index.php?user='.$user1->id.'&id='.$user2->id.'&history=1#m'.$messageid, $user1->id);
+                 redirect($CFG->wwwroot . '/message/index.php?'.VIEW_PARAM.'='.$viewing.'&id='.$user2->id);
              }
          }
      }
diff --cc message/lib.php
@@@ -1548,11 -1953,13 +1956,14 @@@ function message_format_contexturl($mes
  }
  
  /**
-  * Inserts a message into the database, but also forwards it
-  * via other means if appropriate.
+  * Send a message from one user to another. Will be delivered according to the message recipients messaging preferences
+  * @param object $userfrom the message sender
+  * @param object $userto the message recipient
+  * @param string $message the message
+  * @param int $format message format such as FORMAT_PLAIN or FORMAT_HTML
 + * @return int|false the ID of the new message or false
   */
- function message_post_message($userfrom, $userto, $message, $format, $messagetype) {
+ function message_post_message($userfrom, $userto, $message, $format) {
      global $SITE, $CFG, $USER;
  
      $eventdata = new stdClass();
@@@ -1782,7 -2216,17 +2220,18 @@@ function message_mark_message_read($mes
      if (!$messageworkingempty) {
          $DB->delete_records('message_working', array('unreadmessageid' => $messageid));
      }
 -    $DB->insert_record('message_read', $message);
 +    $messagereadid = $DB->insert_record('message_read', $message);
      $DB->delete_records('message', array('id' => $messageid));
 +    return $messagereadid;
  }
+ /**
+  * A helper function that prints a formatted heading
+  * @param string $title the heading to display
+  * @param int $colspan
+  */
+ function message_print_heading($title, $colspan=3) {
+     echo html_writer::start_tag('tr');
+     echo html_writer::tag('td', $title, array('colspan' => $colspan, 'class' => 'heading'));
+     echo html_writer::end_tag('tr');
+ }