meta-tags 2.13.0 → 2.19.0

data/gemfiles/rails_7.1.gemfile.lock

@@ -0,0 +1,231 @@
+PATH
+  remote: ..
+  specs:
+    meta-tags (2.19.0)
+      actionpack (>= 3.2.0, < 7.2)
+
+GEM
+  remote: https://rubygems.org/
+  specs:
+    actionpack (7.1.0)
+      actionview (= 7.1.0)
+      activesupport (= 7.1.0)
+      nokogiri (>= 1.8.5)
+      rack (>= 2.2.4)
+      rack-session (>= 1.0.1)
+      rack-test (>= 0.6.3)
+      rails-dom-testing (~> 2.2)
+      rails-html-sanitizer (~> 1.6)
+    actionview (7.1.0)
+      activesupport (= 7.1.0)
+      builder (~> 3.1)
+      erubi (~> 1.11)
+      rails-dom-testing (~> 2.2)
+      rails-html-sanitizer (~> 1.6)
+    activesupport (7.1.0)
+      base64
+      bigdecimal
+      concurrent-ruby (~> 1.0, >= 1.0.2)
+      connection_pool (>= 2.2.5)
+      drb
+      i18n (>= 1.6, < 2)
+      minitest (>= 5.1)
+      mutex_m
+      tzinfo (~> 2.0)
+    appraisal (2.5.0)
+      bundler
+      rake
+      thor (>= 0.14.0)
+    ast (2.4.2)
+    base64 (0.1.1)
+    bigdecimal (3.1.4)
+    builder (3.2.4)
+    concurrent-ruby (1.2.2)
+    connection_pool (2.4.1)
+    crass (1.0.6)
+    csv (3.2.7)
+    diff-lcs (1.5.0)
+    docile (1.4.0)
+    drb (2.1.1)
+      ruby2_keywords
+    erubi (1.12.0)
+    ffi (1.16.3)
+    fileutils (1.7.1)
+    i18n (1.14.1)
+      concurrent-ruby (~> 1.0)
+    io-console (0.6.0)
+    irb (1.8.1)
+      rdoc
+      reline (>= 0.3.8)
+    json (2.6.3)
+    language_server-protocol (3.17.0.3)
+    lint_roller (1.1.0)
+    listen (3.8.0)
+      rb-fsevent (~> 0.10, >= 0.10.3)
+      rb-inotify (~> 0.9, >= 0.9.10)
+    logger (1.5.3)
+    loofah (2.21.3)
+      crass (~> 1.0.2)
+      nokogiri (>= 1.12.0)
+    minitest (5.20.0)
+    mutex_m (0.1.2)
+    nokogiri (1.15.4-arm64-darwin)
+      racc (~> 1.4)
+    parallel (1.23.0)
+    parser (3.2.2.4)
+      ast (~> 2.4.1)
+      racc
+    psych (5.1.0)
+      stringio
+    racc (1.7.1)
+    rack (3.0.8)
+    rack-session (2.0.0)
+      rack (>= 3.0.0)
+    rack-test (2.1.0)
+      rack (>= 1.3)
+    rackup (2.1.0)
+      rack (>= 3)
+      webrick (~> 1.8)
+    rails-dom-testing (2.2.0)
+      activesupport (>= 5.0.0)
+      minitest
+      nokogiri (>= 1.6)
+    rails-html-sanitizer (1.6.0)
+      loofah (~> 2.21)
+      nokogiri (~> 1.14)
+    railties (7.1.0)
+      actionpack (= 7.1.0)
+      activesupport (= 7.1.0)
+      irb
+      rackup (>= 1.0.0)
+      rake (>= 12.2)
+      thor (~> 1.0, >= 1.2.2)
+      zeitwerk (~> 2.6)
+    rainbow (3.1.1)
+    rake (13.0.6)
+    rb-fsevent (0.11.2)
+    rb-inotify (0.10.1)
+      ffi (~> 1.0)
+    rbs (3.2.2)
+    rdoc (6.5.0)
+      psych (>= 4.0.0)
+    regexp_parser (2.8.1)
+    reline (0.3.9)
+      io-console (~> 0.5)
+    rexml (3.2.6)
+    rspec (3.12.0)
+      rspec-core (~> 3.12.0)
+      rspec-expectations (~> 3.12.0)
+      rspec-mocks (~> 3.12.0)
+    rspec-core (3.12.2)
+      rspec-support (~> 3.12.0)
+    rspec-expectations (3.12.3)
+      diff-lcs (>= 1.2.0, < 2.0)
+      rspec-support (~> 3.12.0)
+    rspec-html-matchers (0.10.0)
+      nokogiri (~> 1)
+      rspec (>= 3.0.0.a)
+    rspec-mocks (3.12.6)
+      diff-lcs (>= 1.2.0, < 2.0)
+      rspec-support (~> 3.12.0)
+    rspec-support (3.12.1)
+    rspec_junit_formatter (0.6.0)
+      rspec-core (>= 2, < 4, != 2.12.0)
+    rubocop (1.56.4)
+      base64 (~> 0.1.1)
+      json (~> 2.3)
+      language_server-protocol (>= 3.17.0)
+      parallel (~> 1.10)
+      parser (>= 3.2.2.3)
+      rainbow (>= 2.2.2, < 4.0)
+      regexp_parser (>= 1.8, < 3.0)
+      rexml (>= 3.2.5, < 4.0)
+      rubocop-ast (>= 1.28.1, < 2.0)
+      ruby-progressbar (~> 1.7)
+      unicode-display_width (>= 2.4.0, < 3.0)
+    rubocop-ast (1.29.0)
+      parser (>= 3.2.1.0)
+    rubocop-capybara (2.19.0)
+      rubocop (~> 1.41)
+    rubocop-factory_bot (2.24.0)
+      rubocop (~> 1.33)
+    rubocop-performance (1.19.1)
+      rubocop (>= 1.7.0, < 2.0)
+      rubocop-ast (>= 0.4.0)
+    rubocop-rails (2.20.2)
+      activesupport (>= 4.2.0)
+      rack (>= 1.1)
+      rubocop (>= 1.33.0, < 2.0)
+    rubocop-rake (0.6.0)
+      rubocop (~> 1.0)
+    rubocop-rspec (2.23.2)
+      rubocop (~> 1.33)
+      rubocop-capybara (~> 2.17)
+      rubocop-factory_bot (~> 2.22)
+    ruby-progressbar (1.13.0)
+    ruby2_keywords (0.0.5)
+    securerandom (0.2.2)
+    simplecov (0.22.0)
+      docile (~> 1.1)
+      simplecov-html (~> 0.11)
+      simplecov_json_formatter (~> 0.1)
+    simplecov-html (0.12.3)
+    simplecov_json_formatter (0.1.4)
+    standard (1.31.1)
+      language_server-protocol (~> 3.17.0.2)
+      lint_roller (~> 1.0)
+      rubocop (~> 1.56.2)
+      standard-custom (~> 1.0.0)
+      standard-performance (~> 1.2)
+    standard-custom (1.0.2)
+      lint_roller (~> 1.0)
+      rubocop (~> 1.50)
+    standard-performance (1.2.0)
+      lint_roller (~> 1.1)
+      rubocop-performance (~> 1.19.0)
+    steep (1.5.3)
+      activesupport (>= 5.1)
+      concurrent-ruby (>= 1.1.10)
+      csv (>= 3.0.9)
+      fileutils (>= 1.1.0)
+      json (>= 2.1.0)
+      language_server-protocol (>= 3.15, < 4.0)
+      listen (~> 3.0)
+      logger (>= 1.3.0)
+      parser (>= 3.1)
+      rainbow (>= 2.2.2, < 4.0)
+      rbs (>= 3.1.0)
+      securerandom (>= 0.1)
+      strscan (>= 1.0.0)
+      terminal-table (>= 2, < 4)
+    stringio (3.0.8)
+    strscan (3.0.6)
+    terminal-table (3.0.2)
+      unicode-display_width (>= 1.1.1, < 3)
+    thor (1.2.2)
+    tzinfo (2.0.6)
+      concurrent-ruby (~> 1.0)
+    unicode-display_width (2.5.0)
+    webrick (1.8.1)
+    zeitwerk (2.6.12)
+
+PLATFORMS
+  arm64-darwin
+
+DEPENDENCIES
+  appraisal (~> 2.5.0)
+  meta-tags!
+  railties (= 7.1.0)
+  rake (~> 13.0)
+  rspec (~> 3.12.0)
+  rspec-html-matchers (~> 0.10.0)
+  rspec_junit_formatter (~> 0.6.0)
+  rubocop-rails (~> 2.20.2)
+  rubocop-rake (~> 0.6.0)
+  rubocop-rspec (~> 2.23.0)
+  simplecov (~> 0.22.0)
+  standard (~> 1.29)
+  steep (~> 1.5.2)
+
+BUNDLED WITH
+   2.4.10

data/lib/generators/meta_tags/install_generator.rb

@@ -4,7 +4,7 @@ module MetaTags
   module Generators
     class InstallGenerator < Rails::Generators::Base
       desc "Copy MetaTags default files"
-      source_root File.expand_path('templates', __dir__)
+      source_root File.expand_path("templates", __dir__)
 
       def copy_config
         template "config/initializers/meta_tags.rb"

data/lib/meta-tags.rb

@@ -1,3 +1,3 @@
 # frozen_string_literal: true
 
-require 'meta_tags'
+require "meta_tags"

data/lib/meta_tags/configuration.rb

@@ -33,6 +33,12 @@ module MetaTags
     # - an array of strings or symbols representing their names or name-prefixes.
     attr_reader :property_tags
 
+    # Configure whenever Meta-Tags should skip canonicals on pages with noindex: true
+    # "shouldn't mix noindex & rel=canonical comes from: they're very contradictory pieces of information for us."
+    # - John Mueller (Webmaster Trends Analyst at Google)
+    # https://www.reddit.com/r/TechSEO/comments/8yahdr/2_questions_about_the_canonical_tag/e2dey9i/
+    attr_accessor :skip_canonical_links_on_noindex
+
     # Initializes a new instance of Configuration class.
     def initialize
       reset_defaults!
@@ -41,25 +47,25 @@ module MetaTags
     def default_property_tags
       [
         # App Link metadata https://developers.facebook.com/docs/applinks/metadata-reference
-        'al',
+        "al",
         # Open Graph Markup https://developers.facebook.com/docs/sharing/webmasters#markup
-        'fb',
-        'og',
+        "fb",
+        "og",
         # Facebook OpenGraph Object Types https://developers.facebook.com/docs/reference/opengraph
         # Note that these tags are used in a regex, so including e.g. 'restaurant' will affect
         # 'restaurant:category', 'restaurant:price_rating', and anything else under that namespace.
-        'article',
-        'book',
-        'books',
-        'business',
-        'fitness',
-        'game',
-        'music',
-        'place',
-        'product',
-        'profile',
-        'restaurant',
-        'video',
+        "article",
+        "book",
+        "books",
+        "business",
+        "fitness",
+        "game",
+        "music",
+        "place",
+        "product",
+        "profile",
+        "restaurant",
+        "video"
       ].freeze
     end
 
@@ -72,11 +78,12 @@ module MetaTags
       @truncate_site_title_first = false
       @description_limit = 300
       @keywords_limit = 255
-      @keywords_separator = ', '
+      @keywords_separator = ", "
       @keywords_lowercase = true
       @property_tags = default_property_tags.dup
       @open_meta_tags = true
       @minify_output = false
+      @skip_canonical_links_on_noindex = false
     end
   end
 end

data/lib/meta_tags/controller_helper.rb

@@ -15,9 +15,9 @@ module MetaTags
     # Processes the <tt>@page_title</tt>, <tt>@page_keywords</tt>, and
     # <tt>@page_description</tt> instance variables and calls +render+.
     def render(*args, &block)
-      meta_tags[:title]       = @page_title       if @page_title
-      meta_tags[:keywords]    = @page_keywords    if @page_keywords
-      meta_tags[:description] = @page_description if @page_description
+      meta_tags[:title] = @page_title if defined?(@page_title) && @page_title
+      meta_tags[:keywords] = @page_keywords if defined?(@page_keywords) && @page_keywords
+      meta_tags[:description] = @page_description if defined?(@page_description) && @page_description
 
       super
     end
@@ -25,7 +25,7 @@ module MetaTags
     # Set meta tags for the page.
     #
     # See <tt>MetaTags::ViewHelper#set_meta_tags</tt> for details.
-    def set_meta_tags(meta_tags) # rubocop:disable Naming/AccessorMethodName
+    def set_meta_tags(meta_tags)
       self.meta_tags.update(meta_tags)
     end
     protected :set_meta_tags

data/lib/meta_tags/meta_tags_collection.rb

@@ -9,7 +9,7 @@ module MetaTags
     # Initializes a new instance of MetaTagsCollection.
     #
     def initialize
-      @meta_tags = HashWithIndifferentAccess.new
+      @meta_tags = ActiveSupport::HashWithIndifferentAccess.new
     end
 
     # Returns meta tag value by name.
@@ -38,7 +38,13 @@ module MetaTags
     # @return [Hash] result of the merge.
     #
     def update(object = {})
-      meta_tags = object.respond_to?(:to_meta_tags) ? object.to_meta_tags : object
+      meta_tags = if object.respond_to?(:to_meta_tags)
+        # @type var object: _MetaTagish & Object
+        object.to_meta_tags
+      else
+        # @type var object: Hash[String | Symbol, untyped]
+        object
+      end
       @meta_tags.deep_merge! normalize_open_graph(meta_tags)
     end
 
@@ -64,14 +70,16 @@ module MetaTags
       with_defaults(defaults) { extract_full_title }
     end
 
-    # Constructs the title without site title (for normalized parameters).
+    # Constructs the title without site title (for normalized parameters). When title is empty,
+    # use the site title instead.
     #
     # @return [String] page title.
     #
     def page_title(defaults = {})
       old_site = @meta_tags[:site]
       @meta_tags[:site] = nil
-      with_defaults(defaults) { extract_full_title }
+      full_title = with_defaults(defaults) { extract_full_title }
+      full_title.presence || old_site || ""
     ensure
       @meta_tags[:site] = old_site
     end
@@ -98,10 +106,10 @@ module MetaTags
     # @return [String] page title.
     #
     def extract_full_title
-      site_title = extract(:site) || ''
-      title      = extract_title || []
-      separator  = extract_separator
-      reverse    = extract(:reverse) == true
+      site_title = extract(:site) || ""
+      title = extract_title
+      separator = extract_separator
+      reverse = extract(:reverse) == true
 
       TextNormalizer.normalize_title(site_title, title, separator, reverse)
     end
@@ -112,8 +120,9 @@ module MetaTags
     #
     def extract_title
       title = extract(:title).presence
-      return unless title
+      return [] unless title
 
+      # @type var title: Array[String]
       title = Array(title)
       return title.map(&:downcase) if extract(:lowercase) == true
 
@@ -127,15 +136,15 @@ module MetaTags
     def extract_separator
       if meta_tags[:separator] == false
         # Special case: if separator is hidden, do not display suffix/prefix
-        prefix = separator = suffix = ''
+        prefix = separator = suffix = ""
       else
-        prefix    = extract_separator_section(:prefix, ' ')
-        separator = extract_separator_section(:separator, '|')
-        suffix    = extract_separator_section(:suffix, ' ')
+        prefix = extract_separator_section(:prefix, " ")
+        separator = extract_separator_section(:separator, "|")
+        suffix = extract_separator_section(:suffix, " ")
       end
       delete(:separator, :prefix, :suffix)
 
-      TextNormalizer.safe_join([prefix, separator, suffix], '')
+      TextNormalizer.safe_join([prefix, separator, suffix], "")
     end
 
     # Extracts noindex settings as a Hash mapping noindex tag name to value.
@@ -150,12 +159,12 @@ module MetaTags
         [:noindex, :index],
         # follow has higher priority than nofollow
         [:follow, :nofollow],
-        :noarchive,
+        :noarchive
       ].each do |attributes|
         calculate_robots_attributes(result, attributes)
       end
 
-      result.transform_values { |v| v.join(', ') }
+      result.transform_values { |v| v.join(", ") }
     end
 
     protected
@@ -163,10 +172,10 @@ module MetaTags
     # Converts input hash to HashWithIndifferentAccess and renames :open_graph to :og.
     #
     # @param [Hash] meta_tags list of meta tags.
-    # @return [HashWithIndifferentAccess] normalized meta tags list.
+    # @return [ActiveSupport::HashWithIndifferentAccess] normalized meta tags list.
     #
     def normalize_open_graph(meta_tags)
-      meta_tags = meta_tags.kind_of?(HashWithIndifferentAccess) ? meta_tags.dup : meta_tags.with_indifferent_access
+      meta_tags = meta_tags.with_indifferent_access
       meta_tags[:og] = meta_tags.delete(:open_graph) if meta_tags.key?(:open_graph)
       meta_tags
     end
@@ -179,7 +188,7 @@ module MetaTags
     # @return [String] separator segment value.
     #
     def extract_separator_section(name, default)
-      meta_tags[name] == false ? '' : (meta_tags[name] || default)
+      (meta_tags[name] == false) ? "" : (meta_tags[name] || default)
     end
 
     # Extracts robots attribute (noindex, nofollow, etc) name and value.
@@ -188,11 +197,11 @@ module MetaTags
     # @return [Array<String>] pair of noindex attribute name and value.
     #
     def extract_robots_attribute(name)
-      noindex       = extract(name)
-      noindex_name  = noindex.kind_of?(String) || noindex.kind_of?(Array) ? noindex : 'robots'
+      noindex = extract(name)
+      noindex_name = (noindex.is_a?(String) || noindex.is_a?(Array)) ? noindex : "robots"
       noindex_value = noindex ? name.to_s : nil
 
-      [ noindex_name, noindex_value ]
+      [noindex_name, noindex_value]
     end
 
     def calculate_robots_attributes(result, attributes)

data/lib/meta_tags/railtie.rb

@@ -2,15 +2,15 @@
 
 module MetaTags
   class Railtie < Rails::Railtie
-    initializer 'meta_tags.setup_action_controller' do
+    initializer "meta_tags.setup_action_controller" do
       ActiveSupport.on_load :action_controller do
-        ActionController::Base.include MetaTags::ControllerHelper
+        include MetaTags::ControllerHelper
       end
     end
 
-    initializer 'meta_tags.setup_action_view' do
+    initializer "meta_tags.setup_action_view" do
       ActiveSupport.on_load :action_view do
-        ActionView::Base.include MetaTags::ViewHelper
+        include MetaTags::ViewHelper
       end
     end
   end

data/lib/meta_tags/renderer.rb

@@ -26,6 +26,7 @@ module MetaTags
       render_with_normalization(tags, :description)
       render_with_normalization(tags, :keywords)
       render_refresh(tags)
+      render_canonical_link(tags)
       render_noindex(tags)
       render_alternate(tags)
       render_open_search(tags)
@@ -70,12 +71,12 @@ module MetaTags
       return unless icon
 
       # String? Value is an href
-      icon = [{ href: icon }] if icon.kind_of?(String)
+      icon = [{href: icon}] if icon.is_a?(String)
       # Hash? Single icon instead of a list of icons
-      icon = [icon] if icon.kind_of?(Hash)
+      icon = [icon] if icon.is_a?(Hash)
 
       icon.each do |icon_params|
-        icon_params = { rel: 'icon', type: 'image/x-icon' }.with_indifferent_access.merge(icon_params)
+        icon_params = {rel: "icon", type: "image/x-icon"}.with_indifferent_access.merge(icon_params)
         tags << Tag.new(:link, icon_params)
       end
     end
@@ -108,7 +109,7 @@ module MetaTags
     #
     def render_refresh(tags)
       refresh = meta_tags.extract(:refresh)
-      tags << Tag.new(:meta, 'http-equiv' => 'refresh', content: refresh.to_s) if refresh.present?
+      tags << Tag.new(:meta, "http-equiv" => "refresh", :content => refresh.to_s) if refresh.present?
     end
 
     # Renders alternate link tags.
@@ -119,13 +120,13 @@ module MetaTags
       alternate = meta_tags.extract(:alternate)
       return unless alternate
 
-      if alternate.kind_of?(Hash)
+      if alternate.is_a?(Hash)
         alternate.each do |hreflang, href|
-          tags << Tag.new(:link, rel: 'alternate', href: href, hreflang: hreflang) if href.present?
+          tags << Tag.new(:link, rel: "alternate", href: href, hreflang: hreflang) if href.present?
         end
-      elsif alternate.kind_of?(Array)
+      elsif alternate.is_a?(Array)
         alternate.each do |link_params|
-          tags << Tag.new(:link, { rel: 'alternate' }.with_indifferent_access.merge(link_params))
+          tags << Tag.new(:link, {rel: "alternate"}.with_indifferent_access.merge(link_params))
         end
       end
     end
@@ -142,7 +143,7 @@ module MetaTags
       title = open_search[:title]
 
       type = "application/opensearchdescription+xml"
-      tags << Tag.new(:link, rel: 'search', type: type, href: href, title: title) if href.present?
+      tags << Tag.new(:link, rel: "search", type: type, href: href, title: title) if href.present?
     end
 
     # Renders links.
@@ -150,7 +151,7 @@ module MetaTags
     # @param [Array<Tag>] tags a buffer object to store tag in.
     #
     def render_links(tags)
-      [ :amphtml, :canonical, :prev, :next, :image_src, :manifest ].each do |tag_name|
+      [:amphtml, :prev, :next, :image_src, :manifest].each do |tag_name|
         href = meta_tags.extract(tag_name)
         if href.present?
           @normalized_meta_tags[tag_name] = href
@@ -159,6 +160,19 @@ module MetaTags
       end
     end
 
+    # Renders canonical link
+    #
+    # @param [Array<Tag>] tags a buffer object to store tag in.
+    #
+    def render_canonical_link(tags)
+      href = meta_tags.extract(:canonical) # extract, so its not used anywhere else
+      return if MetaTags.config.skip_canonical_links_on_noindex && meta_tags[:noindex]
+      return if href.blank?
+
+      @normalized_meta_tags[:canonical] = href
+      tags << Tag.new(:link, rel: :canonical, href: href)
+    end
+
     # Renders complex hash objects.
     #
     # @param [Array<Tag>] tags a buffer object to store tag in.
@@ -175,7 +189,7 @@ module MetaTags
     #
     def render_hash(tags, key, **opts)
       data = meta_tags.meta_tags[key]
-      return unless data.kind_of?(Hash)
+      return unless data.is_a?(Hash)
 
       process_hash(tags, key, data, **opts)
       meta_tags.extract(key)
@@ -188,7 +202,7 @@ module MetaTags
     def render_custom(tags)
       meta_tags.meta_tags.each do |name, data|
         Array(data).each do |val|
-          tags << Tag.new(:meta, configured_name_key(name) => name, content: val)
+          tags << Tag.new(:meta, configured_name_key(name) => name, :content => val)
         end
         meta_tags.extract(name)
       end
@@ -201,16 +215,17 @@ module MetaTags
     # @param [Hash, Array, String, Symbol] content text content or a symbol reference to
     # top-level meta tag.
     #
-    def process_tree(tags, property, content, **opts)
+    def process_tree(tags, property, content, itemprop: nil, **opts)
       method = case content
-               when Hash
-                 :process_hash
-               when Array
-                 :process_array
-               else
-                 :render_tag
-               end
-      __send__(method, tags, property, content, **opts)
+      when Hash
+        :process_hash
+      when Array
+        :process_array
+      else
+        iprop = itemprop
+        :render_tag
+      end
+      __send__(method, tags, property, content, itemprop: iprop, **opts)
     end
 
     # Recursive method to process a hash with meta tags
@@ -220,10 +235,21 @@ module MetaTags
     # @param [Hash] content nested meta tag attributes.
     #
     def process_hash(tags, property, content, **opts)
+      itemprop = content.delete(:itemprop)
       content.each do |key, value|
-        key = key.to_s == '_' ? property : "#{property}:#{key}"
-        value = normalized_meta_tags[value] if value.kind_of?(Symbol)
-        process_tree(tags, key, value, **opts)
+        if key.to_s == "_"
+          iprop = itemprop
+          key = property
+        else
+          key = "#{property}:#{key}"
+        end
+
+        normalized_value = if value.is_a?(Symbol)
+          normalized_meta_tags[value]
+        else
+          value
+        end
+        process_tree(tags, key, normalized_value, **opts.merge(itemprop: iprop))
       end
     end
 
@@ -243,17 +269,18 @@ module MetaTags
     # @param [String, Symbol] name a Hash or a String to render as meta tag.
     # @param [String, Symbol] value text content or a symbol reference to
     # top-level meta tag.
+    # @param [String, Symbol] itemprop value of the itemprop attribute.
     #
-    def render_tag(tags, name, value, name_key: nil, value_key: :content)
+    def render_tag(tags, name, value, itemprop: nil)
       name_key ||= configured_name_key(name)
-      tags << Tag.new(:meta, name_key => name.to_s, value_key => value) if value.present?
+      tags << Tag.new(:meta, name_key => name.to_s, :content => value, :itemprop => itemprop) if value.present?
     end
 
     # Returns meta tag property name for a give meta tag based on the
     # configured list of property tags in MetaTags::Configuration#property_tags.
     #
-    # @param [String, Symbol] meta tag key.
-    # @return [String] meta tag attribute name ("property" or "name").
+    # @param [String, Symbol] name tag key.
+    # @return [Symbol] meta tag attribute name (:property or :name).
     #
     def configured_name_key(name)
       is_property_tag = MetaTags.config.property_tags.any? do |tag_name|

data/lib/meta_tags/tag.rb

@@ -11,7 +11,7 @@ module MetaTags
     # @param [Hash] attributes list of HTML tag attributes
     #
     def initialize(name, attributes = {})
-      @name = name
+      @name = name.to_s
       @attributes = attributes
     end