onebox 1.5.50 → 1.5.60

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 71f76b700cbeee61a2da9f769fb2c50c266809a4
-  data.tar.gz: 42258aef7d0cddad2730a8eaf1e3004704be2b71
+  metadata.gz: 6284a87a3edbd77cb74a40d458a2774f350ec86d
+  data.tar.gz: 2e2e203e7321cd7e453f50e18e3bd73a56ade1e0
 SHA512:
-  metadata.gz: 0a6e0d9a253cd2eda029a9e859f161621c93d92e6f6cfefd3528fa4f022d4b57057085510779a2f5d5f212266cc92d5d89a88e0c3f9f7b0072ca7f02cfe2a85e
-  data.tar.gz: 64bbc89d7885b4c54da395bfcc9a31ea9011badeae4da7d31ee99d552ca401e1f36aaf97e0281306b1c15eb780f577e4f5b51a8b9018e2107d1325502c57f10c
+  metadata.gz: af6c86905744a6da99816a166be42e52dd4d785cba77e4e7fa17d4b097262ad7e57ebe27abea5120fcfd2d5db2bff1fdf3a7ba81f1b743616307315fa90fa787
+  data.tar.gz: 7e68883159aa1313539b1501689aafd5f6080359d587d7b138c36c5b1f334188a37a9f5b02c150148dbaa48af0f570ad68432842ec37960bfb89d6cead4203e2

data/lib/onebox/engine.rb

@@ -164,3 +164,4 @@ require_relative "engine/imgur_onebox"
 require_relative "engine/steam_store_widget_onebox"
 require_relative "engine/pastebin_onebox"
 require_relative "engine/slides_onebox"
+require_relative "engine/xkcd_onebox"

data/lib/onebox/engine/github_commit_onebox.rb

@@ -29,8 +29,7 @@ module Onebox
 
         ulink = URI(link)
         result['commit_date'] = Time.parse(result['commit']['author']['date']).strftime("%I:%M%p - %d %b %y")
-        result['repository_path'] = "#{ulink.host}/#{ulink.path.split('/')[1]}/#{ulink.path.split('/')[2]}"
-        result['repository_url'] = "https://#{result['repository_path']}"
+        result['domain'] = "#{ulink.host}/#{ulink.path.split('/')[1]}/#{ulink.path.split('/')[2]}"
         result
       end
     end

data/lib/onebox/engine/github_issue_onebox.rb

@@ -41,8 +41,7 @@ module Onebox
                    closed_at: (@raw['closed_at'].nil? ? "" : @raw['closed_at'].split("T")[0]),
                    closed_by: @raw['closed_by'],
                    avatar: "https://avatars1.githubusercontent.com/u/#{@raw['user']['id']}?v=2&s=96",
-                   repository_path: "#{ulink.host}/#{ulink.path.split('/')[1]}/#{ulink.path.split('/')[2]}",
-                   repository_url: "https://#{ulink.host}/#{ulink.path.split('/')[1]}/#{ulink.path.split('/')[2]}",
+                   domain: "#{ulink.host}/#{ulink.path.split('/')[1]}/#{ulink.path.split('/')[2]}",
                   }
       end
     end

data/lib/onebox/engine/github_pullrequest_onebox.rb

@@ -23,8 +23,7 @@ module Onebox
         result['link'] = link
         result['created_at'] = Time.parse(result['created_at']).strftime("%I:%M%p - %d %b %y")
         ulink = URI(link)
-        result['repository_path'] = "#{ulink.host}/#{ulink.path.split('/')[1]}/#{ulink.path.split('/')[2]}"
-        result['repository_url'] = "https://#{result['repository_path']}"
+        result['domain'] = "#{ulink.host}/#{ulink.path.split('/')[1]}/#{ulink.path.split('/')[2]}"
         result
       end
     end

data/lib/onebox/engine/slides_onebox.rb

@@ -6,27 +6,24 @@ module Onebox
 
       matches_regexp(/^https?:\/\/slides\.com\/[\p{Alnum}_\-]+\/[\p{Alnum}_\-]+$/)
 
-
       def to_html
-        if uri.path =~ /^\/[\p{Alnum}_\-]+\/[\p{Alnum}_\-]+$/
-          "<iframe src=\"//slides.com#{uri.path}/embed?style=light\" width=\"576\" height=\"420\" scrolling=\"no\" frameborder=\"0\" webkitallowfullscreen mozallowfullscreen allowfullscreen></iframe>"
-        end
+        <<-HTML
+          <iframe src="//slides.com#{uri.path}/embed?style=light"
+                  width="576"
+                  height="420"
+                  scrolling="no"
+                  frameborder="0"
+                  webkitallowfullscreen
+                  mozallowfullscreen
+                  allowfullscreen>
+          </iframe>"
+        HTML
       end
 
       def placeholder_html
-        # get_opengraph_data
-        "<img src='#{get_opengraph_data[:images].first}'>"
+        "<img src='#{raw[:image]}'>"
       end
 
-      private
-
-      def get_opengraph_data
-        return @opengraph_data if @opengraph_data
-        response = Onebox::Helpers.fetch_response(url)
-        html_doc = Nokogiri::HTML(response.body)
-
-        @opengraph_data = parse_open_graph(html_doc, url)
-      end
     end
   end
 end

data/lib/onebox/engine/soundcloud_onebox.rb

@@ -8,27 +8,26 @@ module Onebox
       always_https
 
       def to_html
-        oembed_data = get_oembed_data[:html]
-        oembed_data.gsub!('visual=true', 'visual=false') || oembed_data
+        oembed_data[:html].gsub('visual=true', 'visual=false')
       end
 
       def placeholder_html
-        "<img src='#{get_oembed_data[:thumbnail_url]}'>"
+        return if Onebox::Helpers.blank?(oembed_data[:thumbnail_url])
+        "<img src='#{oembed_data[:thumbnail_url]}'>"
       end
 
       private
 
-      def set?
-        url =~ /\/sets\//
-      end
-
-      def get_oembed_data
-        if set?
-          Onebox::Helpers.symbolize_keys(::MultiJson.load(Onebox::Helpers.fetch_response("https://soundcloud.com/oembed.json?url=#{url}").body))
-        else
-          Onebox::Helpers.symbolize_keys(::MultiJson.load(Onebox::Helpers.fetch_response("https://soundcloud.com/oembed.json?url=#{url}&maxheight=166").body))
+        def oembed_data
+          @oembed_data ||= begin
+            oembed_url = "https://soundcloud.com/oembed.json?url=#{url}"
+            oembed_url << "&maxheight=166" if url["/sets/"]
+            Onebox::Helpers.symbolize_keys(::MultiJson.load(Onebox::Helpers.fetch_response(oembed_url).body))
+          rescue
+            {}
+          end
         end
-      end
+
     end
   end
 end

data/lib/onebox/engine/standard_embed.rb

@@ -6,27 +6,20 @@ module Onebox
         @@oembed_providers ||= {}
       end
 
-      def self.opengraph_providers
-        @@opengraph_providers ||= Array.new
-      end
-
       def self.add_oembed_provider(regexp, endpoint)
         oembed_providers[regexp] = endpoint
       end
 
+      def self.opengraph_providers
+        @@opengraph_providers ||= []
+      end
+
       def self.add_opengraph_provider(regexp)
-        opengraph_providers.push(regexp)
+        opengraph_providers << regexp
       end
 
       # Some oembed providers (like meetup.com) don't provide links to themselves
-      add_oembed_provider /(.*\.)?gfycat\.com\//, 'http://gfycat.com/cajax/oembed'
-      add_oembed_provider /www\.kickstarter\.com\//, 'https://www.kickstarter.com/services/oembed'
-      add_oembed_provider /www\.meetup\.com\//, 'http://api.meetup.com/oembed'
-      add_oembed_provider /www\.ted\.com\//, 'http://www.ted.com/services/v1/oembed.json'
-      add_oembed_provider /(.*\.)?vimeo\.com\//, 'http://vimeo.com/api/oembed.json'
-
-      # Sites that work better with OpenGraph
-      add_opengraph_provider /nytimes\.com\//
+      add_oembed_provider(/www\.meetup\.com\//, 'http://api.meetup.com/oembed')
 
       def always_https?
         WhitelistedGenericOnebox.host_matches(uri, WhitelistedGenericOnebox.https_hosts)
@@ -35,92 +28,76 @@ module Onebox
       def raw
         return @raw if @raw
 
-        StandardEmbed.oembed_providers.each do |regexp, endpoint|
-          if url =~ regexp
-            fetch_oembed_raw("#{endpoint}?url=#{url}")
-            return @raw if @raw
-          end
-        end
+        og = get_opengraph
+        twitter = get_twitter
+        oembed = get_oembed
 
-        response = Onebox::Helpers.fetch_response(url)
-        html_doc = Nokogiri::HTML(response.body)
+        @raw = {}
 
-        StandardEmbed.opengraph_providers.each do |regexp|
-          if url =~ regexp
-            @raw = parse_open_graph(html_doc, url)
-            return @raw if @raw
-          end
-        end
+        og.each { |k, v| @raw[k] ||= v unless Onebox::Helpers::blank?(v) }
+        twitter.each { |k, v| @raw[k] ||= v unless Onebox::Helpers::blank?(v) }
+        oembed.each { |k, v| @raw[k] ||= v unless Onebox::Helpers::blank?(v) }
+
+        @raw
+      end
 
-        # Determine if we should use oEmbed or OpenGraph (prefers oEmbed)
-        oembed_alternate = html_doc.at("//link[@type='application/json+oembed']") || html_doc.at("//link[@type='text/json+oembed']")
-        # Do not use oEmbed for WordPress sites (https://meta.discourse.org/t/onebox-for-wordpress-4-4-sites/36765)
-        fetch_oembed_raw(oembed_alternate) unless oembed_alternate.nil? || oembed_alternate['href'] =~ /public-api.wordpress.com\/oembed/ || oembed_alternate['href'] =~ /wp-json\/oembed/
+      private
 
-        open_graph = parse_open_graph(html_doc, url)
-        if @raw
-          @raw[:image] = open_graph.images.first if @raw[:image].nil? && open_graph && open_graph.images
-          return @raw
+        def html_doc
+          return @html_doc if @html_doc
+          response = Onebox::Helpers.fetch_response(url)
+          @html_doc = Nokogiri::HTML(response.body)
         end
 
-        @raw = open_graph
-      end
+        def get_oembed
+          application_json = html_doc.at("//link[@type='application/json+oembed']/@href")
+          oembed_url = application_json.value if application_json
 
-      private
+          text_json = html_doc.at("//link[@type='text/json+oembed']/@href")
+          oembed_url ||= text_json.value if text_json
 
-      def fetch_oembed_raw(oembed_url)
-        return unless oembed_url
-        oembed_url = oembed_url['href'] unless oembed_url['href'].nil?
-        oembed_data = Onebox::Helpers.symbolize_keys(::MultiJson.load(Onebox::Helpers.fetch_response(oembed_url).body))
-        @raw =
-          if oembed_data[:html] && oembed_data[:html].bytesize > 4000
-            # fallback to OpenGraph if oEmbed data size is more than 4000 bytes
-            nil
-          else
-            oembed_data
+          if Onebox::Helpers.blank?(oembed_url)
+            StandardEmbed.oembed_providers.each do |regexp, endpoint|
+              if url[regexp]
+                oembed_url = "#{endpoint}?url=#{url}"
+                break
+              end
+            end
           end
-      rescue Errno::ECONNREFUSED, Net::HTTPError, MultiJson::LoadError
-        @raw = nil
-      end
 
-      def parse_open_graph(html, og_url)
-        og = Struct.new(:url, :type, :title, :description, :images, :metadata, :html).new
-        og.url = og_url
-        og.images = []
-        og.metadata = {}
-
-        attrs_list = %w(title url type description)
-        html.css('meta').each do |m|
-          if m.attribute('property') && m.attribute('property').to_s.match(/^og:/i)
-            # og properties
-            m_content = m.attribute('content').to_s.strip
-            m_name = m.attribute('property').to_s.gsub('og:', '')
-            og.metadata[m_name.to_sym] ||= []
-            og.metadata[m_name.to_sym].push m_content
-            if m_name == "image"
-              image_uri = URI.parse(m_content) rescue nil
-              if image_uri
-                if image_uri.host.nil?
-                  image_uri.host = URI.parse(og_url).host
-                end
-                og.images.push image_uri.to_s
-              end
-            elsif attrs_list.include? m_name
-              og.send("#{m_name}=", m_content) unless m_content.empty?
+          return {} if Onebox::Helpers.blank?(oembed_url)
+
+          Onebox::Helpers.symbolize_keys(::MultiJson.load(Onebox::Helpers.fetch_response(oembed_url).body))
+        rescue Errno::ECONNREFUSED, Net::HTTPError, MultiJson::LoadError
+          {}
+        end
+
+        def get_opengraph
+          og = {}
+
+          html_doc.css('meta').each do |m|
+            if m["property"] && m["property"][/^og:(.+)$/i]
+              value = m["content"].to_s
+              og[$1.tr('-:','_').to_sym] ||= value unless Onebox::Helpers::blank?(value)
             end
           end
-          if m.attribute('name') && m.attribute('name').to_s.match(/^twitter:/i)
-            # twitter properties
-            m_content = m.attribute('content').to_s.strip if m.attribute('content')
-            m_content = m.attribute('value').to_s.strip if m.attribute('value')
-            m_name = m.attribute('name').to_s
-            og.metadata[m_name.to_sym] ||= []
-            og.metadata[m_name.to_sym].push m_content
+
+          og
+        end
+
+        def get_twitter
+          twitter = {}
+
+          html_doc.css('meta').each do |m|
+            if m["name"] && m["name"][/^twitter:(.+)$/i]
+              value = (m["content"] || m["value"]).to_s
+              twitter[$1.tr('-:','_').to_sym] ||= value unless Onebox::Helpers::blank?(value)
+            end
           end
+
+          twitter
         end
 
-        og
-      end
     end
   end
 end

data/lib/onebox/engine/whitelisted_generic_onebox.rb

@@ -16,7 +16,8 @@ module Onebox
       end
 
       def self.default_whitelist
-        %w(23hq.com
+        %w(
+          23hq.com
           500px.com
           8tracks.com
           abc.net.au
@@ -32,13 +33,14 @@ module Onebox
           blip.tv
           bloomberg.com
           businessinsider.com
+          change.org
           clikthrough.com
           cnet.com
           cnn.com
+          codepen.io
           collegehumor.com
           consider.it
           coursera.org
-          codepen.io
           cracked.com
           dailymail.co.uk
           dailymotion.com
@@ -62,8 +64,8 @@ module Onebox
           gfycat.com
           groupon.com
           howtogeek.com
-          huffingtonpost.com
           huffingtonpost.ca
+          huffingtonpost.com
           hulu.com
           ign.com
           ikea.com
@@ -75,9 +77,9 @@ module Onebox
           khanacademy.org
           kickstarter.com
           kinomap.com
+          lessonplanet.com
           liveleak.com
           livestream.com
-          lessonplanet.com
           mashable.com
           medium.com
           meetup.com
@@ -86,8 +88,8 @@ module Onebox
           myshopify.com
           myspace.com
           nba.com
-          nytimes.com
           npr.org
+          nytimes.com
           photobucket.com
           pinterest.com
           reference.com
@@ -122,16 +124,16 @@ module Onebox
           vine.co
           walmart.com
           washingtonpost.com
+          wi.st
           wikia.com
           wikihow.com
           wired.com
           wistia.com
-          wi.st
           wonderhowto.com
           wsj.com
           zappos.com
           zillow.com
-          change.org)
+        )
       end
 
       # Often using the `html` attribute is not what we want, like for some blogs that
@@ -149,7 +151,7 @@ module Onebox
         @html_providers = new_provs
       end
 
-      # A re-written URL coverts http:// -> https://
+      # A re-written URL converts http:// -> https://
       def self.rewrites
         @rewrites ||= https_hosts.dup
       end
@@ -176,176 +178,130 @@ module Onebox
 
       def self.===(other)
         if other.kind_of?(URI)
-          return WhitelistedGenericOnebox.host_matches(other, WhitelistedGenericOnebox.whitelist) ||
-                 WhitelistedGenericOnebox.probable_wordpress(other) ||
-                 WhitelistedGenericOnebox.probable_discourse(other)
+          host_matches(other, whitelist) || probable_wordpress(other) || probable_discourse(other)
         else
           super
         end
       end
 
-      # Generates the HTML for the embedded content
-      def photo_type?
-        ( (data[:type] =~ /photo/ || data[:type] =~ /image/) && data[:type] !~ /photostream/ )
-      end
-
-      def article_type?
-        data[:type] == "article"
-      end
-
-      def rewrite_https(html)
-        return html unless html
-        uri = URI(@url)
-        if WhitelistedGenericOnebox.host_matches(uri, WhitelistedGenericOnebox.rewrites)
-          html.gsub!(/http:\/\//, 'https://')
-        end
-        html
-      end
-
-      def html_type?
-        return data &&
-               data[:html] &&
-               (
-                 (data[:html] =~ /iframe/) ||
-                 WhitelistedGenericOnebox.html_providers.include?(data[:provider_name])
-               )
-      end
-
-      def generic_html
-        return add_thumbnail_class(data[:html]) if html_type?
-        return layout.to_html if article_type?
-        return html_for_video(data[:video]) if data[:video]
-        return image_html if photo_type?
-        return nil if data[:title].nil? || data[:title].empty?
-
-        layout.to_html
-      end
-
       def to_html
         rewrite_https(generic_html)
       end
 
       def placeholder_html
-        result = nil
-        return to_html if article_type?
-        result = image_html if (data[:html] && data[:html] =~ /iframe/) || data[:video] || photo_type?
-        result || to_html
+        return article_html if is_article?
+        return image_html   if has_image? && (is_video? || is_image?)
+        return article_html if has_text? && is_embedded?
+        to_html
       end
 
       def data
-        if raw.is_a?(Hash)
-          raw[:link] ||= link
-          raw[:title] = HTMLEntities.new.decode(raw[:title])
-          return raw
+        @data ||= begin
+          html_entities = HTMLEntities.new
+          d = { link: link }.merge(raw)
+          if !Onebox::Helpers.blank?(d[:title])
+            d[:title] = html_entities.decode(Onebox::Helpers.truncate(d[:title]))
+          end
+          if !Onebox::Helpers.blank?(d[:description])
+            d[:description] = html_entities.decode(Onebox::Helpers.truncate(d[:description], 250))
+          end
+          d
         end
+      end
 
-        data_hash = { link: link, title: HTMLEntities.new.decode(raw.title), description: raw.description }
-        data_hash[:image] = raw.images.first if raw.images && raw.images.first
-        data_hash[:type] = raw.type if raw.type
+      private
 
-        if raw.metadata && raw.metadata[:"video:secure_url"] && raw.metadata[:"video:secure_url"].first
-          data_hash[:video] = raw.metadata[:"video:secure_url"].first
-        elsif raw.metadata && raw.metadata[:video] && raw.metadata[:video].first
-          data_hash[:video] = raw.metadata[:video].first
+        def rewrite_https(html)
+          return unless html
+          uri = URI(@url)
+          html.gsub!("http://", "https://") if WhitelistedGenericOnebox.host_matches(uri, WhitelistedGenericOnebox.rewrites)
+          html
         end
 
-        if raw.metadata && raw.metadata[:"twitter:label1"] && raw.metadata[:"twitter:data1"]
-          data_hash[:twitter_label1] = raw.metadata[:"twitter:label1"].first
-          data_hash[:twitter_data1] = raw.metadata[:"twitter:data1"].first
+        def generic_html
+          return article_html  if is_article?
+          return embedded_html if is_embedded?
+          return video_html    if is_video?
+          return image_html    if is_image?
+          return article_html  if has_text?
         end
 
-        if raw.metadata && raw.metadata[:"twitter:label2"] && raw.metadata[:"twitter:data2"]
-          data_hash[:twitter_label2] = raw.metadata[:"twitter:label2"].first
-          data_hash[:twitter_data2] = raw.metadata[:"twitter:data2"].first
+        def is_article?
+          data[:type] =~ /article/ &&
+          has_text?
         end
 
-        data_hash
-      end
-
-      private
-
-      def image_html
-        return @image_html if @image_html
-
-        return @image_html = "<img src=\"#{data[:image]}\">" if data[:image]
-
-        if data[:thumbnail_url]
-          @image_html = "<img src=\"#{data[:thumbnail_url]}\""
-          @image_html << " width=\"#{data[:thumbnail_width]}\"" if data[:thumbnail_width]
-          @image_html << " height=\"#{data[:thumbnail_height]}\"" if data[:thumbnail_height]
-          @image_html << ">"
+        def has_text?
+          !Onebox::Helpers.blank?(data[:title]) &&
+          !Onebox::Helpers.blank?(data[:description])
         end
 
-        @image_html
-      end
-
-      def html_for_video(video)
-        if video.is_a?(String)
-          video_url = video
-        elsif video.is_a?(Hash)
-          video_url = video[:_value]
-        else
-          return
+        def is_image?
+          data[:type] =~ /photo|image/ &&
+          data[:type] !~ /photostream/ &&
+          has_image?
         end
 
+        def has_image?
+          !Onebox::Helpers.blank?(data[:image]) ||
+          !Onebox::Helpers.blank?(data[:thumbnail_url])
+        end
 
-        if video_url
-          # opengraph support multiple elements (videos, images ,etc).
-          # We attempt to find a video element with the type of video/mp4
-          # and generate a native <video> element for it.
-
-          if (@raw.metadata && @raw.metadata[:"video:type"])
-            video_type =  @raw.metadata[:"video:type"]
-            if video_type.include? "video/mp4"            # find if there is a video with type
-              if video_type.size > 1                      # if more then one video item based on provided video_type
-                ind = video_type.find_index("video/mp4")  # get the first video index with type video/mp4
-                video_url  = @raw.metadata[:video][ind]   # update video_url
-              end
-
-              attr = append_attribute(:width, attr, video)
-              attr = append_attribute(:height, attr, video)
-
-              site_name_and_title  =  ( ("<span style='color:#fff;background:#9B9B9B;border-radius:3px;padding:3px;margin-right: 5px;'>" + CGI::escapeHTML(@raw.metadata[:site_name][0].to_s) + '</span> ') + CGI::escapeHTML((@raw.title || @raw.description).to_s) )
-              orig_url = @raw.url
-              html_v2 = %Q(
-                <div style='position:relative;padding-top:29px;'>
-                <span style='position: absolute;top:0px;z-index:2;color:#000;white-space:nowrap;text-overflow:ellipsis;word-wrap: break-word;overflow: hidden;display: inline-block;padding: 3px;border-radius: 4px;max-width: 100%;'><a href='#{orig_url}' target='_blank'>#{site_name_and_title}</a></span>
-                <video style='max-width:100%' #{attr} title="#{data[:title]}" controls="" ><source src="#{video_url}"></video>
-                </div>
-                )
-              html = html_v2
-
-            else
-
-              html = "<iframe src=\"#{video_url}\" frameborder=\"0\" title=\"#{data[:title]}\""
-              append_attribute(:width, html, video)
-              append_attribute(:height, html, video)
+        def is_video?
+          data[:type] =~ /video/ &&
+          !Onebox::Helpers.blank?(data[:video])
+        end
 
-              html << "></iframe>"
-            end
+        def is_embedded?
+          data[:html] &&
+          (
+            data[:html]["iframe"] ||
+            WhitelistedGenericOnebox.html_providers.include?(data[:provider_name])
+          )
+        end
 
-          end
-          return html
+        def article_html
+          layout.to_html
         end
-      end
 
-      def append_attribute(attribute, html, video)
-        if video.is_a?(Hash) && video[attribute] && video[attribute].first
-          val = video[attribute].first[:_value]
-          html << " #{attribute.to_s}=\"#{val}\""
+        def image_html
+          src = data[:image] || data[:thumbnail_url]
+          return if Onebox::Helpers.blank?(src)
+
+          alt    = data[:description]  || data[:title]
+          width  = data[:image_width]  || data[:thumbnail_width]
+          height = data[:image_height] || data[:thumbnail_height]
+          "<img src='#{src}' alt='#{alt}' width='#{width}' height='#{height}'>"
         end
-      end
 
-      def add_thumbnail_class(data)
-        fragment = Nokogiri::HTML::fragment(data)
-        if fragment
-          fragment.search('img').each do |img|
-            img['class'] = "thumbnail"
+        def video_html
+          if data[:video_type] == "video/mp4"
+            <<-HTML
+              <video title='#{data[:title]}'
+                     width='#{data[:video_width]}'
+                     height='#{data[:video_height]}'
+                     style='max-width:100%'
+                     controls=''>
+                <source src='#{data[:video]}'>
+              </video>
+            HTML
+          else
+            <<-HTML
+              <iframe src='#{data[:video]}'
+                      title='#{data[:title]}'
+                      width='#{data[:video_width]}'
+                      height='#{data[:video_height]}'
+                      frameborder='0'>
+              </iframe>
+            HTML
           end
         end
 
-        return fragment.to_html
-      end
+        def embedded_html
+          fragment = Nokogiri::HTML::fragment(data[:html])
+          fragment.css("img").each { |img| img["class"] = "thumbnail" }
+          fragment.to_html
+        end
     end
   end
 end