bridgetown-seo-tag 4.0.1 → 6.0.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: abb7aa6ad690aea0cc55f5e18a9dd7be9c1461f4b6d7b57977abfc568ae0f09d
-  data.tar.gz: 27388ed583fc7d3cce45d5eab085bf87d5a30320192a844a97541bb71fbf28da
+  metadata.gz: f52feeb06f0b0cc5bc8a893251e22db438727808592d79027c2069a2c36c05ef
+  data.tar.gz: 68a1b3dce65f5b959de19d2da22611be530cc4ab9fed9bc366e7d052eadbd2ba
 SHA512:
-  metadata.gz: faa21316c33fd7e2bf19228dcc848b147b5b40ba03f1d12da9229b9b7f665fd9f0b5279198fa43644a67fe67374d2554d61cdb5def9ee12a5d1ad3b8a2f6d420
-  data.tar.gz: e95daef1af9a8447ac7248fc39b6b9246910150995073b4f8b61160cbf60ac07e48fd3cf721be13310e28eb3cb53d4a1624ea323a0e5d191c6221a509b3758b9
+  metadata.gz: 2251e9c40bb873fd729e7e74b7f8124a911d85a79ea7f0d9ebd0ad1088cfbee3ebba23c6149a648a37bec38208b8318999e0afc3f0e722470d8540f515204b5c
+  data.tar.gz: c1620483b79ac3cc063fd595d42b64f5dc4faa96cbcc78c59243d6d62577f6c8537c59da83d093c58b15bca90987a7043f35b69d01bf4a141ce7d75ee54ad462

data/.rubocop.yml

@@ -5,7 +5,7 @@ inherit_gem:
   rubocop-bridgetown: .rubocop.yml
 
 AllCops:
-  TargetRubyVersion: 2.5
+  TargetRubyVersion: 2.7
   Exclude:
     - vendor/**/*
 
@@ -17,3 +17,10 @@ Layout/LineLength:
 Metrics/BlockLength:
   Exclude:
     - spec/**/*
+
+Lint/MissingSuper:
+  Enabled: false
+
+Performance/CollectionLiteralInLoop:
+  Exclude:
+    - spec/**/*

data/CHANGELOG.md

@@ -1,31 +1,44 @@
 # Changelog
 
+## 6.0.0 / 2023-01-08
+
+- Fix bug where prev/next rel links for paginated pages were missing
+- Upgrade to initializers system in Bridgetown 1.2
+- Add support for image alt metadata
+- Add fallback for og:type metadata 
+
+## 5.0.0 / 2021-10-17
+
+- Change "Site Title | Site Tagline" format to "Site Title: Site Tagline"
+  - also handle Bridgetown 1.0 templates where the homepage title defaults to `Index`
+  - upgrade rubocop-bridgetown gem to 0.3
+
 ## 4.0.1 / 2021-06-04
 
-* Fix bug where resources' relative URLs weren't included properly
+- Fix bug where resources' relative URLs weren't included properly
 
 ## 4.0.0 / 2021-04-17
 
-* New release with helper to support Ruby templates like ERB
+- New release with helper to support Ruby templates like ERB
 
 ## 3.0.5  / 2020-06-18
 
-* Final release
+- Final release
 
 ## 3.0.5.beta1 / 2020-05-31
 
-* Fix bugs due to Bridgetown 0.15 switch to `render` tag.
-* Switch to using Rubocop Bridgetown gem.
+- Fix bugs due to Bridgetown 0.15 switch to `render` tag.
+- Switch to using Rubocop Bridgetown gem.
 
 ## 3.0.4 / 2020-05-01
 
-Update to require a minimum Ruby version of 2.5.
+- Update to require a minimum Ruby version of 2.5.
 
 ## 3.0.3 / 2020-04-19
 
-Allow `site.metadata.twitter` data if present. Look for `page.subtitle` if
+- Allow `site.metadata.twitter` data if present. Look for `page.subtitle` if
 `page.description` isn't present.
 
 ## 3.0.0 / 2020-04-14
 
-Use Bridgetown gem and rename to bridgetown-seo-tag.
+- Use Bridgetown gem and rename to bridgetown-seo-tag.

data/bridgetown-seo-tag.gemspec

@@ -11,16 +11,17 @@ Gem::Specification.new do |spec|
   spec.homepage      = "https://github.com/bridgetownrb/bridgetown-seo-tag"
   spec.license       = "MIT"
 
-  spec.required_ruby_version = ">= 2.5.0"
+  spec.required_ruby_version = ">= 2.7.0"
 
   spec.files         = `git ls-files -z`.split("\x0").reject { |f| f.match(%r!^(test|script|spec|features)/!) }
   spec.require_paths = ["lib"]
 
-  spec.add_dependency "bridgetown", ">= 0.20.0", "< 2.0"
+  spec.add_dependency "bridgetown", ">= 1.2.0.beta5", "< 2.0"
 
   spec.add_development_dependency "bundler", ">= 1.15"
   spec.add_development_dependency "html-proofer", "~> 3.7"
-  spec.add_development_dependency "rake", "~> 12.0"
+  spec.add_development_dependency "rake", "~> 13.0"
   spec.add_development_dependency "rspec", "~> 3.5"
-  spec.add_development_dependency "rubocop-bridgetown", "~> 0.2"
+  spec.add_development_dependency "rubocop-bridgetown", "~> 0.3"
+  spec.metadata["rubygems_mfa_required"] = "true"
 end

data/lib/bridgetown-seo-tag/author_drop.rb

@@ -41,8 +41,7 @@ module Bridgetown
 
       private
 
-      attr_reader :page
-      attr_reader :site
+      attr_reader :page, :site
 
       # Finds the page author in the page.author, page.authors, or site.author
       #
@@ -62,10 +61,9 @@ module Bridgetown
       # Returns a hash representing additional metadata or an empty hash
       def site_data_hash
         @site_data_hash ||= begin
-          return {} unless resolved_author.is_a?(String)
-          return {} unless site.data["authors"].is_a?(Hash)
-
-          author_hash = site.data["authors"][resolved_author]
+          author_hash = if site.data.authors.is_a?(Hash) && resolved_author.is_a?(String)
+                          site.data.authors[resolved_author]
+                        end
           author_hash.is_a?(Hash) ? author_hash : {}
         end
       end
@@ -74,15 +72,14 @@ module Bridgetown
       # including site-wide metadata if the author is provided as a string,
       # or an empty hash, if the author cannot be resolved
       def author_hash
-        @author_hash ||= begin
-          if resolved_author.is_a? Hash
-            resolved_author
-          elsif resolved_author.is_a? String
-            { "name" => resolved_author }.merge(site_data_hash)
-          else
-            {}
-          end
-        end
+        @author_hash ||= case resolved_author
+                         when Hash
+                           resolved_author
+                         when String
+                           { "name" => resolved_author }.merge(site_data_hash)
+                         else
+                           {}
+                         end
       end
 
       # Since author_hash is aliased to fallback_data, any values in the hash

data/lib/bridgetown-seo-tag/builder.rb

@@ -4,8 +4,8 @@ module Bridgetown
   class SeoTag
     class Builder < Bridgetown::Builder
       def build
-        helper "seo", helpers_scope: true do |title: true|
-          context = Liquid::Context.new({}, {}, { site: site, page: view.page })
+        helper "seo" do |title: true|
+          context = Liquid::Context.new({}, {}, { site: site, page: helpers.view.page })
           tag_output = Liquid::Template.parse(
             "{% seo #{"title=false" unless title} %}"
           ).render!(context, {})
@@ -15,5 +15,3 @@ module Bridgetown
     end
   end
 end
-
-Bridgetown::SeoTag::Builder.register

data/lib/bridgetown-seo-tag/drop.rb

@@ -6,6 +6,7 @@ module Bridgetown
       include Bridgetown::SeoTag::UrlHelper
 
       TITLE_SEPARATOR = " | "
+      TAGLINE_SEPARATOR = ": "
       FORMAT_STRING_METHODS = [
         :markdownify, :strip_html, :normalize_whitespace, :escape_once,
       ].freeze
@@ -47,7 +48,14 @@ module Bridgetown
 
       # Page title without site title or description appended
       def page_title
-        @page_title ||= format_string(page["title"]) || site_title
+        @page_title ||= format_string(
+          if (page["title"] == "Index" || page["title"].blank?) &&
+              site_tagline_or_description
+            "#{site_title}#{TAGLINE_SEPARATOR}#{site_tagline_or_description}"
+          else
+            page["title"]
+          end
+        ) || site_title
       end
 
       def site_tagline_or_description
@@ -55,23 +63,18 @@ module Bridgetown
       end
 
       # Page title with site title or description appended
-      # rubocop:disable Metrics/CyclomaticComplexity
       def title
-        @title ||= begin
-          if site_title && page_title != site_title
-            page_title + TITLE_SEPARATOR + site_title
-          elsif site_description && site_title
-            site_title + TITLE_SEPARATOR + site_tagline_or_description
-          else
-            page_title || site_title
-          end
-        end
+        @title ||= if site_title && page_title != site_title &&
+            !format_string(page_title).start_with?(site_title + TAGLINE_SEPARATOR)
+                     page_title + TITLE_SEPARATOR + site_title
+                   else
+                     page_title || site_title
+                   end
 
         return page_number + @title if page_number
 
         @title
       end
-      # rubocop:enable Metrics/CyclomaticComplexity
 
       def name
         return @name if defined?(@name)
@@ -88,11 +91,9 @@ module Bridgetown
       end
 
       def description
-        @description ||= begin
-          format_string(
-            page["description"] || page["subtitle"] || page["excerpt"]
-          ) || site_description
-        end
+        @description ||= format_string(
+          page["description"] || page["subtitle"] || page["excerpt"]
+        ) || site_description
       end
 
       # A drop representing the page author
@@ -125,39 +126,33 @@ module Bridgetown
       end
 
       def type
-        @type ||= begin
-          if page_seo["type"]
-            page_seo["type"]
-          elsif homepage_or_about?
-            "WebSite"
-          elsif page["date"]
-            "BlogPosting"
-          else
-            "WebPage"
-          end
-        end
+        @type ||= if page_seo["type"]
+                    page_seo["type"]
+                  elsif homepage_or_about?
+                    "WebSite"
+                  elsif page["date"]
+                    "BlogPosting"
+                  else
+                    "WebPage"
+                  end
       end
 
       def links
-        @links ||= begin
-          if page_seo["links"]
-            page_seo["links"]
-          elsif homepage_or_about? && site_social["links"]
-            site_social["links"]
-          end
-        end
+        @links ||= if page_seo["links"]
+                     page_seo["links"]
+                   elsif homepage_or_about? && site_social["links"]
+                     site_social["links"]
+                   end
       end
 
       def logo
-        @logo ||= begin
-          return unless site.data.dig("site_metadata", "logo")
-
-          if absolute_url? site.data.dig("site_metadata", "logo")
-            filters.uri_escape site.data.dig("site_metadata", "logo")
-          else
-            filters.uri_escape filters.absolute_url site.data.dig("site_metadata", "logo")
-          end
-        end
+        @logo ||= if !site.data.dig("site_metadata", "logo")
+                    nil
+                  elsif absolute_url? site.data.dig("site_metadata", "logo")
+                    filters.uri_escape site.data.dig("site_metadata", "logo")
+                  else
+                    filters.uri_escape filters.absolute_url site.data.dig("site_metadata", "logo")
+                  end
       end
 
       def page_lang
@@ -165,15 +160,15 @@ module Bridgetown
       end
 
       def canonical_url
-        @canonical_url ||= begin
-          if page["canonical_url"].to_s.present?
-            page["canonical_url"]
-          elsif page["url"].to_s.present?
-            filters.absolute_url(page["url"]).to_s.gsub(%r!/index\.html$!, "/")
-          else
-            filters.absolute_url(page["relative_url"]).to_s.gsub(%r!/index\.html$!, "/")
-          end
-        end
+        @canonical_url ||= if page["canonical_url"].to_s.present?
+                             page["canonical_url"]
+                           elsif page["url"].to_s.present?
+                             filters.absolute_url(page["url"]).to_s.gsub(%r!/index\.html$!, "/")
+                           else
+                             filters.absolute_url(page["relative_url"]).to_s.gsub(
+                               %r!/index\.html$!, "/"
+                             )
+                           end
       end
 
       private
@@ -227,7 +222,7 @@ module Bridgetown
       end
 
       def site_social
-        @site_social ||= sub_hash(site.data.dig("site_metadata"), "social")
+        @site_social ||= sub_hash(site.data["site_metadata"], "social")
       end
 
       # Safely returns a sub hash

data/lib/bridgetown-seo-tag/image_drop.rb

@@ -34,14 +34,14 @@ module Bridgetown
 
       private
 
-      attr_accessor :page
-      attr_accessor :context
+      attr_accessor :page, :context
 
       # The normalized image hash with a `path` key (which may be nil)
       def image_hash
-        @image_hash ||= if page["image"].is_a?(Hash)
+        @image_hash ||= case page["image"]
+                        when Hash
                           { "path" => nil }.merge(page["image"])
-                        elsif page["image"].is_a?(String)
+                        when String
                           { "path" => page["image"] }
                         else
                           { "path" => nil }
@@ -50,9 +50,7 @@ module Bridgetown
       alias_method :fallback_data, :image_hash
 
       def raw_path
-        @raw_path ||= begin
-          image_hash["path"] || image_hash["facebook"] || image_hash["twitter"]
-        end
+        @raw_path ||= image_hash["path"] || image_hash["facebook"] || image_hash["twitter"]
       end
 
       def absolute_url

data/lib/bridgetown-seo-tag/version.rb

@@ -1,10 +1,10 @@
 # frozen_string_literal: true
 
 # Prevent bundler errors
-module Liquid; class Tag; end; end
+module Liquid; class Tag; end; end # rubocop:disable Lint/EmptyClass
 
 module Bridgetown
   class SeoTag < Liquid::Tag
-    VERSION = "4.0.1"
+    VERSION = "6.0.0"
   end
 end

data/lib/bridgetown-seo-tag.rb

@@ -44,7 +44,9 @@ module Bridgetown
     end
 
     def payload
-      paginator = context.registers[:page].pager if context.registers[:page].respond_to?(:pager)
+      if context.registers[:page].respond_to?(:paginator)
+        paginator = context.registers[:page].paginator
+      end
 
       # site_payload is an instance of UnifiedPayloadDrop
       Bridgetown::Utils.deep_merge_hashes(
@@ -81,20 +83,19 @@ module Bridgetown
       private
 
       def template_contents
-        @template_contents ||= begin
-          File.read(template_path).gsub(MINIFY_REGEX, "")
-        end
+        @template_contents ||= File.read(template_path).gsub(MINIFY_REGEX, "")
       end
 
       def template_path
-        @template_path ||= begin
-          File.expand_path "./template.html", File.dirname(__FILE__)
-        end
+        @template_path ||= File.expand_path "./template.html", File.dirname(__FILE__)
       end
     end
   end
 end
 
 Liquid::Template.register_tag("seo", Bridgetown::SeoTag)
-
 require "bridgetown-seo-tag/builder"
+
+Bridgetown.initializer :"bridgetown-seo-tag" do |config|
+  config.builder Bridgetown::SeoTag::Builder
+end

data/lib/template.html

@@ -35,11 +35,16 @@
   {% if seo_tag.image.width %}
     <meta property="og:image:width" content="{{ seo_tag.image.width }}" />
   {% endif %}
+  {% if seo_tag.image.alt %}
+    <meta property="og:image:alt" content="{{ seo_tag.image.alt }}" />
+  {% endif %}
 {% endif %}
 
 {% if page.date %}
   <meta property="og:type" content="article" />
   <meta property="article:published_time" content="{{ page.date | date_to_xmlschema }}" />
+{% else %}
+  <meta property="og:type" content="website" />
 {% endif %}
 
 {% if paginator.previous_page %}
@@ -49,7 +54,6 @@
   <link rel="next" href="{{ paginator.next_page_path | absolute_url }}" />
 {% endif %}
 
-
 {% if seo_tag.image %}
   <meta name="twitter:card" content="{{ page.twitter.card | default: site.metadata.twitter.card | default: "summary_large_image" }}" />
   <meta property="twitter:image" content="{{ seo_tag.image.path }}" />
@@ -57,6 +61,10 @@
   <meta name="twitter:card" content="summary" />
 {% endif %}
 
+{% if seo_tag.image.alt %}
+  <meta name="twitter:image:alt" content="{{ seo_tag.image.alt }}" />
+{% endif %}
+
 {% if seo_tag.page_title %}
   <meta property="twitter:title" content="{{ seo_tag.page_title }}" />
 {% endif %}

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: bridgetown-seo-tag
 version: !ruby/object:Gem::Version
-  version: 4.0.1
+  version: 6.0.0
 platform: ruby
 authors:
 - Bridgetown Team
-autorequire: 
+autorequire:
 bindir: bin
 cert_chain: []
-date: 2021-06-04 00:00:00.000000000 Z
+date: 2023-01-08 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bridgetown
@@ -16,7 +16,7 @@ dependencies:
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: 0.20.0
+        version: 1.2.0.beta5
     - - "<"
       - !ruby/object:Gem::Version
         version: '2.0'
@@ -26,7 +26,7 @@ dependencies:
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: 0.20.0
+        version: 1.2.0.beta5
     - - "<"
       - !ruby/object:Gem::Version
         version: '2.0'
@@ -64,14 +64,14 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '12.0'
+        version: '13.0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '12.0'
+        version: '13.0'
 - !ruby/object:Gem::Dependency
   name: rspec
   requirement: !ruby/object:Gem::Requirement
@@ -92,15 +92,15 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '0.2'
+        version: '0.3'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '0.2'
-description: 
+        version: '0.3'
+description:
 email: maintainers@bridgetownrb.com
 executables: []
 extensions: []
@@ -128,8 +128,9 @@ files:
 homepage: https://github.com/bridgetownrb/bridgetown-seo-tag
 licenses:
 - MIT
-metadata: {}
-post_install_message: 
+metadata:
+  rubygems_mfa_required: 'true'
+post_install_message:
 rdoc_options: []
 require_paths:
 - lib
@@ -137,15 +138,15 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
-      version: 2.5.0
+      version: 2.7.0
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.1.2
-signing_key: 
+rubygems_version: 3.3.3
+signing_key:
 specification_version: 4
 summary: A Bridgetown plugin to add metadata tags for search engines and social networks
   to better index and display your site's content.