diff --git a/mod/photo.php b/mod/photo.php index 5896c90ba..1a814f5ef 100644 --- a/mod/photo.php +++ b/mod/photo.php @@ -11,7 +11,7 @@ function photo_init(App $a) { $prvcachecontrol = false; $file = ""; - switch($a->argc) { + switch ($a->argc) { case 4: $person = $a->argv[3]; $customres = intval($a->argv[2]); @@ -31,14 +31,13 @@ function photo_init(App $a) { // NOTREACHED } - // strtotime($_SERVER['HTTP_IF_MODIFIED_SINCE']) >= filemtime($localFileName)) { if (isset($_SERVER['HTTP_IF_MODIFIED_SINCE'])) { header('HTTP/1.1 304 Not Modified'); header("Last-Modified: " . gmdate("D, d M Y H:i:s", time()) . " GMT"); header('Etag: '.$_SERVER['HTTP_IF_NONE_MATCH']); header("Expires: " . gmdate("D, d M Y H:i:s", time() + (31536000)) . " GMT"); header("Cache-Control: max-age=31536000"); - if(function_exists('header_remove')) { + if (function_exists('header_remove')) { header_remove('Last-Modified'); header_remove('Expires'); header_remove('Cache-Control'); @@ -48,14 +47,13 @@ function photo_init(App $a) { $default = 'images/person-175.jpg'; - if(isset($type)) { - + if (isset($type)) { /** * Profile photos */ - switch($type) { + switch ($type) { case 'profile': case 'custom': @@ -72,7 +70,11 @@ function photo_init(App $a) { 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", intval($resolution), @@ -82,28 +84,29 @@ function photo_init(App $a) { $data = $r[0]['data']; $mimetype = $r[0]['type']; } - if(! isset($data)) { + if (empty($data)) { $data = file_get_contents($default); $mimetype = 'image/jpeg'; } - } - else { + } else { /** * Other photos */ $resolution = 0; - foreach( Photo::supportedTypes() as $m=>$e){ - $photo = str_replace(".$e",'',$photo); + $photo = str_replace(array('.jpg', '.png', '.gif'), array('', '', ''), $photo); + + foreach (Photo::supportedTypes() AS $m => $e) { + $photo = str_replace('.'.$e, '', $photo); } - if(substr($photo,-2,1) == '-') { - $resolution = intval(substr($photo,-1,1)); - $photo = substr($photo,0,-2); + if (substr($photo, -2, 1) == '-') { + $resolution = intval(substr($photo, -1, 1)); + $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 $r = q("SELECT `uid` FROM `photo` WHERE `resource-id` = '%s' LIMIT 1", dbesc($photo), intval($resolution) @@ -136,9 +139,9 @@ function photo_init(App $a) { } } - if(! isset($data)) { - if(isset($resolution)) { - switch($resolution) { + if (empty($data)) { + if (isset($resolution)) { + switch ($resolution) { case 4: $data = file_get_contents('images/person-175.jpg'); @@ -161,10 +164,10 @@ function photo_init(App $a) { } // Resize only if its not a GIF - if ($mime != "image/gif") { + if ($mimetype != "image/gif") { $ph = new Photo($data, $mimetype); - if($ph->is_valid()) { - if(isset($customres) && $customres > 0 && $customres < 500) { + if ($ph->is_valid()) { + if (isset($customres) && $customres > 0 && $customres < 500) { $ph->scaleImageSquare($customres); } $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("Content-type: ".$mimetype); - if($prvcachecontrol) { + if ($prvcachecontrol) { // it is a private photo that they have no permission to view. // 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"); - } - else { + } else { header("Last-Modified: " . gmdate("D, d M Y H:i:s", time()) . " GMT"); header('Etag: "'.md5($data).'"'); header("Expires: " . gmdate("D, d M Y H:i:s", time() + (31536000)) . " GMT");