wgit 0.0.17 → 0.0.18

data/lib/wgit/database/connection_details.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require_relative '../assertable'
 
 module Wgit
@@ -9,7 +11,7 @@ module Wgit
   CONNECTION_DETAILS = {}
 
   # The keys required for a successful database connection.
-  CONNECTION_KEYS_REQUIRED = ['DB_CONNECTION_STRING']
+  CONNECTION_KEYS_REQUIRED = ['DB_CONNECTION_STRING'].freeze
 
   # Set the database's connection details from the given hash. It is your
   # responsibility to ensure the correct hash vars are present and set.
@@ -34,6 +36,6 @@ module Wgit
   # @raise [KeyError] If any of the required connection details are missing.
   # @return [Hash] Containing the database connection details from the ENV.
   def self.set_connection_details_from_env
-    self.set_connection_details(ENV)
+    set_connection_details(ENV)
   end
 end

data/lib/wgit/database/database.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require_relative '../document'
 require_relative '../url'
 require_relative '../utils'
@@ -7,7 +9,6 @@ require 'logger'
 require 'mongo'
 
 module Wgit
-
   # Class modeling a DB connection and CRUD operations for the Url and
   # Document collections.
   class Database
@@ -73,7 +74,7 @@ module Wgit
     # @yield [url] Given each Url returned from the DB.
     # @return [Array<Wgit::Url>] The Urls obtained from the DB.
     def urls(crawled = nil, limit = 0, skip = 0)
-      crawled.nil? ? query = {} : query = { crawled: crawled }
+      query = crawled.nil? ? {} : { crawled: crawled }
 
       sort = { date_added: 1 }
       results = retrieve(:urls, query, sort, {}, limit, skip)
@@ -129,12 +130,12 @@ module Wgit
     # @return [Array<Wgit::Document>] The search results obtained from the DB.
     def search(query, whole_sentence = false, limit = 10, skip = 0)
       query.strip!
-      query.replace("\"" + query + "\"") if whole_sentence
+      query.replace('"' + query + '"') if whole_sentence
 
       # The sort_proj sorts based on the most search hits.
       # We use the sort_proj hash as both a sort and a projection below.
       # :$caseSensitive => case_sensitive, 3.2+ only.
-      sort_proj = { score: { :$meta => "textScore" } }
+      sort_proj = { score: { :$meta => 'textScore' } }
       query = { :$text => { :$search => query } }
 
       results = retrieve(:documents, query, sort_proj, sort_proj, limit, skip)
@@ -188,8 +189,8 @@ module Wgit
     # @param url [Wgit::Url] The Url to search the DB for.
     # @return [Boolean] True if url exists, otherwise false.
     def url?(url)
-      h = { "url" => url }
-      not @@client[:urls].find(h).none?
+      h = { 'url' => url }
+      @@client[:urls].find(h).any?
     end
 
     # Returns whether or not a record with the given doc.url (which is unique)
@@ -199,8 +200,8 @@ module Wgit
     # @return [Boolean] True if doc exists, otherwise false.
     def doc?(doc)
       url = doc.respond_to?(:url) ? doc.url : doc
-      h = { "url" => url }
-      not @@client[:documents].find(h).none?
+      h = { 'url' => url }
+      @@client[:documents].find(h).any?
     end
 
     ### Update Data ###
@@ -220,19 +221,25 @@ module Wgit
       end
     end
 
-  private
+    protected
 
     # Return if the write to the DB succeeded or not.
+    #
+    # @param result [Mongo::Object] The operation result.
+    # @param count [Integer] The number of records written to.
+    # @param multi [Boolean] True if more than one record is being written to.
+    # @raise [RuntimeError] If result.class isn't supported.
+    # @return [Boolean] True if the write was successful.
     def write_succeeded?(result, count = 1, multi = false)
       case result.class.to_s
       # Single create result.
-      when "Mongo::Operation::Insert::Result"
+      when 'Mongo::Operation::Insert::Result'
         result.documents.first[:err].nil?
       # Multiple create result.
-      when "Mongo::BulkWrite::Result"
+      when 'Mongo::BulkWrite::Result'
         result.inserted_count == count
       # Single and multiple update result.
-      when "Mongo::Operation::Update::Result"
+      when 'Mongo::Operation::Update::Result'
         if multi
           result.n == count
         else
@@ -240,50 +247,65 @@ module Wgit
         end
       # Class no longer used, have you upgraded the 'mongo' gem?
       else
-        raise "Result class not currently supported: #{result.class.to_s}"
+        raise "Result class not currently supported: #{result.class}"
       end
     end
 
     # Insert one or more Url objects into the DB.
+    #
+    # @param url_or_urls [Wgit::Url, Array<Wgit::Url>] The Url or Url's to
+    #   insert.
+    # @raise [RuntimeError] If url_or_urls isn't of the correct type.
+    # @return [Integer] The number of inserted Url's.
     def insert_urls(url_or_urls)
-      unless url_or_urls.respond_to?(:map)
-        assert_type(url_or_urls, Url)
-        url_or_urls = Wgit::Model.url(url_or_urls)
-      else
+      if url_or_urls.respond_to?(:map)
         assert_arr_types(url_or_urls, Url)
         url_or_urls = url_or_urls.map do |url|
           Wgit::Model.url(url)
         end
+      else
+        assert_type(url_or_urls, Url)
+        url_or_urls = Wgit::Model.url(url_or_urls)
       end
       create(:urls, url_or_urls)
     end
 
     # Insert one or more Document objects into the DB.
+    #
+    # @param doc_or_docs [Wgit::Document, Array<Wgit::Document>] The Document
+    #   or Document's to insert.
+    # @raise [RuntimeError] If doc_or_docs isn't of the correct type.
+    # @return [Integer] The number of inserted Document's.
     def insert_docs(doc_or_docs)
-      unless doc_or_docs.respond_to?(:map)
-        assert_type(doc_or_docs, [Document, Hash])
-        unless doc_or_docs.is_a?(Hash)
-          doc_or_docs = Wgit::Model.document(doc_or_docs)
-        end
-      else
+      if doc_or_docs.respond_to?(:map)
         assert_arr_types(doc_or_docs, [Document, Hash])
         doc_or_docs = doc_or_docs.map do |doc|
           Wgit::Model.document(doc) unless doc.is_a?(Hash)
         end
+      else
+        assert_type(doc_or_docs, [Document, Hash])
+        unless doc_or_docs.is_a?(Hash)
+          doc_or_docs = Wgit::Model.document(doc_or_docs)
+        end
       end
       create(:documents, doc_or_docs)
     end
 
     # Create/insert one or more Url or Document records into the DB.
+    #
+    # @param collection [Symbol] Either :urls or :documents.
+    # @param data [Hash, Array<Wgit::Url, Wgit::Document>] The data to insert.
+    # @raise [RuntimeError] If the data type is incorrect or if the write
+    #   fails.
+    # @return [Integer] The number of inserted Objects.
     def create(collection, data)
       assert_type(data, [Hash, Array])
       # Single doc.
       if data.is_a?(Hash)
         data.merge!(Wgit::Model.common_insert_data)
         result = @@client[collection.to_sym].insert_one(data)
-        unless write_succeeded?(result)
-          raise "DB write (insert) failed"
-        end
+        raise 'DB write (insert) failed' unless write_succeeded?(result)
+
         result.n
       # Multiple docs.
       elsif data.is_a?(Array)
@@ -292,9 +314,8 @@ module Wgit
           data_hash.merge(Wgit::Model.common_insert_data)
         end
         result = @@client[collection.to_sym].insert_many(data)
-        unless write_succeeded?(result, data.length)
-          raise "DB write(s) failed"
-        end
+        raise 'DB write(s) failed' unless write_succeeded?(result, data.length)
+
         result.inserted_count
       else
         raise "data must be a Hash or an Array of Hash's"
@@ -302,52 +323,69 @@ module Wgit
     end
 
     # Retrieve Url or Document records from the DB.
+    #
+    # @param collection [Symbol] Either :urls or :documents.
+    # @param query [Hash] The query used during the retrieval.
+    # @param sort [Hash] The sort to use.
+    # @param projection [Hash] The projection to use.
+    # @param limit [Integer] The limit to use.
+    # @param skip [Integer] The skip to use.
+    # @return [Mongo::Object] The Mongo client find result.
     def retrieve(collection, query,
                  sort = {}, projection = {},
                  limit = 0, skip = 0)
       assert_type(query, Hash)
       @@client[collection.to_sym].find(query).projection(projection)
-                                   .skip(skip).limit(limit).sort(sort)
+                                 .skip(skip).limit(limit).sort(sort)
     end
 
     # Update a Url object in the DB.
+    #
+    # @param url [Wgit::Url] The Url to update.
+    # @return [Integer] The number of updated records.
     def update_url(url)
       assert_type(url, Url)
       selection = { url: url }
       url_hash = Wgit::Model.url(url).merge(Wgit::Model.common_update_data)
-      update = { "$set" => url_hash }
+      update = { '$set' => url_hash }
       _update(true, :urls, selection, update)
     end
 
     # Update a Document object in the DB.
+    #
+    # @param doc [Wgit::Document] The Document to update.
+    # @return [Integer] The number of updated records.
     def update_doc(doc)
       assert_type(doc, Document)
       selection = { url: doc.url }
       doc_hash = Wgit::Model.document(doc).merge(Wgit::Model.common_update_data)
-      update = { "$set" => doc_hash }
+      update = { '$set' => doc_hash }
       _update(true, :documents, selection, update)
     end
 
+    private
+
     # Update one or more Url or Document records in the DB.
     # NOTE: The Model.common_update_data should be merged in the calling
-    # method as the update param can be bespoke due to its nature.
+    # method as the update param can be bespoke, due to its nature.
     def _update(single, collection, selection, update)
       assert_arr_types([selection, update], Hash)
-      if single
-        result = @@client[collection.to_sym].update_one(selection, update)
-      else
-        result = @@client[collection.to_sym].update_many(selection, update)
-      end
-      raise "DB write (update) failed" unless write_succeeded?(result)
+      result = if single
+                 @@client[collection.to_sym].update_one(selection, update)
+               else
+                 @@client[collection.to_sym].update_many(selection, update)
+               end
+      raise 'DB write (update) failed' unless write_succeeded?(result)
+
       result.n
     end
 
-    alias :count :size
-    alias :length :size
-    alias :num_documents :num_docs
-    alias :document? :doc?
-    alias :insert_url :insert_urls
-    alias :insert_doc :insert_docs
-    alias :num_objects :num_records
+    alias count size
+    alias length size
+    alias num_documents num_docs
+    alias document? doc?
+    alias insert_url insert_urls
+    alias insert_doc insert_docs
+    alias num_objects num_records
   end
 end

data/lib/wgit/database/model.rb

@@ -1,46 +1,48 @@
+# frozen_string_literal: true
+
 require_relative '../utils'
 
 module Wgit
-
   # Module containing the database (DB) data model structure.
   module Model
-      
     # The data model for a Wgit::Url.
     #
     # @param url [Wgit::Url] The URL DB record.
     # @return [Hash] The URL model ready for DB insertion.
     def self.url(url)
-      raise "url must respond_to? to_h" unless url.respond_to?(:to_h)
+      raise 'url must respond_to? to_h' unless url.respond_to?(:to_h)
+
       model = url.to_h
       Wgit::Utils.remove_non_bson_types(model)
     end
-  
+
     # The data model for a Wgit::Document.
     #
     # @param doc [Wgit::Document] The Document DB record.
     # @return [Hash] The Document model ready for DB insertion.
     def self.document(doc)
-      raise "doc must respond_to? to_h" unless doc.respond_to?(:to_h)
+      raise 'doc must respond_to? to_h' unless doc.respond_to?(:to_h)
+
       model = doc.to_h(false)
       Wgit::Utils.remove_non_bson_types(model)
     end
-  
+
     # Default fields when inserting a record into the DB.
     #
     # @return [Hash] Containing common insertion fields for all models.
     def self.common_insert_data
       {
-        date_added:     Wgit::Utils.time_stamp,
-        date_modified:  Wgit::Utils.time_stamp,
+        date_added: Wgit::Utils.time_stamp,
+        date_modified: Wgit::Utils.time_stamp
       }
     end
-  
+
     # Default fields when updating a record in the DB.
     #
     # @return [Hash] Containing common update fields for all models.
     def self.common_update_data
       {
-        date_modified: Wgit::Utils.time_stamp,
+        date_modified: Wgit::Utils.time_stamp
       }
     end
   end

data/lib/wgit/document.rb

@@ -5,7 +5,6 @@ require 'nokogiri'
 require 'json'
 
 module Wgit
-
   # Class modeling a HTML web document. Also doubles as a search result when
   # loading Documents from the database.
   #
@@ -19,9 +18,9 @@ module Wgit
     # The HTML elements that make up the visible text on a page.
     # These elements are used to initialize the @text of the Document.
     # See the README.md for how to add to this Array dynamically.
-    @text_elements = [
-      :dd, :div, :dl, :dt, :figcaption, :figure, :hr, :li,
-      :main, :ol, :p, :pre, :span, :ul, :h1, :h2, :h3, :h4, :h5
+    @text_elements = %i[
+      dd div dl dt figcaption figure hr li
+      main ol p pre span ul h1 h2 h3 h4 h5
     ]
 
     class << self
@@ -58,14 +57,14 @@ module Wgit
     #   keys.
     # @param html [String] The crawled web page's HTML. This param is only
     #   required if url_or_obj is a String representing the web page's URL.
-    def initialize(url_or_obj, html = "")
+    def initialize(url_or_obj, html = '')
       # Init from URL String and HTML String.
       if url_or_obj.is_a?(String)
         url = url_or_obj
         assert_type(url, Wgit::Url)
 
         @url = url
-        @html = html ||= ""
+        @html = html || ''
         @doc = init_nokogiri
         @score = 0.0
 
@@ -73,9 +72,9 @@ module Wgit
 
         # Dynamically run the init_*_from_html methods.
         Document.private_instance_methods(false).each do |method|
-          if method.to_s.start_with?("init_") &&
-                method.to_s.end_with?("_from_html")
-            self.send(method)
+          if method.to_s.start_with?('init_') &&
+             method.to_s.end_with?('_from_html')
+            send(method)
           end
         end
       # Init from a Hash like object containing Strings as keys e.g. Mongo
@@ -84,18 +83,18 @@ module Wgit
         obj = url_or_obj
         assert_respond_to(obj, :fetch)
 
-        @url = Wgit::Url.new(obj.fetch("url")) # Should always be present.
-        @html = obj.fetch("html", "")
+        @url = Wgit::Url.new(obj.fetch('url')) # Should always be present.
+        @html = obj.fetch('html', '')
         @doc = init_nokogiri
-        @score = obj.fetch("score", 0.0)
+        @score = obj.fetch('score', 0.0)
 
         process_url_and_html
 
         # Dynamically run the init_*_from_object methods.
         Document.private_instance_methods(false).each do |method|
-          if method.to_s.start_with?("init_") &&
-                method.to_s.end_with?("_from_object")
-            self.send(method, obj)
+          if method.to_s.start_with?('init_') &&
+             method.to_s.end_with?('_from_object')
+            send(method, obj)
           end
         end
       end
@@ -108,7 +107,8 @@ module Wgit
     # @return [Boolean] True if @url and @html are equal, false if not.
     def ==(other_doc)
       return false unless other_doc.is_a? Wgit::Document
-      @url == other_doc.url and @html == other_doc.html
+
+      (@url == other_doc.url) && (@html == other_doc.html)
     end
 
     # Is a shortcut for calling Document#html[range].
@@ -148,7 +148,7 @@ module Wgit
         assert_type(link, Wgit::Url)
         raise "link must be relative: #{link}" unless link.is_relative?
 
-        if link.is_anchor? or link.is_query_string?
+        if link.is_anchor? || link.is_query_string?
           base_url = @base ? get_base.call : @url
           return base_url.without_anchor.without_query_string
         end
@@ -166,8 +166,8 @@ module Wgit
     #   returned Hash.
     # @return [Hash] Containing self's instance vars.
     def to_h(include_html = false)
-      ignore = include_html ? [] : ["@html"]
-      ignore << "@doc" # Always ignore "@doc"
+      ignore = include_html ? [] : ['@html']
+      ignore << '@doc' # Always ignore "@doc"
       Wgit::Utils.to_h(self, ignore)
     end
 
@@ -200,8 +200,9 @@ module Wgit
         # Else take the var's #length method return value.
         else
           next unless instance_variable_get(var).respond_to?(:length)
+
           hash[var[1..-1].to_sym] =
-                              instance_variable_get(var).send(:length)
+            instance_variable_get(var).send(:length)
         end
       end
       hash
@@ -219,6 +220,7 @@ module Wgit
     # @return [Boolean] True if @html is nil/empty, false otherwise.
     def empty?
       return true if @html.nil?
+
       @html.empty?
     end
 
@@ -252,12 +254,12 @@ module Wgit
     def internal_links
       return [] if @links.empty?
 
-      links = @links.
-        reject { |link| !link.is_relative?(host: @url.to_base) }.
-        map(&:without_base).
-        map do |link| # We map @url.to_host into / because it's a duplicate.
-          link.to_host == @url.to_host ? Wgit::Url.new('/') : link
-        end
+      links = @links
+              .select { |link| link.is_relative?(host: @url.to_base) }
+              .map(&:without_base)
+              .map do |link| # We map @url.to_host into / because it's a duplicate.
+        link.to_host == @url.to_host ? Wgit::Url.new('/') : link
+      end
 
       Wgit::Utils.process_arr(links)
     end
@@ -271,6 +273,7 @@ module Wgit
     def internal_full_links
       links = internal_links
       return [] if links.empty?
+
       links.map { |link| base_url(link: link).concat(link) }
     end
 
@@ -281,9 +284,9 @@ module Wgit
     def external_links
       return [] if @links.empty?
 
-      links = @links.
-        reject { |link| link.relative_link?(host: @url.to_base) }.
-        map(&:without_trailing_slash)
+      links = @links
+              .reject { |link| link.relative_link?(host: @url.to_base) }
+              .map(&:without_trailing_slash)
 
       Wgit::Utils.process_arr(links)
     end
@@ -304,24 +307,25 @@ module Wgit
     #   sentence.
     # @return [Array<String>] Representing the search results.
     def search(query, sentence_limit = 80)
-      raise "A search query must be provided" if query.empty?
-      raise "The sentence_limit value must be even" if sentence_limit.odd?
+      raise 'A search query must be provided' if query.empty?
+      raise 'The sentence_limit value must be even' if sentence_limit.odd?
 
       results = {}
       regex = Regexp.new(query, Regexp::IGNORECASE)
 
       @text.each do |sentence|
         hits = sentence.scan(regex).count
-        if hits > 0
-          sentence.strip!
-          index = sentence.index(regex)
-          Wgit::Utils.format_sentence_length(sentence, index, sentence_limit)
-          results[sentence] = hits
-        end
+        next unless hits > 0
+
+        sentence.strip!
+        index = sentence.index(regex)
+        Wgit::Utils.format_sentence_length(sentence, index, sentence_limit)
+        results[sentence] = hits
       end
 
       return [] if results.empty?
-      results = Hash[results.sort_by { |k, v| v }]
+
+      results = Hash[results.sort_by { |_k, v| v }]
       results.keys.reverse
     end
 
@@ -347,12 +351,13 @@ module Wgit
     #
     # @return [String] An xpath String to obtain a webpage's text elements.
     def self.text_elements_xpath
-      xpath = ""
+      xpath = ''
       return xpath if Wgit::Document.text_elements.empty?
-      el_xpath = "//%s/text()"
+
+      el_xpath = '//%s/text()'
       Wgit::Document.text_elements.each_with_index do |el, i|
-        xpath += " | " unless i == 0
-        xpath += el_xpath % [el]
+        xpath += ' | ' unless i == 0
+        xpath += format(el_xpath, el)
       end
       xpath
     end
@@ -429,35 +434,42 @@ module Wgit
       false
     end
 
-  private
+    protected
 
-    # Initializes the nokogiri object using @html, which must be already set.
+    # Initializes the nokogiri object using @html, which cannot be nil.
+    # Override this method to custom configure the Nokogiri object returned.
+    # Gets called from Wgit::Document.new.
+    #
+    # @return [Nokogiri::HTML] The initialised Nokogiri HTML object.
     def init_nokogiri
-      raise "@html must be set" unless @html
+      raise '@html must be set' unless @html
+
       Nokogiri::HTML(@html) do |config|
         # TODO: Remove #'s below when crawling in production.
-        #config.options = Nokogiri::XML::ParseOptions::STRICT |
+        # config.options = Nokogiri::XML::ParseOptions::STRICT |
         #                 Nokogiri::XML::ParseOptions::NONET
       end
     end
 
-    # Ensure the @url and @html Strings are correctly encoded etc.
-    def process_url_and_html
-      @url = Wgit::Utils.process_str(@url)
-      @html = Wgit::Utils.process_str(@html)
-    end
-
-    # Returns an object/value from this Document's @html using the provided
-    # xpath param.
-    # singleton ? results.first (single Object) : results (Array)
-    # text_content_only ? result.content (String) : result (nokogiri Object)
-    # A block can be used to set the final value before it is returned.
-    # Return nil from the block if you don't want to override the value.
+    # Returns a value/object from this Document's @html using the given xpath
+    # parameter.
+    #
+    # @param xpath [String] Used to find the value/object in @html.
+    # @param singleton [Boolean] singleton ? results.first (single Nokogiri
+    #   Object) : results (Array).
+    # @param text_content_only [Boolean] text_content_only ? result.content
+    #   (String) : result (Nokogiri Object).
+    # @yield [String/Object, Symbol] Given the value before it's set as an
+    #   instance variable so that you can inspect/alter the value if desired.
+    #   Return nil from the block if you don't want to override the value. Also
+    #   given the source which is always :html.
+    # @return [String, Object] The value found in the html or the default value
+    #   (singleton ? nil : []).
     def find_in_html(xpath, singleton: true, text_content_only: true)
       xpath = xpath.call if xpath.respond_to?(:call)
       results = @doc.xpath(xpath)
 
-      if results and not results.empty?
+      if results && !results.empty?
         result =  if singleton
                     text_content_only ? results.first.content : results.first
                   else
@@ -477,10 +489,17 @@ module Wgit
       result
     end
 
-    # Finds a value in the obj using the key.
-    # singleton is used to set the value if not found in obj.
-    # A block can be used to set the final value before it is returned.
-    # Return nil from the block if you don't want to override the value.
+    # Returns a value from the obj using the given key via obj#fetch.
+    #
+    # @param obj [Object#fetch] The object containing the key/value.
+    # @param key [String] Used to find the value in the obj.
+    # @param singleton [Boolean] True if a single value, false otherwise.
+    # @yield [String/Object, Symbol] Given the value before it's set as an
+    #   instance variable so that you can inspect/alter the value if desired.
+    #   Return nil from the block if you don't want to override the value. Also
+    #   given the source which is always :object.
+    # @return [String, Object] The value found in the obj or the default value
+    #   (singleton ? nil : []).
     def find_in_object(obj, key, singleton: true)
       assert_respond_to(obj, :fetch)
 
@@ -496,14 +515,17 @@ module Wgit
       result
     end
 
+    private
+
     # Initialises an instance variable and defines a getter method for it.
+    #
     # @param var [Symbol] The name of the variable to be initialized.
     # @param value [Object] The newly initialized variable's value.
     # @return [Symbol] The name of the newly created getter method.
     def init_var(var, value)
       # instance_var_name starts with @, var_name doesn't.
       var = var.to_s
-      var_name = (var.start_with?("@") ? var[1..-1] : var).to_sym
+      var_name = (var.start_with?('@') ? var[1..-1] : var).to_sym
       instance_var_name = "@#{var_name}".to_sym
 
       instance_variable_set(instance_var_name, value)
@@ -513,13 +535,19 @@ module Wgit
       end
     end
 
-    alias :relative_links :internal_links
-    alias :relative_urls :internal_links
-    alias :relative_full_links :internal_full_links
-    alias :relative_full_urls :internal_full_links
-    alias :internal_absolute_links :internal_full_links
-    alias :relative_absolute_links :internal_full_links
-    alias :relative_absolute_urls :internal_full_links
-    alias :external_urls :external_links
+    # Ensure the @url and @html Strings are correctly encoded etc.
+    def process_url_and_html
+      @url = Wgit::Utils.process_str(@url)
+      @html = Wgit::Utils.process_str(@html)
+    end
+
+    alias relative_links internal_links
+    alias relative_urls internal_links
+    alias relative_full_links internal_full_links
+    alias relative_full_urls internal_full_links
+    alias internal_absolute_links internal_full_links
+    alias relative_absolute_links internal_full_links
+    alias relative_absolute_urls internal_full_links
+    alias external_urls external_links
   end
 end