From 402c0fc77020499377109518c79810cbd921dcf4 Mon Sep 17 00:00:00 2001
From: Michael <heluecht@pirati.ca>
Date: Sun, 25 Mar 2018 17:26:40 +0000
Subject: [PATCH] Using the default avatar when no avatar was found for the
 self contact

---
 src/Model/Contact.php | 46 ++++++++++++++++++++++++-------------------
 1 file changed, 26 insertions(+), 20 deletions(-)

diff --git a/src/Model/Contact.php b/src/Model/Contact.php
index d184dacdbf..60dec8b289 100644
--- a/src/Model/Contact.php
+++ b/src/Model/Contact.php
@@ -167,36 +167,42 @@ class Contact extends BaseObject
 			return;
 		}
 
-		$avatar = dba::selectFirst('photo', ['resource-id', 'type'], ['uid' => $uid, 'profile' => true]);
-
 		$fields = ['name' => $profile['name'], 'nick' => $user['nickname'],
 			'avatar-date' => $self['avatar-date'], 'location' => Profile::formatLocation($profile),
 			'about' => $profile['about'], 'keywords' => $profile['pub_keywords'],
 			'gender' => $profile['gender'], 'avatar' => $profile['photo'],
 			'contact-type' => $user['account-type'], 'xmpp' => $profile['xmpp']];
 
-		if ($update_avatar) {
-			$fields['avatar-date'] = DateTimeFormat::utcNow();
-		}
+		$avatar = dba::selectFirst('photo', ['resource-id', 'type'], ['uid' => $uid, 'profile' => true]);
+		if (DBM::is_result($avatar)) {
+			if ($update_avatar) {
+				$fields['avatar-date'] = DateTimeFormat::utcNow();
+			}
 
-		// Creating the path to the avatar, beginning with the file suffix
-		$types = Image::supportedTypes();
-		if (isset($types[$avatar['type']])) {
-			$file_suffix = $types[$avatar['type']];
+			// Creating the path to the avatar, beginning with the file suffix
+			$types = Image::supportedTypes();
+			if (isset($types[$avatar['type']])) {
+				$file_suffix = $types[$avatar['type']];
+			} else {
+				$file_suffix = 'jpg';
+			}
+
+			// We are adding a timestamp value so that other systems won't use cached content
+			$timestamp = strtotime($fields['avatar-date']);
+
+			$prefix = System::baseUrl() . '/photo/' .$avatar['resource-id'] . '-';
+			$suffix = '.' . $file_suffix . '?ts=' . $timestamp;
+
+			$fields['photo'] = $prefix . '4' . $suffix;
+			$fields['thumb'] = $prefix . '5' . $suffix;
+			$fields['micro'] = $prefix . '6' . $suffix;
 		} else {
-			$file_suffix = 'jpg';
+			// We hadn't found a photo entry, so we use the default avatar
+			$fields['photo'] = System::baseUrl() . '/images/person-175.jpg';
+			$fields['thumb'] = System::baseUrl() . '/images/person-80.jpg';
+			$fields['micro'] = System::baseUrl() . '/images/person-48.jpg';
 		}
 
-		// We are adding a timestamp value so that other systems won't use cached content
-		$timestamp = strtotime($fields['avatar-date']);
-
-		$prefix = System::baseUrl() . '/photo/' .$avatar['resource-id'] . '-';
-		$suffix = '.' . $file_suffix . '?ts=' . $timestamp;
-
-		$fields['photo'] = $prefix . '4' . $suffix;
-		$fields['thumb'] = $prefix . '5' . $suffix;
-		$fields['micro'] = $prefix . '6' . $suffix;
-
 		$fields['forum'] = $user['page-flags'] == PAGE_COMMUNITY;
 		$fields['prv'] = $user['page-flags'] == PAGE_PRVGROUP;