From 548bf469cacf389e939f06eb80675c18d1352601 Mon Sep 17 00:00:00 2001 From: Michael 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 6377560962..a69ccdff85 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(); -- 2.39.5