From: martinlanghoff <martinlanghoff>
Date: Tue, 2 May 2006 05:20:35 +0000 (+0000)
Subject: mod/chat - fix another instance where empty $USER->lastIP was affecting us
X-Git-Url: http://git.mjollnir.org/gw?a=commitdiff_plain;h=274f009184d340c4d7d278ded9f5693f27e5820f;p=moodle.git

mod/chat - fix another instance where empty $USER->lastIP was affecting us

The original commit was:

mod/chat - use empty value instead of erroring out if $USER->lastIP and getremoteaddr() don't yield any interesting value

Sometimes we can't get any useful IP for the user -- and this may even be a
valid situation. So fudge an empty value -- the DB needs it as the IP field
is NOT NULL.
---

diff --git a/mod/chat/lib.php b/mod/chat/lib.php
index 31382401fa..8a4051b07a 100644
--- a/mod/chat/lib.php
+++ b/mod/chat/lib.php
@@ -416,6 +416,17 @@ function chat_login_user($chatid, $version, $groupid, $course) {
         $chatuser->course   = $course->id; //caching - needed for current_language too
         $chatuser->lang     = current_language(); //caching - to resource intensive to find out later
 
+        // Sometimes $USER->lastIP is not setup properly
+        // during login. Update with current value if possible
+        // or provide a dummy value for the db
+        if (empty($chatuser->ip)) {
+            $chatuser->ip = getremoteaddr();
+            if (empty($chatuser->ip)) {
+                $chatuser->ip = '';
+            }
+        }
+
+
         if (!insert_record('chat_users', $chatuser)) {
             return false;
         }