diff --git a/lib/private/Avatar.php b/lib/private/Avatar.php index d107bb47c52..a75dfb4d60c 100644 --- a/lib/private/Avatar.php +++ b/lib/private/Avatar.php @@ -62,15 +62,15 @@ class Avatar implements IAvatar { /** * https://github.com/sebdesign/cap-height -- for 500px height - * Open Sans cap-height is 0.72 and we want a 200px caps height size (0.4 letter-to-total-height ratio, 500*0.4=200). 200/0.72 = 278px. + * Nunito cap-height is 0.72 and we want a 200px caps height size (0.4 letter-to-total-height ratio, 500*0.4=200). 200/0.72 = 278px. * Since we start from the baseline (text-anchor) we need to shift the y axis by 100px (half the caps height): 500/2+100=350 - * - * @var string + * + * @var string */ private $svgTemplate = ' - {letter} + {letter} '; /** @@ -189,7 +189,7 @@ class Avatar implements IAvatar { // } $this->user->triggerChange('avatar', $file); - } + } /** * remove the users avatar @@ -285,15 +285,15 @@ class Avatar implements IAvatar { } throw new NotFoundException; } - + /** * {size} = 500 * {fill} = hex color to fill * {letter} = Letter to display - * + * * Generate SVG avatar * @return string - * + * */ private function getAvatarVector(int $size): string { $userDisplayName = $this->user->getDisplayName(); @@ -301,14 +301,14 @@ class Avatar implements IAvatar { $bgRGB = $this->avatarBackgroundColor($userDisplayName); $bgHEX = sprintf("%02x%02x%02x", $bgRGB->r, $bgRGB->g, $bgRGB->b); $letter = mb_strtoupper(mb_substr($userDisplayName, 0, 1), 'UTF-8'); - + $toReplace = ['{size}', '{fill}', '{letter}']; return str_replace($toReplace, [$size, $bgHEX, $letter], $this->svgTemplate); } /** * Generate png avatar from svg with Imagick - * + * * @param int $size * @return string|boolean */ @@ -317,7 +317,7 @@ class Avatar implements IAvatar { return false; } try { - $font = __DIR__ . '/../../core/fonts/OpenSans-Semibold.ttf'; + $font = __DIR__ . '/../../core/fonts/Nunito-Regular.ttf'; $svg = $this->getAvatarVector($size); $avatar = new Imagick(); $avatar->setFont($font); @@ -333,7 +333,7 @@ class Avatar implements IAvatar { /** * Generate png avatar with GD - * + * * @param string $userDisplayName * @param int $size * @return string @@ -347,7 +347,7 @@ class Avatar implements IAvatar { $white = imagecolorallocate($im, 255, 255, 255); imagefilledrectangle($im, 0, 0, $size, $size, $background); - $font = __DIR__ . '/../../core/fonts/OpenSans-Semibold.ttf'; + $font = __DIR__ . '/../../core/fonts/Nunito-Regular.ttf'; $fontSize = $size * 0.4; @@ -455,7 +455,7 @@ class Avatar implements IAvatar { public function avatarBackgroundColor(string $hash) { // Normalize hash $hash = strtolower($hash); - + // Already a md5 hash? if( preg_match('/^([0-9a-f]{4}-?){8}$/', $hash, $matches) !== 1 ) { $hash = md5($hash); diff --git a/tests/lib/AvatarTest.php b/tests/lib/AvatarTest.php index 759dd385564..2f88d2b1883 100644 --- a/tests/lib/AvatarTest.php +++ b/tests/lib/AvatarTest.php @@ -48,7 +48,7 @@ class AvatarTest extends \Test\TestCase { $this->createMock(ILogger::class), $this->config ); - + // abcdefghi is a convenient name that our algorithm convert to our nextcloud blue 0082c9 $this->user->method('getDisplayName')->willReturn('abcdefghi'); } @@ -231,11 +231,11 @@ class AvatarTest extends \Test\TestCase { public function testGenerateSvgAvatar() { $avatar = $this->invokePrivate($this->avatar, 'getAvatarVector', [64]); - + $svg = ' - A + A '; $this->assertEquals($avatar, $svg); }