bridgetown-core 0.16.0.beta1 → 0.18.0

data/lib/bridgetown-core/concerns/site/extensible.rb

@@ -1,20 +1,21 @@
 # frozen_string_literal: true
 
-module Bridgetown
-  module Site::Extensible
+class Bridgetown::Site
+  module Extensible
     # Load necessary libraries, plugins, converters, and generators.
-    # @see Bridgetown::Converter
-    # @see Bridgetown::Generator
+    # @see Converter
+    # @see Generator
     # @see PluginManager
     # @return [void]
     def setup
       plugin_manager.require_plugin_files
+      plugin_manager.setup_component_loaders
       self.converters = instantiate_subclasses(Bridgetown::Converter)
       self.generators = instantiate_subclasses(Bridgetown::Generator)
     end
 
     # Run all Generators.
-    # @see Bridgetown::Generator
+    # @see Generator
     # @return [void]
     def generate
       generators.each do |generator|
@@ -33,10 +34,10 @@ module Bridgetown
       end
     end
 
-    # Get the implementation class for the given Converter.
-    # @param klass [Object] The Class of the Converter to fetch.
-    # @return [Bridgetown::Converter] Returns the {Bridgetown::Converter}
-    #   instance implementing the given +Converter+.
+    # Get the implementation for the given Converter class.
+    # @param klass [Class] The Class of the Converter to fetch.
+    # @return [Converter] Returns the {Converter}
+    #   instance implementing the given `Converter` class.
     def find_converter_instance(klass)
       @find_converter_instance ||= {}
       @find_converter_instance[klass] ||= begin
@@ -45,11 +46,11 @@ module Bridgetown
       end
     end
 
-    # Create an array of instances of the subclasses of the class or module
+    # Create an array of instances of the subclasses of the class
     #   passed in as argument.
-    # @param klass [Class, Module] - class or module containing the subclasses.
-    # @return [Array<Object>] Returns an array of instances of subclasses of
-    #   +klass+.
+    # @param klass [Class] - class which is the parent of the subclasses.
+    # @return [Array<Converter, Generator>] Returns an array of instances of
+    #   subclasses of `klass`.
     def instantiate_subclasses(klass)
       klass.descendants.sort.map do |c|
         c.new(config)

data/lib/bridgetown-core/concerns/site/localizable.rb

@@ -0,0 +1,24 @@
+# frozen_string_literal: true
+
+class Bridgetown::Site
+  module Localizable
+    # Returns the current and/or default configured locale
+    # @return String
+    def locale
+      if @locale
+        @locale
+      else
+        @locale = ENV.fetch("BRIDGETOWN_LOCALE", config[:default_locale]).to_sym
+        I18n.load_path << Dir[in_source_dir("_locales") + "/*.yml"]
+        I18n.available_locales = config[:available_locales]
+        I18n.default_locale = @locale
+      end
+    end
+
+    # Sets the current locale for the site
+    # @param new_locale [String] for example: "en" for English, "es" for Spanish
+    def locale=(new_locale)
+      I18n.locale = @locale = new_locale.to_sym
+    end
+  end
+end

data/lib/bridgetown-core/concerns/site/processable.rb

@@ -1,7 +1,7 @@
 # frozen_string_literal: true
 
-module Bridgetown
-  module Site::Processable
+class Bridgetown::Site
+  module Processable
     # Reset, Read, Generate, Render, Cleanup, Process, and Write this Site to output.
     # @return [void]
     # @see #reset
@@ -22,18 +22,19 @@ module Bridgetown
 
     # rubocop:disable Metrics/AbcSize
 
-    # Reset Site details.
+    # Reset all in-memory data and content.
     # @return [void]
     def reset
-      self.time = if config["time"]
-                    Utils.parse_date(config["time"].to_s, "Invalid time in bridgetown.config.yml.")
-                  else
-                    Time.now
-                  end
-      self.layouts = ActiveSupport::HashWithIndifferentAccess.new
+      self.time = Time.now
+      if config["time"]
+        self.time = Bridgetown::Utils.parse_date(
+          config["time"].to_s, "Invalid time in bridgetown.config.yml."
+        )
+      end
+      self.layouts = HashWithDotAccess::Hash.new
       self.pages = []
       self.static_files = []
-      self.data = ActiveSupport::HashWithIndifferentAccess.new
+      self.data = HashWithDotAccess::Hash.new
       @post_attr_hash = {}
       @collections = nil
       @documents = nil
@@ -50,7 +51,7 @@ module Bridgetown
 
     # rubocop:enable Metrics/AbcSize
 
-    # Read Site data from disk and load it into internal data structures.
+    # Read data from disk and load it into internal memory.
     # @return [void]
     def read
       Bridgetown::Hooks.trigger :site, :pre_read, self

data/lib/bridgetown-core/concerns/site/renderable.rb

@@ -1,24 +1,20 @@
 # frozen_string_literal: true
 
-module Bridgetown
-  module Site::Renderable
-    # Render the site to the destination.
+class Bridgetown::Site
+  module Renderable
+    # Render all pages & documents so they're ready to be written out to disk.
     # @return [void]
+    # @see Page
+    # @see Document
     def render
-      payload = site_payload
-
-      Bridgetown::Hooks.trigger :site, :pre_render, self, payload
-
+      Bridgetown::Hooks.trigger :site, :pre_render, self
       execute_inline_ruby_for_layouts!
-
-      render_docs(payload)
-      render_pages(payload)
-
-      Bridgetown::Hooks.trigger :site, :post_render, self, payload
+      render_docs
+      render_pages
+      Bridgetown::Hooks.trigger :site, :post_render, self
     end
 
-    # Executes inline Ruby frontmatter if
-    # +ENV+["BRIDGETOWN_RUBY_IN_FRONTMATTER"] equals "true"
+    # Executes inline Ruby frontmatter
     #
     # @example
     #   calculation: !ruby/string:Rb |
@@ -34,36 +30,47 @@ module Bridgetown
     end
 
     # Renders all documents
-    # @param payload [Hash] A hash of site data.
     # @return [void]
-    # @see Bridgetown::Site::Content#site_payload
-    def render_docs(payload)
+    def render_docs
       collections.each_value do |collection|
         collection.docs.each do |document|
-          render_regenerated(document, payload)
+          render_with_locale(document) do
+            render_regenerated document
+          end
         end
       end
     end
 
     # Renders all pages
-    # @param payload [Hash] A hash of site data.
     # @return [void]
-    # @see Bridgetown::Site::Content#site_payload
-    def render_pages(payload)
+    def render_pages
       pages.each do |page|
-        render_regenerated(page, payload)
+        render_regenerated page
+      end
+    end
+
+    # Renders a document while ensuring site locale is set if the data is available.
+    # @param document [Document] The document to render
+    # @yield Runs the block in between locale setting and resetting
+    # @return [void]
+    def render_with_locale(document)
+      if document.data["locale"]
+        previous_locale = locale
+        self.locale = document.data["locale"]
+        yield
+        self.locale = previous_locale
+      else
+        yield
       end
     end
 
-    # Regenerates a site using {Bridgetown::Renderer}
-    # @param document [Post] The document to regenerate.
-    # @param payload [Hash] A hash of site data.
+    # Regenerates a site using {Renderer}
+    # @param document [Document] The document to regenerate.
     # @return [void]
-    # @see Bridgetown::Renderer
-    def render_regenerated(document, payload)
+    def render_regenerated(document)
       return unless regenerator.regenerate?(document)
 
-      Bridgetown::Renderer.new(self, document, payload).run
+      Bridgetown::Renderer.new(self, document).run
     end
   end
 end

data/lib/bridgetown-core/concerns/site/writable.rb

@@ -1,7 +1,7 @@
 # frozen_string_literal: true
 
-module Bridgetown
-  module Site::Writable
+class Bridgetown::Site
+  module Writable
     # Remove orphaned files and empty directories in destination.
     #
     # @return [void]
@@ -9,7 +9,7 @@ module Bridgetown
       @cleaner.cleanup!
     end
 
-    # Write static files, pages, and posts.
+    # Write static files, pages, and documents to the destination folder.
     #
     # @return [void]
     def write
@@ -20,22 +20,14 @@ module Bridgetown
       Bridgetown::Hooks.trigger :site, :post_write, self
     end
 
-    # Yields the pages from {#pages}, {#static_files}, and {#docs_to_write}.
+    # Yields all content objects while looping through {#pages},
+    #   {#static_files_to_write}, and {#docs_to_write}.
     #
-    # @yieldparam item [Document, Page, StaticFile] Yields a
-    # {#Bridgetown::Page}, {#Bridgetown::StaticFile}, or
-    # {#Bridgetown::Document} object.
+    # @yieldparam item [Document, Page, StaticFile]
     #
     # @return [void]
-    #
-    # @see #pages
-    # @see #static_files
-    # @see #docs_to_write
-    # @see Page
-    # @see StaticFile
-    # @see Document
     def each_site_file
-      %w(pages static_files docs_to_write).each do |type|
+      %w(pages static_files_to_write docs_to_write).each do |type|
         send(type).each do |item|
           yield item
         end

data/lib/bridgetown-core/concerns/validatable.rb

@@ -22,7 +22,7 @@ module Bridgetown
                                  **Utils.merged_file_read_opts(site, opts))
         if content =~ Document::YAML_FRONT_MATTER_REGEXP
           self.content = $POSTMATCH
-          self.data = SafeYAML.load(Regexp.last_match(1))&.with_indifferent_access
+          self.data = SafeYAML.load(Regexp.last_match(1))&.with_dot_access
         end
       rescue Psych::SyntaxError => e
         Bridgetown.logger.warn "YAML Exception reading #{filename}: #{e.message}"
@@ -32,7 +32,7 @@ module Bridgetown
         raise e if site.config["strict_front_matter"]
       end
 
-      self.data ||= ActiveSupport::HashWithIndifferentAccess.new
+      self.data ||= HashWithDotAccess::Hash.new
 
       validate_data! filename
       validate_permalink! filename

data/lib/bridgetown-core/configuration.rb

@@ -1,10 +1,12 @@
 # frozen_string_literal: true
 
 module Bridgetown
-  # TODO: refactor this whole object! Already had to fix obscure
-  # bugs just making minor changes, and all the indirection is
-  # quite hard to decipher. -JW
-  class Configuration < ActiveSupport::HashWithIndifferentAccess
+  # Holds the processed configuration loaded from the YAML config file.
+  #
+  # @todo refactor this whole object! Already had to fix obscure
+  #   bugs just making minor changes, and all the indirection is
+  #   quite hard to decipher. -JW
+  class Configuration < HashWithDotAccess::Hash
     # Default options. Overridden by values in bridgetown.config.yml.
     # Strings rather than symbols are used for compatibility with YAML.
     DEFAULTS = {
@@ -35,7 +37,7 @@ module Bridgetown
       "limit_posts"          => 0,
       "future"               => false,
       "unpublished"          => false,
-      "ruby_in_front_matter" => true, # requires BRIDGETOWN_RUBY_IN_FRONT_MATTER == "true"
+      "ruby_in_front_matter" => true,
 
       # Conversion
       "markdown"             => "kramdown",
@@ -52,6 +54,8 @@ module Bridgetown
       "show_dir_listing"     => false,
 
       # Output Configuration
+      "available_locales"    => ["en"],
+      "default_locale"       => "en",
       "permalink"            => "date",
       "timezone"             => nil, # use the local timezone
 
@@ -276,7 +280,7 @@ module Bridgetown
     end
 
     def should_execute_inline_ruby?
-      ENV["BRIDGETOWN_RUBY_IN_FRONT_MATTER"] == "true" &&
+      ENV["BRIDGETOWN_RUBY_IN_FRONT_MATTER"] != "false" &&
         self["ruby_in_front_matter"]
     end
 
@@ -309,6 +313,10 @@ module Bridgetown
         raise Bridgetown::Errors::InvalidConfigurationError,
               "'#{option}' should be set as an array, but was: #{self[option].inspect}."
       end
+
+      # add _pages to includes set
+      self[:include] << "_pages"
+
       self
     end
   end

data/lib/bridgetown-core/converter.rb

@@ -16,34 +16,6 @@ module Bridgetown
       end
     end
 
-    # Public: Get or set the highlighter prefix. When an argument is specified,
-    # the prefix will be set. If no argument is specified, the current prefix
-    # will be returned.
-    #
-    # highlighter_prefix - The String prefix (default: nil).
-    #
-    # Returns the String prefix.
-    def self.highlighter_prefix(highlighter_prefix = nil)
-      unless defined?(@highlighter_prefix) && highlighter_prefix.nil?
-        @highlighter_prefix = highlighter_prefix
-      end
-      @highlighter_prefix
-    end
-
-    # Public: Get or set the highlighter suffix. When an argument is specified,
-    # the suffix will be set. If no argument is specified, the current suffix
-    # will be returned.
-    #
-    # highlighter_suffix - The String suffix (default: nil).
-    #
-    # Returns the String suffix.
-    def self.highlighter_suffix(highlighter_suffix = nil)
-      unless defined?(@highlighter_suffix) && highlighter_suffix.nil?
-        @highlighter_suffix = highlighter_suffix
-      end
-      @highlighter_suffix
-    end
-
     # Initialize the converter.
     #
     # Returns an initialized Converter.
@@ -70,19 +42,5 @@ module Bridgetown
     def output_ext(_ext)
       ".html"
     end
-
-    # Get the highlighter prefix.
-    #
-    # Returns the String prefix.
-    def highlighter_prefix
-      self.class.highlighter_prefix
-    end
-
-    # Get the highlighter suffix.
-    #
-    # Returns the String suffix.
-    def highlighter_suffix
-      self.class.highlighter_suffix
-    end
   end
 end

data/lib/bridgetown-core/converters/erb_templates.rb

@@ -1,52 +1,114 @@
 # frozen_string_literal: true
 
-require "tilt/erb"
-require "active_support/core_ext/hash/keys"
+require "tilt/erubi"
 
 module Bridgetown
+  class ERBBuffer < String
+    def concat_to_s(input)
+      concat input.to_s
+    end
+
+    alias_method :safe_append=, :concat_to_s
+    alias_method :append=, :concat_to_s
+    alias_method :safe_expr_append=, :concat_to_s
+  end
+
+  class ERBEngine < Erubi::Engine
+    private
+
+    def add_code(code)
+      @src << code
+      @src << ";#{@bufvar};" if code.strip.split(".").first == "end"
+      @src << ";" unless code[Erubi::RANGE_LAST] == "\n"
+    end
+
+    # pulled from Rails' ActionView
+    BLOCK_EXPR = %r!\s*((\s+|\))do|\{)(\s*\|[^|]*\|)?\s*\Z!.freeze
+
+    def add_expression(indicator, code)
+      if BLOCK_EXPR.match?(code)
+        src << "#{@bufvar}.append= " << code
+      else
+        super
+      end
+    end
+
+    # Don't allow == to output escaped strings, as that's the opposite of Rails
+    def add_expression_result_escaped(code)
+      add_expression_result(code)
+    end
+  end
+
   class ERBView < RubyTemplateView
-    include ERB::Util
+    def h(input)
+      Erubi.h(input)
+    end
 
     def partial(partial_name, options = {})
       options.merge!(options[:locals]) if options[:locals]
+      options[:content] = yield if block_given?
 
       partial_segments = partial_name.split("/")
       partial_segments.last.sub!(%r!^!, "_")
       partial_name = partial_segments.join("/")
 
-      Tilt::ERBTemplate.new(
+      Tilt::ErubiTemplate.new(
         site.in_source_dir(site.config[:partials_dir], "#{partial_name}.erb"),
-        trim: "<>-",
-        outvar: "@_erbout"
+        outvar: "@_erbout",
+        bufval: "Bridgetown::ERBBuffer.new",
+        engine_class: ERBEngine
       ).render(self, options)
     end
 
-    def markdownify
+    def markdownify(input = nil, &block)
+      content = Bridgetown::Utils.reindent_for_markdown(
+        block.nil? ? input.to_s : capture(&block)
+      )
+      converter = site.find_converter_instance(Bridgetown::Converters::Markdown)
+      converter.convert(content).strip
+    end
+
+    def capture(obj = nil, &block)
       previous_buffer_state = @_erbout
-      @_erbout = +""
-      result = yield
+      @_erbout = ERBBuffer.new
+
+      # For compatibility with ActionView, not used by Bridgetown normally
+      previous_ob_state = @output_buffer
+      @output_buffer = ERBBuffer.new
+
+      result = instance_exec(obj, &block)
+      if @output_buffer != ""
+        # use Rails' ActionView buffer if present
+        result = @output_buffer
+      end
       @_erbout = previous_buffer_state
+      @output_buffer = previous_ob_state
 
-      content = Bridgetown::Utils.reindent_for_markdown(result)
-      converter = site.find_converter_instance(Bridgetown::Converters::Markdown)
-      md_output = converter.convert(content).strip
-      @_erbout << md_output
+      result.respond_to?(:html_safe) ? result.html_safe : result
     end
   end
 
   module Converters
     class ERBTemplates < Converter
+      priority :highest
       input :erb
 
-      # Logic to do the content conversion.
+      # Logic to do the ERB content conversion.
       #
-      # content - String content of file (without front matter).
+      # @param content [String] Content of the file (without front matter).
+      # @params convertible [Bridgetown::Page, Bridgetown::Document, Bridgetown::Layout]
+      #   The instantiated object which is processing the file.
       #
-      # Returns a String of the converted content.
+      # @return [String] The converted content.
       def convert(content, convertible)
         erb_view = Bridgetown::ERBView.new(convertible)
 
-        erb_renderer = Tilt::ERBTemplate.new(trim: "<>-", outvar: "@_erbout") { content }
+        erb_renderer = Tilt::ErubiTemplate.new(
+          convertible.relative_path,
+          outvar: "@_erbout",
+          bufval: "Bridgetown::ERBBuffer.new",
+          engine_class: ERBEngine
+        ) { content }
 
         if convertible.is_a?(Bridgetown::Layout)
           erb_renderer.render(erb_view) do
@@ -56,6 +118,20 @@ module Bridgetown
           erb_renderer.render(erb_view)
         end
       end
+
+      def matches(ext, convertible)
+        if convertible.data[:template_engine] == "erb" ||
+            (convertible.data[:template_engine].nil? &&
+              @config[:template_engine] == "erb")
+          return true
+        end
+
+        super(ext)
+      end
+
+      def output_ext(ext)
+        ext == ".erb" ? ".html" : ext
+      end
     end
   end
 end