From 548bf469cacf389e939f06eb80675c18d1352601 Mon Sep 17 00:00:00 2001
From: Michael <heluecht@pirati.ca>
Date: Mon, 9 May 2022 22:36:25 +0000
Subject: [PATCH] Added logging

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

diff --git a/src/Contact/Avatar.php b/src/Contact/Avatar.php
index 637756096..a69ccdff8 100644
--- a/src/Contact/Avatar.php
+++ b/src/Contact/Avatar.php
@@ -124,19 +124,33 @@ class Avatar
 		foreach (explode('/', dirname($filename)) as $part) {
 			$dirpath .= $part . '/';
 			if (!file_exists($dirpath)) {
-				mkdir($dirpath, $dir_perm);
+				if (!mkdir($dirpath, $dir_perm)) {
+					Logger::warning('Directory could not be created', ['directory' => $dirpath]);
+				}
 			} elseif (fileperms($dirpath) & 0777 != $dir_perm) {
-				chmod($dirpath, $dir_perm);
+				if (!chmod($dirpath, $dir_perm)) {
+					Logger::warning('Directory permissions could not be changed', ['directory' => $dirpath]);
+				}
 			}
 
 			if (filegroup($dirpath) != $group) {
-				chgrp($dirpath, $group);
+				if (!chgrp($dirpath, $group)) {
+					Logger::warning('Directory group could not be changed', ['directory' => $dirpath]);
+				}
 			}
 		}
 
-		file_put_contents($filepath, $image->asString());
-		chmod($filepath, $file_perm);
-		chgrp($filepath, $group);
+		if (!file_put_contents($filepath, $image->asString())) {
+			Logger::warning('File could not be created', ['file' => $filepath]);
+		}
+
+		if (!chmod($filepath, $file_perm)) {
+			Logger::warning('File permissions could not be changed', ['file' => $filepath]);
+		}
+
+		if (!chgrp($filepath, $group)) {
+			Logger::warning('File group could not be changed', ['file' => $filepath]);
+		}
 
 		DI::profiler()->stopRecording();