Testing... using photo info for temp thumbnails

This commit is contained in:
Brion Vibber 2010-11-08 15:50:06 -08:00
parent 551b196a35
commit a2994e3aa2
2 changed files with 17 additions and 7 deletions

View File

@ -352,11 +352,10 @@ class File extends Memcached_DataObject
$mimetype = substr($mimetype,0,$semicolon);
}
if(in_array($mimetype,$notEnclosureMimeTypes)){
// Never treat HTML as an enclosure type!
return false;
} else {
// Never treat generic HTML links as an enclosure type!
// But if we have oEmbed info, we'll consider it golden.
$oembed = File_oembed::staticGet('file_id',$this->id);
if($oembed){
if($oembed && in_array($oembed->type, array('photo', 'video'))){
$mimetype = strtolower($oembed->mimetype);
$semicolon = strpos($mimetype,';');
if($semicolon){

View File

@ -212,19 +212,30 @@ class AttachmentListItem extends Widget
}
}
/**
* Pull a thumbnail image reference for the given file.
* In order we check:
* 1) file_thumbnail table (thumbnails found via oEmbed)
* 2) image URL from direct dereference or oEmbed 'photo' type URL
* 3) ???
*
* @return mixed object with (url, width, height) properties, or false
*/
function getThumbInfo()
{
$thumbnail = File_thumbnail::staticGet('file_id', $this->attachment->id);
if ($thumbnail) {
return $thumbnail;
} else {
switch ($this->attachment->mimetype) {
}
$enc = $this->attachment->getEnclosure();
if ($enc) {
switch ($enc->mimetype) {
case 'image/gif':
case 'image/png':
case 'image/jpg':
case 'image/jpeg':
$thumb = (object)array();
$thumb->url = $this->attachment->url;
$thumb->url = $enc->url;
$thumb->width = 100;
$thumb->height = 75; // @fixme
return $thumb;