From 7874aa6eb786455228eb2dd82cc090e7b255812c Mon Sep 17 00:00:00 2001
From: Simey Lameze <lameze@gmail.com>
Date: Wed, 21 May 2014 12:26:07 +0800
Subject: [PATCH] MDL-45544 events: Add event user_profile_viewed trigger to
 user/profile.php

---
 lib/classes/event/user_profile_viewed.php | 17 +++++---
 lib/tests/event_test.php                  | 49 +++++++++++++++++++++++
 user/profile.php                          |  9 +++++
 3 files changed, 70 insertions(+), 5 deletions(-)

diff --git a/lib/classes/event/user_profile_viewed.php b/lib/classes/event/user_profile_viewed.php
index 5756c15fe22..7cfdeed504f 100644
--- a/lib/classes/event/user_profile_viewed.php
+++ b/lib/classes/event/user_profile_viewed.php
@@ -53,8 +53,9 @@ class user_profile_viewed extends base {
      * @return string
      */
     public function get_description() {
-        return 'User ' . $this->userid . ' viewed the profile for user ' . $this->relateduserid . ' in the course ' .
-            $this->courseid;
+        $desc = "The user with id '$this->userid' viewed the profile for the user with id '$this->relateduserid'";
+        $desc .= ($this->contextlevel == CONTEXT_COURSE) ? " in the course with id '$this->courseid'." : ".";
+        return $desc;
     }
 
     /**
@@ -63,7 +64,10 @@ class user_profile_viewed extends base {
      * @return \moodle_url
      */
     public function get_url() {
-        return new \moodle_url('/user/view.php', array('id' => $this->relateduserid, 'course' => $this->courseid));
+        if ($this->contextlevel == CONTEXT_COURSE) {
+            return new \moodle_url('/user/view.php', array('id' => $this->relateduserid, 'course' => $this->courseid));
+        }
+        return new \moodle_url('/user/profile.php', array('id' => $this->relateduserid));
     }
 
     /**
@@ -72,7 +76,10 @@ class user_profile_viewed extends base {
      * @return array
      */
     protected function get_legacy_logdata() {
-        return array($this->courseid, 'user', 'view', 'view.php?id=' . $this->relateduserid . '&course=' .
-            $this->courseid, $this->relateduserid);
+        if ($this->contextlevel == CONTEXT_COURSE) {
+            return array($this->courseid, 'user', 'view', 'view.php?id=' . $this->relateduserid . '&course=' .
+                $this->courseid, $this->relateduserid);
+        }
+        return null;
     }
 }
diff --git a/lib/tests/event_test.php b/lib/tests/event_test.php
index 84b2bcdc2d4..8f68ef032f8 100644
--- a/lib/tests/event_test.php
+++ b/lib/tests/event_test.php
@@ -732,4 +732,53 @@ class core_event_testcase extends advanced_testcase {
 
         $this->assertSame($event->get_data(), $data);
     }
+
+    public function test_user_profile_viewed() {
+        $this->resetAfterTest();
+        $this->setAdminUser();
+
+        $user = $this->getDataGenerator()->create_user();
+        $course = $this->getDataGenerator()->create_course();
+        $coursecontext = context_course::instance($course->id);
+
+        // User profile viewed in course context.
+        $eventparams = array(
+            'objectid' => $user->id,
+            'relateduserid' => $user->id,
+            'courseid' => $course->id,
+            'context' => $coursecontext,
+            'other' => array(
+                'courseid' => $course->id,
+                'courseshortname' => $course->shortname,
+                'coursefullname' => $course->fullname
+            )
+        );
+        $event = \core\event\user_profile_viewed::create($eventparams);
+
+        // Trigger and capture the event.
+        $sink = $this->redirectEvents();
+        $event->trigger();
+        $events = $sink->get_events();
+        $event = reset($events);
+
+        $this->assertInstanceOf('\core\event\user_profile_viewed', $event);
+        $log = array($course->id, 'user', 'view', 'view.php?id=' . $user->id . '&course=' . $course->id, $user->id);
+        $this->assertEventLegacyLogData($log, $event);
+
+        // User profile viewed in user context.
+        $usercontext = context_user::instance($user->id);
+        $eventparams['context'] = $usercontext;
+        unset($eventparams['courseid'], $eventparams['other']);
+        $event = \core\event\user_profile_viewed::create($eventparams);
+
+        // Trigger and capture the event.
+        $sink = $this->redirectEvents();
+        $event->trigger();
+        $events = $sink->get_events();
+        $event = reset($events);
+
+        $this->assertInstanceOf('\core\event\user_profile_viewed', $event);
+        $expected = null;
+        $this->assertEventLegacyLogData($expected, $event);
+    }
 }
diff --git a/user/profile.php b/user/profile.php
index 911efda89f5..06bde7872be 100644
--- a/user/profile.php
+++ b/user/profile.php
@@ -199,6 +199,15 @@ if ($currentpage->userid == 0) {
     $CFG->blockmanagerclass = 'my_syspage_block_manager';
 }
 
+// Trigger a user profile viewed event.
+$event = \core\event\user_profile_viewed::create(array(
+    'objectid' => $user->id,
+    'relateduserid' => $user->id,
+    'context' => $usercontext
+));
+$event->add_record_snapshot('user', $user);
+$event->trigger();
+
 // TODO WORK OUT WHERE THE NAV BAR IS!
 
 echo $OUTPUT->header();