From 43be72297a281b41e7b77ba6078aeb48ee007299 Mon Sep 17 00:00:00 2001
From: Fabio Comuni <fabrix.xm@gmail.com>
Date: Wed, 26 Jan 2011 17:17:02 +0100
Subject: [PATCH] Rewrite oembed plugin to use new bbcode tag [embed] and make
 discovery server-side

---
 addon/oembed/oembed.js  | 47 +++--------------------------------------
 addon/oembed/oembed.php | 45 +++++++++++++++++++++++++++++++++++++++
 2 files changed, 48 insertions(+), 44 deletions(-)

diff --git a/addon/oembed/oembed.js b/addon/oembed/oembed.js
index 26541490b1..54547a86ec 100644
--- a/addon/oembed/oembed.js
+++ b/addon/oembed/oembed.js
@@ -3,49 +3,8 @@ function oembed(){
 }
 
 function oembed_do(){
-  embedurl = $('#oembed_url').attr('value');
-  var url = 'http://oohembed.com/oohembed/?url='+escape( embedurl )+"&callback=?";
-  
-  $.getJSON(url, function(data) {
-    var ret="";
-    switch(data.type){
-      case "video": {
-        if (data.thumbnail_url){
-          tw = 200; if (data.thumbnail_width) tw=data.thumbnail_width;
-          th = 180; if (data.thumbnail_height) tw=data.thumbnail_height;
-          ret = "<a href='"+embedurl+"'>";
-          // tiny mce bbcode plugin not support image size......
-          ret += "<img width='"+tw+"' height='"+th+"' src='"+data.thumbnail_url+"'></a>";
-        } else {
-          ret = data.html;  
-        }
-      }; break;
-      case "photo": {
-          // tiny mce bbcode plugin not support image size......        
-          ret = "<img width='"+data.width+"' height='"+data.height+"' src='"+data.url+"'>";
-      }; break;
-      case "link": {
-          ret = "<a href='"+embedurl+"'>"+data.title+"</a>";
-      }; break;
-      case "rich": {
-          ret = data.html; // not so safe... http://www.oembed.com/ : "Consumers may wish to load the HTML in an off-domain iframe to avoid XSS" 
-      }; break;
-      default: {
-        alert("Error retriving data!");
-        return;
-      }
-    }
-    var embedlink = embedurl;
-    if (data.title) embedlink = data.title
-    ret+="<br><a href='"+embedurl+"'>"+embedlink+"</a>";
-    if (data.author_name) {
-      ret+=" by "+data.author_name;
-    }
-    if (data.provider_name) {
-      ret+=" on "+data.provider_name;
-    }
-    tinyMCE.execCommand('mceInsertRawHTML',false,ret);
-    oembed();
-  });
+  embed = "[embed]"+$('#oembed_url').attr('value')+"[/embed]";
   
+  tinyMCE.execCommand('mceInsertRawHTML',false,embed);
+  oembed();
 }
diff --git a/addon/oembed/oembed.php b/addon/oembed/oembed.php
index 55654a5432..82183f3cce 100644
--- a/addon/oembed/oembed.php
+++ b/addon/oembed/oembed.php
@@ -10,11 +10,13 @@
 function oembed_install() {
   register_hook('jot_tool', 'addon/oembed/oembed.php', 'oembed_hook_jot_tool');
   register_hook('page_header', 'addon/oembed/oembed.php', 'oembed_hook_page_header');
+  register_hook('bbcode', 'addon/oembed/oembed.php', 'oembed_hook_bbcode');
 }
 
 function oembed_uninstall() {
   unregister_hook('jot_tool', 'addon/oembed/oembed.php', 'oembed_hook_jot_tool');
   unregister_hook('page_header', 'addon/oembed/oembed.php', 'oembed_hook_page_header');
+  unregister_hook('bbcode', 'addon/oembed/oembed.php', 'oembed_hook_bbcode');
 }
 
 function oembed_hook_page_header($a, &$b){
@@ -51,6 +53,49 @@ function oembed_hook_jot_tool($a, &$b) {
   ';
 }
 
+function oembed_replacecb($matches){
+  $embedurl=$matches[1];
+  $ourl = "http://oohembed.com/oohembed/?url=".urlencode($embedurl);  
+  $txt = fetch_url($ourl);
+  $j = json_decode($txt);
+  $ret="<!-- oembed $embedurl -->";
+  switch ($j->type) {
+    case "video": {
+       if (isset($j->thumbnail_url)) {
+         $tw = (isset($j->thumbnail_width)) ? $j->thumbnail_width:200;
+         $th = (isset($j->thumbnail_height)) ? $j->thumbnail_height:180;
+         $ret = "<a href='#' onclick='this.innerHTML=unescape(\"".urlencode($j->html)."\").replace(/\+/g,\" \"); return false;' >";
+         $ret.= "<img width='$tw' height='$th' src='".$j->thumbnail_url."'>";
+         $ret.= "</a>";
+       } else {
+         $ret=$j->html;
+       }
+       $ret.="<br>";
+    }; break;
+    case "photo": {
+      $ret = "<img width='".$j->width."' height='".$j->height."' src='".$j->url."'>";
+      $ret.="<br>";
+    }; break;  
+    case "link": {
+      //$ret = "<a href='".$embedurl."'>".$j->title."</a>";
+    }; break;  
+    case "rich": {
+      // not so safe.. 
+      $ret = "<blockquote>".$j->html."</blockquote>";
+    }; break;
+  }
+  
+  $embedlink = (isset($j->title))?$j->title:$embedurl;
+  $ret .= "<a href='$embedurl'>$embedlink</a>";
+  if (isset($j->author_name)) $ret.=" by ".$j->author_name;
+  if (isset($j->provider_name)) $ret.=" on ".$j->provider_name;
+  $ret.="<!-- /oembed $embedurl -->";
+  return $ret;
+}
+
+function oembed_hook_bbcode($a, &$text){
+  $text = preg_replace_callback("/\[embed\](.+?)\[\/embed\]/is", oembed_replacecb ,$text);
+}
 
 
 ?>
\ No newline at end of file