Hello,

Last night, I opened eglug.org on an internet cyber with an IE browser. And I was shocked with how the PNGs look shitty on it.

This is a solution I came up with today. It will give the browser GIFs when it's IE otherwise it will give it PNGs.

Place this script in your drupal root. Then all you need to do is, replacing your img paths with something like "image.php?name=logo"

so a previous img tag was like

<img src="/themes/eglug/logo.png" ...more attribs>

<img src="/image.php?name=logo" ..more attribs>

For example,
To add this functionality to your drupal-powered logo image you need to:
1. Go to Administer -> Themes -> Configure
2. In "Path to custom logo:" field, put "image.php?name=logo" where logo is your logo image name of course.
3. Scroll down and hit Save Configuration.

That solution is not that flexible and doesn't integrate into drupal nicely. I was hopping to write a better solution that will fit into drupal like a filter or a module, but from a quick look into drupal, I realized this is not possible.

So everyone is very welcomed to write a better implementation of it.

I've got some more other stuff to put into it in order to make it more flexible. However, I needed to know if the idea itself is accepted and can be used or not so I can put further development time on it.

- Amr

Attached "image.php":

/**
 * Copyright (c) Amr Mostafa
 * This stuff is GPL v2 licensed, so feel free to use it in anyway.
 */

/* change this to wherever your images are */
$prefix = "themes/eglug";

if (!isset($_GET['name'])) {
  exit;
}

$name = $_GET['name'];

/* should this be a regexp ? strpos is lot faster anyway */

if (isset($_SERVER['HTTP_USER_AGENT']) && strpos($_SERVER['HTTP_USER_AGENT'], "MSIE")) {
  $image_ext = "gif";
  $content_type = "image/gif";
}
else {
  $image_ext = "png";
  $content_type = "image/png";
}

$prefix = rtrim($prefix, "/");
$image_path = sprintf("%s/%s.%s", $prefix, $name, $image_ext);

/**
 * The following block of code will make sure that even if a .gif or a .png file doesn't exist
 * the other file will be used.
 * So you can safely remove those next 16 lines of code if you don't need this functionality :-)
 */
if (!file_exists($image_path)) {
  if ($image_ext == "gif") {
    $image_ext = "png";
    $image_path = sprintf("%s/%s.%s", $prefix, $name, $image_ext);
    if (!file_exists($image_path)) {
      exit;
    }
  }
  else {
    $image_ext = "gif";
    $image_path = sprintf("%s/%s.%s", $prefix, $name, $image_ext);
    if (!file_exists($image_path)) {
      exit;
    }
  }
}

//$gmt_date = gmdate("D, d M Y H:i:s") . " GMT";
$size = filesize($image_path);

$fp = fopen($image_path, "rb");
if (!$fp) {
  exit;
}

//header("Date: " . $gmt_date);
header("Content-Type: " . $content_type);
header("Content-Length: " . $size);
fpassthru($fp);
exit;