From ff9142fc5e001d878ad797498206e3469a04548d Mon Sep 17 00:00:00 2001
From: Robert Lyon <robertl@catalyst.net.nz>
Date: Thu, 21 Nov 2013 11:51:37 +1300
Subject: [PATCH] Checking for $user->urlid before trying to access it (bug
 #1253462)

In some cases the $user->urlid may not be present - this checks for it
before accessing it otherwise it falls back to the $id to get info

Change-Id: I989e68321402f458b167879e994150213c3e4893
Signed-off-by: Robert Lyon <robertl@catalyst.net.nz>
---
 htdocs/lib/user.php | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/htdocs/lib/user.php b/htdocs/lib/user.php
index 8120b7d333..db1af82ad1 100644
--- a/htdocs/lib/user.php
+++ b/htdocs/lib/user.php
@@ -1609,7 +1609,8 @@ function profile_url($user, $full=true, $useid=false) {
     }
     else if (is_numeric($user)) {
         $id = $user;
-        $urlid = $wantclean ? get_user_for_display($id)->urlid : null;
+        $user = get_user_for_display($id);
+        $urlid = ($wantclean && !empty($user->urlid)) ? $user->urlid : null;
     }
     else if (isset($user->id)) {
         $id = $user->id;
-- 
GitLab