Get jpg png and gif pictures

PHP
1

I have script showing profile pictures of users from my site but it’s hardcoded to only show jpg, problem is that users can upload png and gif as well and when that happens their picture is not displayed, how can I fix this, not changing this line too much as it’s part of the bigger thing and it can’t be all reworked just for this one line?

$imgurl =$_SERVER["REQUEST_SCHEME"] . "://" . $_SERVER["HTTP_HOST"] . "/files/pictures/picture-" . ($user_id) . ".jpg";

2

I think the correct approach to this problem would be to use something like createimagefrompng to convert uploaded images to jpg. This would keep everything consistent.

However, if you can’t do that then a more hacky approach would be to check if the file exists as a jpg, then png, then gif…so that line above would be turned into something like:

$path = $_SERVER["REQUEST_SCHEME"] . "://" . $_SERVER["HTTP_HOST"] . "/files/pictures/picture-" . ($user_id);

if (file_exists($path . ".jpg")) $imgurl = $path. ".jpg";
elseif (file_exists($path . ".png")) $imgurl = $path. ".png";
elseif (file_exists($path . ".gif")) $imgurl = $path. ".gif";
else $imgurl = '404.jpg'

I don’t recommend that though because you would be creating lots of unnecessary requests which will slow down your page.

1 Like
3

Well I think we’re close here but there’s no need for the last bit as I have default pic for users, perhaps I should’ve included the whole piece of the code responsible for showing the pic.

 function get_avatar($image, $user_id, $account) 
 {
   $imgurl =$_SERVER["REQUEST_SCHEME"] . "://" . $_SERVER["HTTP_HOST"] . "/files/pictures/picture-" . ($user_id) . ".jpg";

   if (!is_imgurl_good($imgurl)) {
     $imgurl =$_SERVER["REQUEST_SCHEME"] . "://" . $_SERVER["HTTP_HOST"] . "/sites/all/themes/simple_custom/user.png";
   }
   return $imgurl;
 }
4

Another quick hacky “fix” would be to prevent them from uploading anything other than .jpg files, if the site can only display profile pictures in that format. Really, whoever added the upload code should be anticipated the situation, knowing that the output is a particular format. As labofoz said above, you could just convert them during the upload process, too.

5

Yes I agree but it’s a drupal site and this code is in totally separate chat it’s not even a module, it just gets profile pics to display in chat.

6

Asc for image MIME-type e.g. with https://www.php.net/manual/de/function.mime-content-type.php and use required creating function.

1 Like
7

I did it this way

 function get_avatar($image, $user_id, $account) 
 {
   $imgurl =$_SERVER["REQUEST_SCHEME"] . "://" . $_SERVER["HTTP_HOST"] . "/files/pictures/picture-" . ($user_id) . ".jpg";

   if (!is_imgurl_good($imgurl)) {
   $imgurl =$_SERVER["REQUEST_SCHEME"] . "://" . $_SERVER["HTTP_HOST"] . "/files/pictures/picture-" . ($user_id) . ".png";
}
   if (!is_imgurl_good($imgurl)) {
   $imgurl =$_SERVER["REQUEST_SCHEME"] . "://" . $_SERVER["HTTP_HOST"] . "/files/pictures/picture-" . ($user_id) . ".gif";
 }  
if (!is_imgurl_good($imgurl)) {
     $imgurl =$_SERVER["REQUEST_SCHEME"] . "://" . $_SERVER["HTTP_HOST"] . "/sites/all/themes/simple_custom/user.png";
   }
   return $imgurl;
 }

Trouble is when user changes his profile picture format from jpg to png the jpg is gonna be displayed as it’s the first one to check for.

8

delete the old one?

9

That would be logical but like I said it’s drupal I can’t change entire core system that’s how it works.

10

Another option would be to modify your function to retrieve the last-modified date/time of the images, and use the most recent one.

You’d have to check whether the file date/time is set when the file is uploaded, or whether the original date/time from the users computer is retained - I can’t remember how it works. As the $_FILES array doesn’t seem to contain a date/time entry, I would guess it gets a new one.

12

This topic was automatically closed 91 days after the last reply. New replies are no longer allowed.