onebox 1.8.81 → 1.8.82

data/lib/onebox/engine.rb

@@ -1,188 +1,188 @@
-module Onebox
-  module Engine
-    def self.included(object)
-      object.extend(ClassMethods)
-    end
-
-    def self.engines
-      constants.select do |constant|
-        constant.to_s =~ /Onebox$/
-      end.map(&method(:const_get))
-    end
-
-    attr_reader :url, :uri
-    attr_reader :cache
-    attr_reader :timeout
-
-    DEFAULT = {}
-    def options
-      @options
-    end
-
-    def options=(opt)
-      return @options if opt.nil? #make sure options provided
-      opt = opt.to_h  if opt.instance_of?(OpenStruct)
-      @options.merge!(opt)
-      @options
-    end
-
-    def initialize(link, cache = nil, timeout = nil)
-      @options = DEFAULT
-      class_name = self.class.name.split("::").last.to_s
-      self.options = Onebox.options[class_name] || {} #Set the engine options extracted from global options.
-
-      @url = link
-      @uri = URI(link)
-      if always_https?
-        @uri.scheme = 'https'
-        @url = @uri.to_s
-      end
-      @cache = cache || Onebox.options.cache
-      @timeout = timeout || Onebox.options.timeout
-    end
-
-    # raises error if not defined in onebox engine.
-    # This is the output method for an engine.
-    def to_html
-      fail NoMethodError, "Engines need to implement this method"
-    end
-
-    # Some oneboxes create iframes or other complicated controls. If you're using
-    # a live editor with HTML preview, rendering those complicated controls can
-    # be slow or cause flickering.
-    #
-    # This method allows engines to produce a placeholder such as static image
-    # frame of a video.
-    #
-    # By default it just calls `to_html` unless implemented.
-    def placeholder_html
-      to_html
-    end
-
-    private
-
-    def record
-      url_result = url
-      result = cache.fetch(url_result) { data }
-      cache[url_result] = result if cache.respond_to?(:key?)
-      result
-    end
-
-    # raises error if not defined in onebox engine
-    # in each onebox, uses either Nokogiri or StandardEmbed to get raw HTML from url
-    def raw
-      fail NoMethodError, "Engines need to implement this method"
-    end
-
-    # raises error if not defined in onebox engine
-    # in each onebox, returns hash of desired onebox content
-    def data
-      fail NoMethodError, "Engines need this method defined"
-    end
-
-    def link
-      @url.gsub(/['\"<>]/,
-        "'" => '&#39;',
-        '"' => '&quot;',
-        '<' => '&lt;',
-        '>' => '&gt;',
-      )
-    end
-
-    def always_https?
-      self.class.always_https?
-    end
-
-    module ClassMethods
-      def ===(other)
-        if other.kind_of?(URI)
-          !!(other.to_s =~ class_variable_get(:@@matcher))
-        else
-          super
-        end
-      end
-
-      def priority
-        100
-      end
-
-      def matches_regexp(r)
-        class_variable_set :@@matcher, r
-      end
-
-      # calculates a name for onebox using the class name of engine
-      def onebox_name
-        name.split("::").last.downcase.gsub(/onebox/, "")
-      end
-
-      def always_https
-        @https = true
-      end
-
-      def always_https?
-        @https
-      end
-    end
-  end
-end
-
-require_relative "helpers"
-require_relative "layout_support"
-require_relative "file_type_finder"
-require_relative "engine/standard_embed"
-require_relative "engine/html"
-require_relative "engine/json"
-require_relative "engine/amazon_onebox"
-require_relative "engine/github_issue_onebox"
-require_relative "engine/github_blob_onebox"
-require_relative "engine/github_commit_onebox"
-require_relative "engine/github_gist_onebox"
-require_relative "engine/github_pullrequest_onebox"
-require_relative "engine/google_calendar_onebox"
-require_relative "engine/google_docs_onebox"
-require_relative "engine/google_maps_onebox"
-require_relative "engine/google_play_app_onebox"
-require_relative "engine/image_onebox"
-require_relative "engine/video_onebox"
-require_relative "engine/audio_onebox"
-require_relative "engine/stack_exchange_onebox"
-require_relative "engine/twitter_status_onebox"
-require_relative "engine/wikimedia_onebox"
-require_relative "engine/wikipedia_onebox"
-require_relative "engine/youtube_onebox"
-require_relative "engine/youku_onebox"
-require_relative "engine/douban_onebox"
-require_relative "engine/whitelisted_generic_onebox"
-require_relative "engine/pubmed_onebox"
-require_relative "engine/soundcloud_onebox"
-require_relative "engine/imgur_onebox"
-require_relative "engine/pastebin_onebox"
-require_relative "engine/slides_onebox"
-require_relative "engine/xkcd_onebox"
-require_relative "engine/giphy_onebox"
-require_relative "engine/gfycat_onebox"
-require_relative "engine/typeform_onebox"
-require_relative "engine/vimeo_onebox"
-require_relative "engine/steam_store_onebox"
-require_relative "engine/sketchfab_onebox"
-require_relative "engine/audioboom_onebox"
-require_relative "engine/replit_onebox"
-require_relative "engine/asciinema_onebox"
-require_relative "engine/mixcloud_onebox"
-require_relative "engine/bandcamp_onebox"
-require_relative "engine/coub_onebox"
-require_relative "engine/flickr_onebox"
-require_relative "engine/flickr_shortened_onebox"
-require_relative "engine/five_hundred_px_onebox"
-require_relative "engine/pdf_onebox"
-require_relative "engine/twitch_clips_onebox"
-require_relative "engine/twitch_stream_onebox"
-require_relative "engine/twitch_video_onebox"
-require_relative "engine/trello_onebox"
-require_relative "engine/wechat_mp_onebox"
-require_relative "engine/cloudapp_onebox"
-require_relative "engine/wistia_onebox"
-require_relative "engine/instagram_onebox"
-require_relative "engine/gitlab_blob_onebox"
-require_relative "engine/google_photos_onebox"
-require_relative "engine/kaltura_onebox"
+module Onebox
+  module Engine
+    def self.included(object)
+      object.extend(ClassMethods)
+    end
+
+    def self.engines
+      constants.select do |constant|
+        constant.to_s =~ /Onebox$/
+      end.map(&method(:const_get))
+    end
+
+    attr_reader :url, :uri
+    attr_reader :cache
+    attr_reader :timeout
+
+    DEFAULT = {}
+    def options
+      @options
+    end
+
+    def options=(opt)
+      return @options if opt.nil? #make sure options provided
+      opt = opt.to_h  if opt.instance_of?(OpenStruct)
+      @options.merge!(opt)
+      @options
+    end
+
+    def initialize(link, cache = nil, timeout = nil)
+      @options = DEFAULT
+      class_name = self.class.name.split("::").last.to_s
+      self.options = Onebox.options[class_name] || {} #Set the engine options extracted from global options.
+
+      @url = link
+      @uri = URI(link)
+      if always_https?
+        @uri.scheme = 'https'
+        @url = @uri.to_s
+      end
+      @cache = cache || Onebox.options.cache
+      @timeout = timeout || Onebox.options.timeout
+    end
+
+    # raises error if not defined in onebox engine.
+    # This is the output method for an engine.
+    def to_html
+      fail NoMethodError, "Engines need to implement this method"
+    end
+
+    # Some oneboxes create iframes or other complicated controls. If you're using
+    # a live editor with HTML preview, rendering those complicated controls can
+    # be slow or cause flickering.
+    #
+    # This method allows engines to produce a placeholder such as static image
+    # frame of a video.
+    #
+    # By default it just calls `to_html` unless implemented.
+    def placeholder_html
+      to_html
+    end
+
+    private
+
+    def record
+      url_result = url
+      result = cache.fetch(url_result) { data }
+      cache[url_result] = result if cache.respond_to?(:key?)
+      result
+    end
+
+    # raises error if not defined in onebox engine
+    # in each onebox, uses either Nokogiri or StandardEmbed to get raw HTML from url
+    def raw
+      fail NoMethodError, "Engines need to implement this method"
+    end
+
+    # raises error if not defined in onebox engine
+    # in each onebox, returns hash of desired onebox content
+    def data
+      fail NoMethodError, "Engines need this method defined"
+    end
+
+    def link
+      @url.gsub(/['\"<>]/,
+        "'" => '&#39;',
+        '"' => '&quot;',
+        '<' => '&lt;',
+        '>' => '&gt;',
+      )
+    end
+
+    def always_https?
+      self.class.always_https?
+    end
+
+    module ClassMethods
+      def ===(other)
+        if other.kind_of?(URI)
+          !!(other.to_s =~ class_variable_get(:@@matcher))
+        else
+          super
+        end
+      end
+
+      def priority
+        100
+      end
+
+      def matches_regexp(r)
+        class_variable_set :@@matcher, r
+      end
+
+      # calculates a name for onebox using the class name of engine
+      def onebox_name
+        name.split("::").last.downcase.gsub(/onebox/, "")
+      end
+
+      def always_https
+        @https = true
+      end
+
+      def always_https?
+        @https
+      end
+    end
+  end
+end
+
+require_relative "helpers"
+require_relative "layout_support"
+require_relative "file_type_finder"
+require_relative "engine/standard_embed"
+require_relative "engine/html"
+require_relative "engine/json"
+require_relative "engine/amazon_onebox"
+require_relative "engine/github_issue_onebox"
+require_relative "engine/github_blob_onebox"
+require_relative "engine/github_commit_onebox"
+require_relative "engine/github_gist_onebox"
+require_relative "engine/github_pullrequest_onebox"
+require_relative "engine/google_calendar_onebox"
+require_relative "engine/google_docs_onebox"
+require_relative "engine/google_maps_onebox"
+require_relative "engine/google_play_app_onebox"
+require_relative "engine/image_onebox"
+require_relative "engine/video_onebox"
+require_relative "engine/audio_onebox"
+require_relative "engine/stack_exchange_onebox"
+require_relative "engine/twitter_status_onebox"
+require_relative "engine/wikimedia_onebox"
+require_relative "engine/wikipedia_onebox"
+require_relative "engine/youtube_onebox"
+require_relative "engine/youku_onebox"
+require_relative "engine/douban_onebox"
+require_relative "engine/whitelisted_generic_onebox"
+require_relative "engine/pubmed_onebox"
+require_relative "engine/soundcloud_onebox"
+require_relative "engine/imgur_onebox"
+require_relative "engine/pastebin_onebox"
+require_relative "engine/slides_onebox"
+require_relative "engine/xkcd_onebox"
+require_relative "engine/giphy_onebox"
+require_relative "engine/gfycat_onebox"
+require_relative "engine/typeform_onebox"
+require_relative "engine/vimeo_onebox"
+require_relative "engine/steam_store_onebox"
+require_relative "engine/sketchfab_onebox"
+require_relative "engine/audioboom_onebox"
+require_relative "engine/replit_onebox"
+require_relative "engine/asciinema_onebox"
+require_relative "engine/mixcloud_onebox"
+require_relative "engine/bandcamp_onebox"
+require_relative "engine/coub_onebox"
+require_relative "engine/flickr_onebox"
+require_relative "engine/flickr_shortened_onebox"
+require_relative "engine/five_hundred_px_onebox"
+require_relative "engine/pdf_onebox"
+require_relative "engine/twitch_clips_onebox"
+require_relative "engine/twitch_stream_onebox"
+require_relative "engine/twitch_video_onebox"
+require_relative "engine/trello_onebox"
+require_relative "engine/wechat_mp_onebox"
+require_relative "engine/cloudapp_onebox"
+require_relative "engine/wistia_onebox"
+require_relative "engine/instagram_onebox"
+require_relative "engine/gitlab_blob_onebox"
+require_relative "engine/google_photos_onebox"
+require_relative "engine/kaltura_onebox"

data/lib/onebox/engine/amazon_onebox.rb

@@ -1,167 +1,167 @@
-require 'json'
-require "onebox/open_graph"
-
-module Onebox
-  module Engine
-    class AmazonOnebox
-      include Engine
-      include LayoutSupport
-      include HTML
-
-      always_https
-      matches_regexp(/^https?:\/\/(?:www\.)?(?:smile\.)?(amazon|amzn)\.(?<tld>com|ca|de|it|es|fr|co\.jp|co\.uk|cn|in|com\.br)\//)
-
-      def url
-        if match && match[:id]
-          return "https://www.amazon.#{tld}/gp/aw/d/#{URI::encode(match[:id])}"
-        end
-
-        @url
-      end
-
-      def tld
-        @tld || @@matcher.match(@url)["tld"]
-      end
-
-      def http_params
-        {
-          'User-Agent' =>
-          'Mozilla/5.0 (iPhone; CPU iPhone OS 5_0_1 like Mac OS X) AppleWebKit/534.46 (KHTML, like Gecko) Version/5.1 Mobile/9A405 Safari/7534.48.3'
-        }
-      end
-
-      private
-
-      def match
-        @match ||= @url.match(/(?:d|g)p\/(?:product\/)?(?<id>[^\/]+)(?:\/|$)/mi)
-      end
-
-      def image
-        if (main_image = raw.css("#main-image")) && main_image.any?
-          attributes = main_image.first.attributes
-
-          return attributes["data-a-hires"].to_s if attributes["data-a-hires"]
-
-          if attributes["data-a-dynamic-image"]
-            return ::JSON.parse(attributes["data-a-dynamic-image"].value).keys.first
-          end
-        end
-
-        if (landing_image = raw.css("#landingImage")) && landing_image.any?
-          landing_image.first["src"].to_s
-        end
-
-        if (ebook_image = raw.css("#ebooksImgBlkFront")) && ebook_image.any?
-          ::JSON.parse(ebook_image.first.attributes["data-a-dynamic-image"].value).keys.first
-        end
-      end
-
-      def price
-        # get item price (Amazon markup is inconsistent, deal with it)
-        if raw.css("#priceblock_ourprice .restOfPrice")[0] && raw.css("#priceblock_ourprice .restOfPrice")[0].inner_text
-          "#{raw.css("#priceblock_ourprice .restOfPrice")[0].inner_text}#{raw.css("#priceblock_ourprice .buyingPrice")[0].inner_text}.#{raw.css("#priceblock_ourprice .restOfPrice")[1].inner_text}"
-        elsif raw.css("#priceblock_dealprice") && (dealprice = raw.css("#priceblock_dealprice span")[0])
-          dealprice.inner_text
-        elsif !raw.css("#priceblock_ourprice").inner_text.empty?
-          raw.css("#priceblock_ourprice").inner_text
-        else
-          raw.css(".mediaMatrixListItem.a-active .a-color-price").inner_text
-        end
-      end
-
-      def multiple_authors(authors_xpath)
-        author_list = raw.xpath(authors_xpath)
-        authors = []
-        author_list.each { |a| authors << a.inner_text.strip }
-        authors.join(", ")
-      end
-
-      def data
-        og = ::Onebox::OpenGraph.new(raw)
-
-        if raw.at_css('#dp.book_mobile') #printed books
-          title = raw.at("h1#title")&.inner_text
-          authors = raw.at_css('#byline_secondary_view_div') ? multiple_authors("//div[@id='byline_secondary_view_div']//span[@class='a-text-bold']") : raw.at("#byline")&.inner_text
-          rating = raw.at("#averageCustomerReviews_feature_div .a-icon")&.inner_text || raw.at("#cmrsArcLink .a-icon")&.inner_text
-
-          table_xpath = "//div[@id='productDetails_secondary_view_div']//table[@id='productDetails_techSpec_section_1']"
-          isbn = raw.xpath("#{table_xpath}//tr[8]//td").inner_text.strip
-
-          # if ISBN is misplaced or absent it's hard to find out which data is
-          # available and where to find it so just set it all to nil
-          if /^\d(\-?\d){12}$/.match(isbn)
-            publisher = raw.xpath("#{table_xpath}//tr[1]//td").inner_text.strip
-            published = raw.xpath("#{table_xpath}//tr[2]//td").inner_text.strip
-            book_length = raw.xpath("#{table_xpath}//tr[6]//td").inner_text.strip
-          else
-            isbn = publisher = published = book_length = nil
-          end
-
-          result = {
-            link: link,
-            title: title,
-            by_info: authors,
-            image: og.image || image,
-            description: raw.at("#productDescription")&.inner_text,
-            rating: "#{rating}#{', ' if rating && (!isbn&.empty? || !price&.empty?)}",
-            price: price,
-            isbn_asin_text: "ISBN",
-            isbn_asin: isbn,
-            publisher: publisher,
-            published: "#{published}#{', ' if published && !price&.empty?}"
-          }
-
-        elsif raw.at_css('#dp.ebooks_mobile') # ebooks
-          title = raw.at("#ebooksTitle")&.inner_text
-          authors = raw.at_css('#a-popover-mobile-udp-contributor-popover-id') ? multiple_authors("//div[@id='a-popover-mobile-udp-contributor-popover-id']//span[contains(@class,'a-text-bold')]") : (raw.at("#byline")&.inner_text&.strip || raw.at("#bylineInfo")&.inner_text&.strip)
-          rating = raw.at("#averageCustomerReviews_feature_div .a-icon")&.inner_text || raw.at("#cmrsArcLink .a-icon")&.inner_text || raw.at("#acrCustomerReviewLink .a-icon")&.inner_text
-
-          table_xpath = "//div[@id='detailBullets_secondary_view_div']//ul"
-          asin = raw.xpath("#{table_xpath}//li[4]/span/span[2]").inner_text
-
-          # if ASIN is misplaced or absent it's hard to find out which data is
-          # available and where to find it so just set it all to nil
-          if /^[0-9A-Z]{10}$/.match(asin)
-            publisher = raw.xpath("#{table_xpath}//li[2]/span/span[2]").inner_text
-            published = raw.xpath("#{table_xpath}//li[1]/span/span[2]").inner_text
-          else
-            asin = publisher = published = nil
-          end
-
-          result = {
-            link: link,
-            title: title,
-            by_info: authors,
-            image: og.image || image,
-            description: raw.at("#productDescription")&.inner_text,
-            rating: "#{rating}#{', ' if rating && (!asin&.empty? || !price&.empty?)}",
-            price: price,
-            isbn_asin_text: "ASIN",
-            isbn_asin: asin,
-            publisher: publisher,
-            published: "#{published}#{', ' if published && !price&.empty?}"
-          }
-
-        else
-          title = og.title || CGI.unescapeHTML(raw.css("title").inner_text)
-          result = {
-            link: link,
-            title: title,
-            image: og.image || image,
-            price: price
-          }
-
-          result[:by_info] = raw.at("#by-line")
-          result[:by_info] = Onebox::Helpers.clean(result[:by_info].inner_html) if result[:by_info]
-
-          summary = raw.at("#productDescription")
-          result[:description] = og.description || (summary && summary.inner_text)
-        end
-
-        result[:price] = nil if result[:price].start_with?("$0") || result[:price] == 0
-
-        result
-      end
-    end
-  end
-end
+require 'json'
+require "onebox/open_graph"
+
+module Onebox
+  module Engine
+    class AmazonOnebox
+      include Engine
+      include LayoutSupport
+      include HTML
+
+      always_https
+      matches_regexp(/^https?:\/\/(?:www\.)?(?:smile\.)?(amazon|amzn)\.(?<tld>com|ca|de|it|es|fr|co\.jp|co\.uk|cn|in|com\.br)\//)
+
+      def url
+        if match && match[:id]
+          return "https://www.amazon.#{tld}/gp/aw/d/#{URI::encode(match[:id])}"
+        end
+
+        @url
+      end
+
+      def tld
+        @tld || @@matcher.match(@url)["tld"]
+      end
+
+      def http_params
+        {
+          'User-Agent' =>
+          'Mozilla/5.0 (iPhone; CPU iPhone OS 5_0_1 like Mac OS X) AppleWebKit/534.46 (KHTML, like Gecko) Version/5.1 Mobile/9A405 Safari/7534.48.3'
+        }
+      end
+
+      private
+
+      def match
+        @match ||= @url.match(/(?:d|g)p\/(?:product\/)?(?<id>[^\/]+)(?:\/|$)/mi)
+      end
+
+      def image
+        if (main_image = raw.css("#main-image")) && main_image.any?
+          attributes = main_image.first.attributes
+
+          return attributes["data-a-hires"].to_s if attributes["data-a-hires"]
+
+          if attributes["data-a-dynamic-image"]
+            return ::JSON.parse(attributes["data-a-dynamic-image"].value).keys.first
+          end
+        end
+
+        if (landing_image = raw.css("#landingImage")) && landing_image.any?
+          landing_image.first["src"].to_s
+        end
+
+        if (ebook_image = raw.css("#ebooksImgBlkFront")) && ebook_image.any?
+          ::JSON.parse(ebook_image.first.attributes["data-a-dynamic-image"].value).keys.first
+        end
+      end
+
+      def price
+        # get item price (Amazon markup is inconsistent, deal with it)
+        if raw.css("#priceblock_ourprice .restOfPrice")[0] && raw.css("#priceblock_ourprice .restOfPrice")[0].inner_text
+          "#{raw.css("#priceblock_ourprice .restOfPrice")[0].inner_text}#{raw.css("#priceblock_ourprice .buyingPrice")[0].inner_text}.#{raw.css("#priceblock_ourprice .restOfPrice")[1].inner_text}"
+        elsif raw.css("#priceblock_dealprice") && (dealprice = raw.css("#priceblock_dealprice span")[0])
+          dealprice.inner_text
+        elsif !raw.css("#priceblock_ourprice").inner_text.empty?
+          raw.css("#priceblock_ourprice").inner_text
+        else
+          raw.css(".mediaMatrixListItem.a-active .a-color-price").inner_text
+        end
+      end
+
+      def multiple_authors(authors_xpath)
+        author_list = raw.xpath(authors_xpath)
+        authors = []
+        author_list.each { |a| authors << a.inner_text.strip }
+        authors.join(", ")
+      end
+
+      def data
+        og = ::Onebox::OpenGraph.new(raw)
+
+        if raw.at_css('#dp.book_mobile') #printed books
+          title = raw.at("h1#title")&.inner_text
+          authors = raw.at_css('#byline_secondary_view_div') ? multiple_authors("//div[@id='byline_secondary_view_div']//span[@class='a-text-bold']") : raw.at("#byline")&.inner_text
+          rating = raw.at("#averageCustomerReviews_feature_div .a-icon")&.inner_text || raw.at("#cmrsArcLink .a-icon")&.inner_text
+
+          table_xpath = "//div[@id='productDetails_secondary_view_div']//table[@id='productDetails_techSpec_section_1']"
+          isbn = raw.xpath("#{table_xpath}//tr[8]//td").inner_text.strip
+
+          # if ISBN is misplaced or absent it's hard to find out which data is
+          # available and where to find it so just set it all to nil
+          if /^\d(\-?\d){12}$/.match(isbn)
+            publisher = raw.xpath("#{table_xpath}//tr[1]//td").inner_text.strip
+            published = raw.xpath("#{table_xpath}//tr[2]//td").inner_text.strip
+            book_length = raw.xpath("#{table_xpath}//tr[6]//td").inner_text.strip
+          else
+            isbn = publisher = published = book_length = nil
+          end
+
+          result = {
+            link: link,
+            title: title,
+            by_info: authors,
+            image: og.image || image,
+            description: raw.at("#productDescription")&.inner_text,
+            rating: "#{rating}#{', ' if rating && (!isbn&.empty? || !price&.empty?)}",
+            price: price,
+            isbn_asin_text: "ISBN",
+            isbn_asin: isbn,
+            publisher: publisher,
+            published: "#{published}#{', ' if published && !price&.empty?}"
+          }
+
+        elsif raw.at_css('#dp.ebooks_mobile') # ebooks
+          title = raw.at("#ebooksTitle")&.inner_text
+          authors = raw.at_css('#a-popover-mobile-udp-contributor-popover-id') ? multiple_authors("//div[@id='a-popover-mobile-udp-contributor-popover-id']//span[contains(@class,'a-text-bold')]") : (raw.at("#byline")&.inner_text&.strip || raw.at("#bylineInfo")&.inner_text&.strip)
+          rating = raw.at("#averageCustomerReviews_feature_div .a-icon")&.inner_text || raw.at("#cmrsArcLink .a-icon")&.inner_text || raw.at("#acrCustomerReviewLink .a-icon")&.inner_text
+
+          table_xpath = "//div[@id='detailBullets_secondary_view_div']//ul"
+          asin = raw.xpath("#{table_xpath}//li[4]/span/span[2]").inner_text
+
+          # if ASIN is misplaced or absent it's hard to find out which data is
+          # available and where to find it so just set it all to nil
+          if /^[0-9A-Z]{10}$/.match(asin)
+            publisher = raw.xpath("#{table_xpath}//li[2]/span/span[2]").inner_text
+            published = raw.xpath("#{table_xpath}//li[1]/span/span[2]").inner_text
+          else
+            asin = publisher = published = nil
+          end
+
+          result = {
+            link: link,
+            title: title,
+            by_info: authors,
+            image: og.image || image,
+            description: raw.at("#productDescription")&.inner_text,
+            rating: "#{rating}#{', ' if rating && (!asin&.empty? || !price&.empty?)}",
+            price: price,
+            isbn_asin_text: "ASIN",
+            isbn_asin: asin,
+            publisher: publisher,
+            published: "#{published}#{', ' if published && !price&.empty?}"
+          }
+
+        else
+          title = og.title || CGI.unescapeHTML(raw.css("title").inner_text)
+          result = {
+            link: link,
+            title: title,
+            image: og.image || image,
+            price: price
+          }
+
+          result[:by_info] = raw.at("#by-line")
+          result[:by_info] = Onebox::Helpers.clean(result[:by_info].inner_html) if result[:by_info]
+
+          summary = raw.at("#productDescription")
+          result[:description] = og.description || (summary && summary.inner_text)
+        end
+
+        result[:price] = nil if result[:price].start_with?("$0") || result[:price] == 0
+
+        result
+      end
+    end
+  end
+end