keith/friendica
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity

Merge pull request #3819 from annando/photo-rework

Browse source 
Better handling for GIF even, if not supported by the graphics library
This commit is contained in:
Hypolite Petovan 2017-10-23 13:04:52 -04:00 committed by GitHub
commit 0c347d1466
2 changed files with 37 additions and 32 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
include/Photo.php
View file

@ -26,7 +26,7 @@ class Photo {
/** /**
* @brief supported mimetypes and corresponding file extensions * @brief supported mimetypes and corresponding file extensions
*/ */
static function supportedTypes() { public static function supportedTypes() {
if (class_exists('Imagick')) { if (class_exists('Imagick')) {
// Imagick::queryFormats won't help us a lot there... // Imagick::queryFormats won't help us a lot there...
@ -49,7 +49,7 @@ class Photo {
public function __construct($data, $type=null) { public function __construct($data, $type=null) {
$this->imagick = class_exists('Imagick'); $this->imagick = class_exists('Imagick');
$this->types = $this->supportedTypes(); $this->types = static::supportedTypes();
if (!array_key_exists($type, $this->types)){ if (!array_key_exists($type, $this->types)){
$type='image/jpeg'; $type='image/jpeg';
} }

37
mod/photo.php
View file

@ -31,7 +31,6 @@ 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");
@ -50,7 +49,6 @@ function photo_init(App $a) {
if (isset($type)) { if (isset($type)) {
/** /**
* Profile photos * Profile photos
*/ */
@ -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,20 +84,21 @@ 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) == '-') {
@ -136,7 +139,7 @@ function photo_init(App $a) {
} }
} }
if(! isset($data)) { if (empty($data)) {
if (isset($resolution)) { if (isset($resolution)) {
switch ($resolution) { switch ($resolution) {
@ -160,8 +163,8 @@ function photo_init(App $a) {
} }
} }
// Resize only if its not a GIF // Resize only if its not a GIF and it is supported by the library
if ($mime != "image/gif") { if (($mimetype != "image/gif") && in_array($mimetype, Photo::supportedTypes())) {
$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) {
@ -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");
@ -200,13 +202,16 @@ function photo_init(App $a) {
if ($public and ($file != "")) { if ($public and ($file != "")) {
// If the photo path isn't there, try to create it // If the photo path isn't there, try to create it
$basepath = $a->get_basepath(); $basepath = $a->get_basepath();
if (!is_dir($basepath."/photo")) if (!is_dir($basepath."/photo")) {
if (is_writable($basepath)) if (is_writable($basepath)) {
mkdir($basepath."/photo"); mkdir($basepath."/photo");
}
}
if (is_dir($basepath."/photo")) if (is_dir($basepath."/photo")) {
file_put_contents($basepath."/photo/".$file, $data); file_put_contents($basepath."/photo/".$file, $data);
} }
}
killme(); killme();
// NOTREACHED // NOTREACHED