diff options
| author | Henry Sudhof <kellanved@phpbb.com> | 2007-04-25 13:14:11 +0000 |
|---|---|---|
| committer | Henry Sudhof <kellanved@phpbb.com> | 2007-04-25 13:14:11 +0000 |
| commit | c3c5da879ea87db67b9f9aeca4095e371ab55752 (patch) | |
| tree | f79fb59b59bb5cf49e5e75b29437bda12c61f41d /phpBB/includes/functions_convert.php | |
| parent | 8a5452d4ebaf2eb9e02612192b49515897a32c00 (diff) | |
| download | forums-c3c5da879ea87db67b9f9aeca4095e371ab55752.tar forums-c3c5da879ea87db67b9f9aeca4095e371ab55752.tar.gz forums-c3c5da879ea87db67b9f9aeca4095e371ab55752.tar.bz2 forums-c3c5da879ea87db67b9f9aeca4095e371ab55752.tar.xz forums-c3c5da879ea87db67b9f9aeca4095e371ab55752.zip | |
This should ensure that most avatars are displayed with correct dimensions. Might slow down conversions and there can be no absolute guarantee.
#10105
git-svn-id: file:///svn/phpbb/trunk@7399 89ea8834-ac86-4346-8a33-228a782c2dd0
Diffstat (limited to 'phpBB/includes/functions_convert.php')
| -rw-r--r-- | phpBB/includes/functions_convert.php | 31 |
1 files changed, 23 insertions, 8 deletions
diff --git a/phpBB/includes/functions_convert.php b/phpBB/includes/functions_convert.php index 6da8aedf66..841a6625ee 100644 --- a/phpBB/includes/functions_convert.php +++ b/phpBB/includes/functions_convert.php @@ -911,17 +911,32 @@ function get_remote_avatar_dim($src,$axis) { return $avatar_cache[$src][$axis]; } - - $avatar_cache[$src] = getimagesize($src); - + + $timeout = ini_get('default_socket_timeout'); + ini_set('default_socket_timeout', 5); + $avatar_cache[$src] = @getimagesize($src); + + $default_x = (defined('DEFAULT_AVATAR_X_CUSTOM')) ? DEFAULT_AVATAR_X_CUSTOM : DEFAULT_AVATAR_X; + $default_y = (defined('DEFAULT_AVATAR_Y_CUSTOM')) ? DEFAULT_AVATAR_Y_CUSTOM : DEFAULT_AVATAR_Y; + $default = array($default_x, $default_y); + if (empty($avatar_cache[$src]) || empty($avatar_cache[$src][0]) || empty($avatar_cache[$src][1])) { - $default_x = (defined('DEFAULT_AVATAR_X_CUSTOM')) ? DEFAULT_AVATAR_X_CUSTOM : DEFAULT_AVATAR_X; - $default_y = (defined('DEFAULT_AVATAR_Y_CUSTOM')) ? DEFAULT_AVATAR_Y_CUSTOM : DEFAULT_AVATAR_Y; - - $avatar_cache[$src] = array($default_x, $default_y); + $avatar_cache[$src] = $default; } - + else + { + // We trust gallery and uploaded avatars to conform to the size settings; we might have to adjust here + if ($avatar_cache[$src][0] > $default_x || $avatar_cache[$src][1] > $default_y) + { + $bigger = ($avatar_cache[$src][0] > $avatar_cache[$src][1]) ? 0 : 1; + $ratio = $default[$bigger] / $avatar_cache[$src][$bigger]; + $avatar_cache[$src][0] = (int)($avatar_cache[$src][0] * $ratio); + $avatar_cache[$src][1] = (int)($avatar_cache[$src][1] * $ratio); + } + } + + ini_set('default_socket_timeout', $timeout); return $avatar_cache[$src][$axis]; } |