From: tjhunt <tjhunt>
Date: Mon, 13 Mar 2006 10:37:21 +0000 (+0000)
Subject: Added a comment to explain that add_to_log is the way it is for performance reasons.
X-Git-Url: http://git.mjollnir.org/gw?a=commitdiff_plain;h=e8395a09f4d2247ec231248fd4db2f92cadcf3e0;p=moodle.git

Added a comment to explain that add_to_log is the way it is for performance reasons.
---

diff --git a/lib/datalib.php b/lib/datalib.php
index f265cb0905..f53f750954 100644
--- a/lib/datalib.php
+++ b/lib/datalib.php
@@ -2801,7 +2801,9 @@ function instance_is_visible($moduletype, $module) {
  * @param    string  $user    If log regards $user other than $USER
  */
 function add_to_log($courseid, $module, $action, $url='', $info='', $cm=0, $user=0) {
-
+    // Note that this function intentionally does not follow the normal Moodle DB access idioms.
+    // This is for a good reason: it is the most frequently used DB update function,
+    // so it has been optimised for speed.
     global $db, $CFG, $USER;
 
     if ($cm === '' || is_null($cm)) { // postgres won't translate empty string to its default