X-Git-Url: https://git.mxchange.org/?a=blobdiff_plain;f=mod%2Fphoto.php;h=653802d30e8bd27ffe4b3f7eec21c371323ab446;hb=5c63de1692c5be4ca457e03fc037683909a44348;hp=6562599ca3cf0ccc5b19e997b2734fcbdd408e0d;hpb=7b352f3f74b045690cdeae507b609f7a9cc8db03;p=friendica.git

diff --git a/mod/photo.php b/mod/photo.php
index 6562599ca3..653802d30e 100644
--- a/mod/photo.php
+++ b/mod/photo.php
@@ -1,15 +1,22 @@
 <?php
 
-require_once('include/security.php');
-require_once('include/Photo.php');
+/**
+ * @file mod/photo.php
+ */
+use Friendica\App;
+use Friendica\Database\DBM;
+use Friendica\Object\Image;
 
-function photo_init(App $a) {
+require_once 'include/security.php';
+
+function photo_init(App $a)
+{
 	global $_SERVER;
 
 	$prvcachecontrol = false;
 	$file = "";
 
-	switch($a->argc) {
+	switch ($a->argc) {
 		case 4:
 			$person = $a->argv[3];
 			$customres = intval($a->argv[2]);
@@ -29,14 +36,15 @@ 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");
+		if (!empty($_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("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');
@@ -45,16 +53,11 @@ function photo_init(App $a) {
 	}
 
 	$default = 'images/person-175.jpg';
+	$public = true;
 
-	if(isset($type)) {
-
-
-		/**
-		 * Profile photos
-		 */
-
-		switch($type) {
-
+	if (isset($type)) {
+		// Profile photos
+		switch ($type) {
 			case 'profile':
 			case 'custom':
 				$resolution = 4;
@@ -70,59 +73,56 @@ function photo_init(App $a) {
 				break;
 		}
 
-		$uid = str_replace(array('.jpg','.png'),array('',''), $person);
+		$uid = str_replace(['.jpg', '.png', '.gif'], ['', '', ''], $person);
 
-		$r = qu("SELECT * FROM `photo` WHERE `scale` = %d AND `uid` = %d AND `profile` = 1 LIMIT 1",
+		foreach (Image::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),
 			intval($uid)
 		);
-		if (dbm::is_result($r)) {
+		if (DBM::is_result($r)) {
 			$data = $r[0]['data'];
 			$mimetype = $r[0]['type'];
 		}
-		if(! isset($data)) {
+		if (empty($data)) {
 			$data = file_get_contents($default);
 			$mimetype = 'image/jpeg';
 		}
-	}
-	else {
-
-		/**
-		 * Other photos
-		 */
-
+	} else {
+		// Other photos
 		$resolution = 0;
-		foreach( Photo::supportedTypes() as $m=>$e){
-			$photo = str_replace(".$e",'',$photo);
+		$photo = str_replace(['.jpg', '.png', '.gif'], ['', '', ''], $photo);
+
+		foreach (Image::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
-		$r = qu("SELECT `uid` FROM `photo` WHERE `resource-id` = '%s' LIMIT 1",
+		// 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)
 		);
-		if (dbm::is_result($r)) {
-
+		if (DBM::is_result($r)) {
 			$sql_extra = permissions_sql($r[0]['uid']);
 
 			// Now we'll see if we can access the photo
-
-			$r = qu("SELECT * FROM `photo` WHERE `resource-id` = '%s' AND `scale` <= %d $sql_extra ORDER BY scale DESC LIMIT 1",
+			$r = q("SELECT * FROM `photo` WHERE `resource-id` = '%s' AND `scale` <= %d $sql_extra ORDER BY scale DESC LIMIT 1",
 				dbesc($photo),
 				intval($resolution)
 			);
-
-			$public = (dbm::is_result($r)) AND ($r[0]['allow_cid'] == '') AND ($r[0]['allow_gid'] == '') AND ($r[0]['deny_cid']  == '') AND ($r[0]['deny_gid']  == '');
-
-			if (dbm::is_result($r)) {
+			if (DBM::is_result($r)) {
 				$resolution = $r[0]['scale'];
 				$data = $r[0]['data'];
 				$mimetype = $r[0]['type'];
+				$public = $r[0]['allow_cid'] == '' && $r[0]['allow_gid'] == '' && $r[0]['deny_cid'] == '' && $r[0]['deny_gid'] == '';
 			} else {
 				// The picure exists. We already checked with the first query.
 				// obviously, this is not an authorized viev!
@@ -134,10 +134,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');
 					$mimetype = 'image/jpeg';
@@ -158,52 +157,51 @@ function photo_init(App $a) {
 		}
 	}
 
-	// Resize only if its not a GIF
-	if ($mime != "image/gif") {
-		$ph = new Photo($data, $mimetype);
-		if($ph->is_valid()) {
-			if(isset($customres) && $customres > 0 && $customres < 500) {
-				$ph->scaleImageSquare($customres);
+	// Resize only if its not a GIF and it is supported by the library
+	if ($mimetype != "image/gif" && in_array($mimetype, Image::supportedTypes())) {
+		$Image = new Image($data, $mimetype);
+		if ($Image->isValid()) {
+			if (isset($customres) && $customres > 0 && $customres < 500) {
+				$Image->scaleToSquare($customres);
 			}
-			$data = $ph->imageString();
-			$mimetype = $ph->getType();
+			$data = $Image->asString();
+			$mimetype = $Image->getType();
 		}
 	}
 
-	if(function_exists('header_remove')) {
+	if (function_exists('header_remove')) {
 		header_remove('Pragma');
 		header_remove('pragma');
 	}
 
-	header("Content-type: ".$mimetype);
-
-	if($prvcachecontrol) {
+	header("Content-type: " . $mimetype);
 
+	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
 		// and subsequently have permission to see it
-
 		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");
+		header('Etag: "' . md5($data) . '"');
+		header("Expires: " . gmdate("D, d M Y H:i:s", time() + (31536000)) . " GMT");
 		header("Cache-Control: max-age=31536000");
 	}
 	echo $data;
 
 	// If the photo is public and there is an existing photo directory store the photo there
-	if ($public and ($file != "")) {
+	if ($public and $file != '') {
 		// If the photo path isn't there, try to create it
 		$basepath = $a->get_basepath();
-		if (!is_dir($basepath."/photo"))
-			if (is_writable($basepath))
-				mkdir($basepath."/photo");
+		if (!is_dir($basepath . "/photo")) {
+			if (is_writable($basepath)) {
+				mkdir($basepath . "/photo");
+			}
+		}
 
-		if (is_dir($basepath."/photo"))
-			file_put_contents($basepath."/photo/".$file, $data);
+		if (is_dir($basepath . "/photo")) {
+			file_put_contents($basepath . "/photo/" . $file, $data);
+		}
 	}
 
 	killme();