move html for oembeded videos in template. fix oembed discovery
This commit is contained in:
parent
718a9009be
commit
75d4f33233
2 changed files with 26 additions and 16 deletions
|
@ -16,8 +16,7 @@ function oembed_fetch_url($embedurl){
|
|||
// try oembed autodiscovery
|
||||
$redirects = 0;
|
||||
$html_text = fetch_url($embedurl, false, $redirects, 15);
|
||||
if(! $html_text)
|
||||
return;
|
||||
if($html_text){
|
||||
$dom = @DOMDocument::loadHTML($html_text);
|
||||
if ($dom){
|
||||
$xpath = new DOMXPath($dom);
|
||||
|
@ -30,6 +29,7 @@ function oembed_fetch_url($embedurl){
|
|||
$txt = fetch_url($href);
|
||||
}
|
||||
}
|
||||
}
|
||||
|
||||
if ($txt==false || $txt==""){
|
||||
// try oohembed service
|
||||
|
@ -59,9 +59,15 @@ function oembed_format_object($j){
|
|||
/*$tw = (isset($j->thumbnail_width)) ? $j->thumbnail_width:200;
|
||||
$th = (isset($j->thumbnail_height)) ? $j->thumbnail_height:180;*/
|
||||
$tw=150; $th=120;
|
||||
$ret.= "<a href='".$embedurl."' onclick='this.innerHTML=unescape(\"".urlencode($j->html)."\").replace(/\+/g,\" \"); return false;' style='float:left; margin: 1em; '>";
|
||||
$ret.= "<img width='$tw' height='$th' src='".$j->thumbnail_url."'>";
|
||||
$ret.= "</a>";
|
||||
$tpl=get_markup_template('oembed_video.tpl');
|
||||
$ret.=replace_macros($tpl, array(
|
||||
'$embedurl'=>$embedurl,
|
||||
'$escapedhtml'=>urlencode($j->html),
|
||||
'$tw'=>$tw,
|
||||
'$th'=>$th,
|
||||
'$turl'=>$j->thumbnail_url,
|
||||
));
|
||||
|
||||
} else {
|
||||
$ret=$j->html;
|
||||
}
|
||||
|
|
4
view/oembed_video.tpl
Normal file
4
view/oembed_video.tpl
Normal file
|
@ -0,0 +1,4 @@
|
|||
<a href='$embedurl' onclick='this.innerHTML=unescape("$escapedhtml").replace(/\+/g," "); return false;' style='float:left; margin: 1em; position: relative;'>
|
||||
<img width='$tw' height='$th' src='$turl' >
|
||||
<div style='position: absolute; top: 0px; left: 0px; width: $twpx; height: $thpx; background: url(images/icons/48/play.png) no-repeat center center;'></div>
|
||||
</a>
|
Loading…
Reference in a new issue