X-Git-Url: https://git.mxchange.org/?a=blobdiff_plain;f=src%2FModule%2FPhoto.php;h=0c2baeac4019155cbfdc969bcdb1f118b42ed2e0;hb=6cbfa5b862ce7a241fd4339129749f87dc19760c;hp=9165af8bedbbaad213c98833563887df25849d64;hpb=51210af22157327376954721bfdca7161b66a7be;p=friendica.git diff --git a/src/Module/Photo.php b/src/Module/Photo.php index 9165af8bed..0c2baeac40 100644 --- a/src/Module/Photo.php +++ b/src/Module/Photo.php @@ -6,8 +6,10 @@ namespace Friendica\Module; use Friendica\BaseModule; +use Friendica\Core\L10n; use Friendica\Core\Logger; use Friendica\Core\System; +use Friendica\DI; use Friendica\Model\Photo as MPhoto; use Friendica\Object\Image; @@ -22,12 +24,12 @@ class Photo extends BaseModule * Fetch a photo or an avatar, in optional size, check for permissions and * return the image */ - public static function init() + public static function init(array $parameters = []) { - $a = self::getApp(); + $a = DI::app(); // @TODO: Replace with parameter from router if ($a->argc <= 1 || $a->argc > 4) { - System::httpExit(400); + throw new \Friendica\Network\HTTPException\BadRequestException(); } if (isset($_SERVER["HTTP_IF_MODIFIED_SINCE"])) { @@ -52,15 +54,15 @@ class Photo extends BaseModule switch($a->argc) { case 4: $customsize = intval($a->argv[2]); - $uid = self::stripExtension($a->argv[3]); + $uid = MPhoto::stripExtension($a->argv[3]); $photo = self::getAvatar($uid, $a->argv[1]); break; case 3: - $uid = self::stripExtension($a->argv[2]); + $uid = MPhoto::stripExtension($a->argv[2]); $photo = self::getAvatar($uid, $a->argv[1]); break; case 2: - $photoid = self::stripExtension($a->argv[1]); + $photoid = MPhoto::stripExtension($a->argv[1]); $scale = 0; if (substr($photoid, -2, 1) == "-") { $scale = intval(substr($photoid, -1, 1)); @@ -74,9 +76,7 @@ class Photo extends BaseModule } if ($photo === false) { - // not using System::httpExit() because we don't want html here. - header($_SERVER["SERVER_PROTOCOL"] . " 404 Not Found" , true, 404); - exit(); + System::httpExit('404', 'Not Found'); } $cacheable = ($photo["allow_cid"] . $photo["allow_gid"] . $photo["deny_cid"] . $photo["deny_gid"] === "") && (isset($photo["cacheable"]) ? $photo["cacheable"] : true); @@ -85,7 +85,7 @@ class Photo extends BaseModule if (is_null($img) || !$img->isValid()) { Logger::log("Invalid photo with id {$photo["id"]}."); - System::httpExit(500, ["description" => "Invalid photo with id {$photo["id"]}."]); + throw new \Friendica\Network\HTTPException\InternalServerErrorException(L10n::t('Invalid photo with id %s.', $photo["id"])); } // if customsize is set and image is not a gif, resize it @@ -118,15 +118,6 @@ class Photo extends BaseModule exit(); } - private static function stripExtension($name) - { - $name = str_replace([".jpg", ".png", ".gif"], ["", "", ""], $name); - foreach (Image::supportedTypes() as $m => $e) { - $name = str_replace("." . $e, "", $name); - } - return $name; - } - private static function getAvatar($uid, $type="avatar") {