Imagefield compatibility with editablefields

Hey there,

I was looking for a solution to the lists that appear in places of images for imagefields and came across a http://anexusit.com/blog/drupal-how-enable-filefield-image-print-images-print-module-htmlpdf">post with a template.php hook that you can use to make the filefield list be an actual image when the filefield is an image versus an actual file such as txt, doc, etc. If this seems like a desirable feature I could go ahead and roll up a module patch for it, may end up doing that and posting it in a follow up. I personally think it might be a nice touch for the editablefields module but others might not feel the same way.

<?php
/**
 * Theme function for the 'generic' single file formatter.
 */
function mytheme_filefield_file($file) {
  // Views may call this function with a NULL value, return an empty string.
  if (empty($file['fid'])) {
    return '';
  }

  $path = $file['filepath'];
  $url = file_create_url($path);

  $options = array(
    'attributes' => array(
      'type' => $file['filemime'] . '; length=' . $file['filesize'],
    ),
  );

  // Use the description as the link text if available.
  if (empty($file['data']['description'])) {
    $link_text = $file['filename'];
  }
  else {
    $link_text = $file['data']['description'];
    $options['attributes']['title'] = $file['filename'];
  }
  
  
  if(strstr($file['filemime'],'image')) {
    // change the_preset to your imagecache preset's machine readable name
    $image = theme('imagecache','the_preset',$path); 
    return '
' . $image .'
';
  } else { 
    $icon = theme('filefield_icon', $file);
    return '
'. $icon . l($link_text, $url, $options) .'
';
  }  
}
?>

This goes in your theme's template.php -- Don't forget to change out "mytheme" in the function name for your actual theme name and "the_preset" with the machine readable name of the imagecache preset that you would like to appear by default. Perhaps this can be improved by hooking what is set for Display Fields for an imagefield.