Better handling for GIF even, if not supported by the graphics library

This commit is contained in:
Michael 2017-10-21 15:09:47 +00:00
parent 6367439e8a
commit 09727024ba

View file

@ -11,7 +11,7 @@ function photo_init(App $a) {
$prvcachecontrol = false; $prvcachecontrol = false;
$file = ""; $file = "";
switch($a->argc) { switch ($a->argc) {
case 4: case 4:
$person = $a->argv[3]; $person = $a->argv[3];
$customres = intval($a->argv[2]); $customres = intval($a->argv[2]);
@ -31,14 +31,13 @@ function photo_init(App $a) {
// NOTREACHED // NOTREACHED
} }
// strtotime($_SERVER['HTTP_IF_MODIFIED_SINCE']) >= filemtime($localFileName)) {
if (isset($_SERVER['HTTP_IF_MODIFIED_SINCE'])) { if (isset($_SERVER['HTTP_IF_MODIFIED_SINCE'])) {
header('HTTP/1.1 304 Not Modified'); header('HTTP/1.1 304 Not Modified');
header("Last-Modified: " . gmdate("D, d M Y H:i:s", time()) . " GMT"); header("Last-Modified: " . gmdate("D, d M Y H:i:s", time()) . " GMT");
header('Etag: '.$_SERVER['HTTP_IF_NONE_MATCH']); header('Etag: '.$_SERVER['HTTP_IF_NONE_MATCH']);
header("Expires: " . gmdate("D, d M Y H:i:s", time() + (31536000)) . " GMT"); header("Expires: " . gmdate("D, d M Y H:i:s", time() + (31536000)) . " GMT");
header("Cache-Control: max-age=31536000"); header("Cache-Control: max-age=31536000");
if(function_exists('header_remove')) { if (function_exists('header_remove')) {
header_remove('Last-Modified'); header_remove('Last-Modified');
header_remove('Expires'); header_remove('Expires');
header_remove('Cache-Control'); header_remove('Cache-Control');
@ -48,14 +47,13 @@ function photo_init(App $a) {
$default = 'images/person-175.jpg'; $default = 'images/person-175.jpg';
if(isset($type)) { if (isset($type)) {
/** /**
* Profile photos * Profile photos
*/ */
switch($type) { switch ($type) {
case 'profile': case 'profile':
case 'custom': case 'custom':
@ -72,7 +70,11 @@ function photo_init(App $a) {
break; break;
} }
$uid = str_replace(array('.jpg','.png'),array('',''), $person); $uid = str_replace(array('.jpg', '.png', '.gif'), array('', '', ''), $person);
foreach (Photo::supportedTypes() AS $m => $e) {
$uid = str_replace('.'.$e, '', $uid);
}
$r = q("SELECT * FROM `photo` WHERE `scale` = %d AND `uid` = %d AND `profile` = 1 LIMIT 1", $r = q("SELECT * FROM `photo` WHERE `scale` = %d AND `uid` = %d AND `profile` = 1 LIMIT 1",
intval($resolution), intval($resolution),
@ -82,25 +84,26 @@ function photo_init(App $a) {
$data = $r[0]['data']; $data = $r[0]['data'];
$mimetype = $r[0]['type']; $mimetype = $r[0]['type'];
} }
if(! isset($data)) { if (empty($data)) {
$data = file_get_contents($default); $data = file_get_contents($default);
$mimetype = 'image/jpeg'; $mimetype = 'image/jpeg';
} }
} } else {
else {
/** /**
* Other photos * Other photos
*/ */
$resolution = 0; $resolution = 0;
foreach( Photo::supportedTypes() as $m=>$e){ $photo = str_replace(array('.jpg', '.png', '.gif'), array('', '', ''), $photo);
$photo = str_replace(".$e",'',$photo);
foreach (Photo::supportedTypes() AS $m => $e) {
$photo = str_replace('.'.$e, '', $photo);
} }
if(substr($photo,-2,1) == '-') { if (substr($photo, -2, 1) == '-') {
$resolution = intval(substr($photo,-1,1)); $resolution = intval(substr($photo, -1, 1));
$photo = substr($photo,0,-2); $photo = substr($photo, 0, -2);
} }
// check if the photo exists and get the owner of the photo // check if the photo exists and get the owner of the photo
@ -136,9 +139,9 @@ function photo_init(App $a) {
} }
} }
if(! isset($data)) { if (empty($data)) {
if(isset($resolution)) { if (isset($resolution)) {
switch($resolution) { switch ($resolution) {
case 4: case 4:
$data = file_get_contents('images/person-175.jpg'); $data = file_get_contents('images/person-175.jpg');
@ -161,10 +164,10 @@ function photo_init(App $a) {
} }
// Resize only if its not a GIF // Resize only if its not a GIF
if ($mime != "image/gif") { if ($mimetype != "image/gif") {
$ph = new Photo($data, $mimetype); $ph = new Photo($data, $mimetype);
if($ph->is_valid()) { if ($ph->is_valid()) {
if(isset($customres) && $customres > 0 && $customres < 500) { if (isset($customres) && $customres > 0 && $customres < 500) {
$ph->scaleImageSquare($customres); $ph->scaleImageSquare($customres);
} }
$data = $ph->imageString(); $data = $ph->imageString();
@ -172,14 +175,14 @@ function photo_init(App $a) {
} }
} }
if(function_exists('header_remove')) { if (function_exists('header_remove')) {
header_remove('Pragma'); header_remove('Pragma');
header_remove('pragma'); header_remove('pragma');
} }
header("Content-type: ".$mimetype); header("Content-type: ".$mimetype);
if($prvcachecontrol) { if ($prvcachecontrol) {
// it is a private photo that they have no permission to view. // it is a private photo that they have no permission to view.
// tell the browser not to cache it, in case they authenticate // tell the browser not to cache it, in case they authenticate
@ -187,8 +190,7 @@ function photo_init(App $a) {
header("Cache-Control: no-store, no-cache, must-revalidate"); header("Cache-Control: no-store, no-cache, must-revalidate");
} } else {
else {
header("Last-Modified: " . gmdate("D, d M Y H:i:s", time()) . " GMT"); header("Last-Modified: " . gmdate("D, d M Y H:i:s", time()) . " GMT");
header('Etag: "'.md5($data).'"'); header('Etag: "'.md5($data).'"');
header("Expires: " . gmdate("D, d M Y H:i:s", time() + (31536000)) . " GMT"); header("Expires: " . gmdate("D, d M Y H:i:s", time() + (31536000)) . " GMT");