curlybars 1.8.0 → 1.10.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: acf93500c679c1ecff9dfc08b9f5ec8a5ee026465a0d2e39c4f47b9127670715
-  data.tar.gz: 9308ce377a66177b91338e0f16c20ce4096fad3cf4b6591e5ad99baa294113f4
+  metadata.gz: a21d1b2f9bc23315fd45fc0f3e1f1dba730d7d7555b221968956635e00f2f989
+  data.tar.gz: d834d004783806af09cec7fa411bd5184529a30b1d60995cebcba564d54cd5e5
 SHA512:
-  metadata.gz: 8b0e37a737948d1e60acb3bd3e0f37876463433fbe05003d3ed8179d622ed04c78494fec37c2405a0ddcb9554277215570d3fcf3a377d275a9e0a171c2e30c29
-  data.tar.gz: f3bc6fde32e9f8f7d4d6df7da6b77a6ab750932b61baf17d8ad35f9f134df36b7107724fcc6a27911da405a5d7f440cff573a9cce89de6e673478a77dd03b7d3
+  metadata.gz: '066406790ee118acfcd150c5d604e41298dfca0436be419482e05a71253396827c2314313036dfe03d853a01e2b615fec66ab26fa43058fbd94fa27160dad970'
+  data.tar.gz: 21580e1338a0200d0210e298752030320c38fbf7a9f1021912ee638d5b4a3492bfdae79e64eb754f919fb34d17f88c74f624e5aaab577bf566dcaf31ef173188

data/lib/curlybars/error/lex.rb

@@ -9,7 +9,7 @@ module Curlybars
 
         error_line = source.split("\n")[line_number - 1]
         before_error = error_line.first(line_offset).last(10)
-        after_error = error_line[line_offset + 1..-1].first(10)
+        after_error = error_line[line_offset + 1..].first(10)
         error = error_line[line_offset]
 
         details = [before_error, error, after_error]

data/lib/curlybars/error/parse.rb

@@ -16,7 +16,7 @@ module Curlybars
 
           error_line = source.split("\n")[line_number - 1]
           before_error = error_line.first(line_offset).last(10)
-          after_error = error_line[line_offset + length..-1].first(10)
+          after_error = error_line[line_offset + length..].first(10)
           error = error_line.slice(line_offset, length)
 
           details = [before_error, error, after_error]

data/lib/curlybars/node/path.rb

@@ -109,20 +109,18 @@ module Curlybars
 
           offset_adjustment = 0
           dotted_path_side.split(/\./).map(&:to_sym).inject(base_tree) do |sub_tree, step|
-            begin
-              if step == :this
-                next sub_tree
-              elsif step == :length && (sub_tree.is_a?(Array) && sub_tree.first.is_a?(Hash))
-                next nil # :length is synthesised leaf
-              elsif !(sub_tree.is_a?(Hash) && sub_tree.key?(step))
-                message = step.to_s == path ? "'#{path}' does not exist" : "not possible to access `#{step}` in `#{path}`"
-                raise Curlybars::Error::Validate.new('unallowed_path', message, position, offset_adjustment, path: path, step: step)
-              end
-
-              sub_tree[step]
-            ensure
-              offset_adjustment += step.length + 1 # '1' is the length of the dot
+            if step == :this
+              next sub_tree
+            elsif step == :length && (sub_tree.is_a?(Array) && sub_tree.first.is_a?(Hash))
+              next nil # :length is synthesised leaf
+            elsif !(sub_tree.is_a?(Hash) && sub_tree.key?(step))
+              message = step.to_s == path ? "'#{path}' does not exist" : "not possible to access `#{step}` in `#{path}`"
+              raise Curlybars::Error::Validate.new('unallowed_path', message, position, offset_adjustment, path: path, step: step)
             end
+
+            sub_tree[step]
+          ensure
+            offset_adjustment += step.length + 1 # '1' is the length of the dot
           end
         end
       end

data/lib/curlybars/presenter.rb

@@ -276,8 +276,8 @@ module Curlybars
     # Delegates private method calls to the current view context.
     #
     # The view context, an instance of ActionView::Base, is set by Rails.
-    def method_missing(method, *args, &block)
-      @_context.public_send(method, *args, &block)
+    def method_missing(method, *args, **kwargs, &block)
+      @_context.public_send(method, *args, **kwargs, &block)
     end
 
     # Tells ruby (and developers) what methods we can accept.

data/lib/curlybars/rendering_support.rb

@@ -61,7 +61,8 @@ module Curlybars
       scope = variables.first(variables_position).reverse.find do |vars|
         vars.key? variable
       end
-      return scope[variable] if scope
+
+      scope[variable] if scope
     end
 
     def path(path, position)

data/lib/curlybars/template_handler.rb

@@ -13,17 +13,9 @@ module Curlybars
       # template - The ActionView::Template template that should be compiled.
       #
       # Returns a String containing the Ruby code representing the template.
-      if ActionView::VERSION::MAJOR < 6
-        def call(template)
-          instrument(template) do
-            compile_for_actionview5(template)
-          end
-        end
-      else
-        def call(template, source)
-          instrument(template) do
-            compile(template, source)
-          end
+      def call(template, source)
+        instrument(template) do
+          compile(template, source)
         end
       end
 
@@ -51,10 +43,6 @@ module Curlybars
 
       private
 
-      def compile_for_actionview5(template)
-        compile(template, template.source)
-      end
-
       def compile(template, source)
         # Template is empty, so there's no need to initialize a presenter.
         return %("") if source.empty?

data/lib/curlybars/version.rb

@@ -1,3 +1,3 @@
 module Curlybars
-  VERSION = '1.8.0'.freeze
+  VERSION = '1.10.0'.freeze
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: curlybars
 version: !ruby/object:Gem::Version
-  version: 1.8.0
+  version: 1.10.0
 platform: ruby
 authors:
 - Libo Cannici
@@ -12,10 +12,10 @@ authors:
 - Andreas Garnæs
 - Augusto Silva
 - Attila Večerek
-autorequire:
+autorequire: 
 bindir: bin
 cert_chain: []
-date: 2022-11-09 00:00:00.000000000 Z
+date: 2024-01-16 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: actionpack
@@ -23,40 +23,40 @@ dependencies:
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '4.2'
+        version: '6.0'
     - - "<"
       - !ruby/object:Gem::Version
-        version: '7.1'
+        version: '7.2'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '4.2'
+        version: '6.0'
     - - "<"
       - !ruby/object:Gem::Version
-        version: '7.1'
+        version: '7.2'
 - !ruby/object:Gem::Dependency
   name: activesupport
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '4.2'
+        version: '6.0'
     - - "<"
       - !ruby/object:Gem::Version
-        version: '7.1'
+        version: '7.2'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '4.2'
+        version: '6.0'
     - - "<"
       - !ruby/object:Gem::Version
-        version: '7.1'
+        version: '7.2'
 - !ruby/object:Gem::Dependency
   name: ffi
   requirement: !ruby/object:Gem::Requirement
@@ -119,20 +119,20 @@ dependencies:
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '4.2'
+        version: '6.0'
     - - "<"
       - !ruby/object:Gem::Version
-        version: '7.1'
+        version: '7.2'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '4.2'
+        version: '6.0'
     - - "<"
       - !ruby/object:Gem::Version
-        version: '7.1'
+        version: '7.2'
 - !ruby/object:Gem::Dependency
   name: rake
   requirement: !ruby/object:Gem::Requirement
@@ -151,72 +151,72 @@ dependencies:
   name: rspec-rails
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
-        version: '3.8'
+        version: '0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
-        version: '3.8'
+        version: '0'
 - !ruby/object:Gem::Dependency
   name: rubocop
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 1.6.0
+        version: '1.57'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 1.6.0
+        version: '1.57'
 - !ruby/object:Gem::Dependency
   name: rubocop-performance
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 1.9.0
+        version: '1.19'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 1.9.0
+        version: '1.19'
 - !ruby/object:Gem::Dependency
   name: rubocop-rake
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.5.0
+        version: '0.6'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.5.0
+        version: '0.6'
 - !ruby/object:Gem::Dependency
   name: rubocop-rspec
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 2.1.0
+        version: '2.25'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 2.1.0
+        version: '2.25'
 description: |-
   A view layer for your Rails apps that separates structure and logic, using Handlebars templates.
   Strongly inspired by Curly Template gem by Daniel Schierbeck.
@@ -267,49 +267,11 @@ files:
 - lib/curlybars/template_handler.rb
 - lib/curlybars/version.rb
 - lib/curlybars/visitor.rb
-- spec/acceptance/application_layout_spec.rb
-- spec/acceptance/collection_blocks_spec.rb
-- spec/acceptance/global_helper_spec.rb
-- spec/curlybars/configuration_spec.rb
-- spec/curlybars/error/base_spec.rb
-- spec/curlybars/error/compile_spec.rb
-- spec/curlybars/error/lex_spec.rb
-- spec/curlybars/error/parse_spec.rb
-- spec/curlybars/error/render_spec.rb
-- spec/curlybars/error/validate_spec.rb
-- spec/curlybars/lexer_spec.rb
-- spec/curlybars/method_whitelist_spec.rb
-- spec/curlybars/processor/tilde_spec.rb
-- spec/curlybars/rendering_support_spec.rb
-- spec/curlybars/safe_buffer_spec.rb
-- spec/curlybars/template_handler_spec.rb
-- spec/integration/cache_spec.rb
-- spec/integration/comment_spec.rb
-- spec/integration/exception_spec.rb
-- spec/integration/node/block_helper_else_spec.rb
-- spec/integration/node/each_else_spec.rb
-- spec/integration/node/each_spec.rb
-- spec/integration/node/escape_spec.rb
-- spec/integration/node/helper_spec.rb
-- spec/integration/node/if_else_spec.rb
-- spec/integration/node/if_spec.rb
-- spec/integration/node/output_spec.rb
-- spec/integration/node/partial_spec.rb
-- spec/integration/node/path_spec.rb
-- spec/integration/node/root_spec.rb
-- spec/integration/node/sub_expression_spec.rb
-- spec/integration/node/template_spec.rb
-- spec/integration/node/unless_else_spec.rb
-- spec/integration/node/unless_spec.rb
-- spec/integration/node/with_spec.rb
-- spec/integration/processor/tilde_spec.rb
-- spec/integration/processors_spec.rb
-- spec/integration/visitor_spec.rb
 homepage: https://github.com/zendesk/curlybars
 licenses:
 - Apache-2.0
 metadata: {}
-post_install_message:
+post_install_message: 
 rdoc_options:
 - "--charset=UTF-8"
 require_paths:
@@ -318,53 +280,15 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
-      version: '2.4'
+      version: '3.0'
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.1.6
-signing_key:
+rubygems_version: 3.5.3
+signing_key: 
 specification_version: 4
 summary: Create your views using Handlebars templates!
-test_files:
-- spec/curlybars/rendering_support_spec.rb
-- spec/curlybars/method_whitelist_spec.rb
-- spec/curlybars/configuration_spec.rb
-- spec/curlybars/processor/tilde_spec.rb
-- spec/curlybars/template_handler_spec.rb
-- spec/curlybars/error/compile_spec.rb
-- spec/curlybars/error/render_spec.rb
-- spec/curlybars/error/parse_spec.rb
-- spec/curlybars/error/validate_spec.rb
-- spec/curlybars/error/lex_spec.rb
-- spec/curlybars/error/base_spec.rb
-- spec/curlybars/safe_buffer_spec.rb
-- spec/curlybars/lexer_spec.rb
-- spec/integration/processor/tilde_spec.rb
-- spec/integration/exception_spec.rb
-- spec/integration/processors_spec.rb
-- spec/integration/visitor_spec.rb
-- spec/integration/node/escape_spec.rb
-- spec/integration/node/unless_else_spec.rb
-- spec/integration/node/output_spec.rb
-- spec/integration/node/if_spec.rb
-- spec/integration/node/sub_expression_spec.rb
-- spec/integration/node/unless_spec.rb
-- spec/integration/node/with_spec.rb
-- spec/integration/node/partial_spec.rb
-- spec/integration/node/block_helper_else_spec.rb
-- spec/integration/node/root_spec.rb
-- spec/integration/node/template_spec.rb
-- spec/integration/node/each_spec.rb
-- spec/integration/node/each_else_spec.rb
-- spec/integration/node/helper_spec.rb
-- spec/integration/node/path_spec.rb
-- spec/integration/node/if_else_spec.rb
-- spec/integration/comment_spec.rb
-- spec/integration/cache_spec.rb
-- spec/acceptance/collection_blocks_spec.rb
-- spec/acceptance/global_helper_spec.rb
-- spec/acceptance/application_layout_spec.rb
+test_files: []

data/spec/acceptance/application_layout_spec.rb

@@ -1,60 +0,0 @@
-describe "Using Curlybars for the application layout", type: :request do
-  example "A simple layout view in Curlybars" do
-    get '/'
-
-    expect(body).to eq(<<~HTML)
-      <html>
-      <head>
-        <title>Dummy app</title>
-      </head>
-      <body>
-      <h1>Dashboard</h1>
-      <p>Hello, World!</p>
-      <p>Welcome!</p>
-
-      </body>
-      </html>
-    HTML
-  end
-
-  describe "Curlybars" do
-    before do
-      Curlybars.configure do |config|
-        config.presenters_namespace = 'curlybars_presenters'
-      end
-    end
-
-    after do
-      Curlybars.reset
-    end
-
-    example "A simple layout view in Curlybars with html safe logic" do
-      get '/articles/1'
-
-      expect(body).to eq(<<~HTML)
-        <html>
-        <head>
-          <title>Dummy app</title>
-        </head>
-        <body>
-        <p>Hi Admin</p>
-
-        <h1>Article: The Prince</h1>
-
-        <p>This is <strong>important</strong>!</p>
-
-        <p>&lt;script&gt;alert(&#39;bad&#39;)&lt;/script&gt;</p>
-
-
-        <p>
-          author Nicolò
-          <img src="http://example.com/foo.png" />
-        </p>
-
-
-        </body>
-        </html>
-      HTML
-    end
-  end
-end

data/spec/acceptance/collection_blocks_spec.rb

@@ -1,28 +0,0 @@
-describe "Collection blocks", type: :request do
-  before do
-    Curlybars.configure do |config|
-      config.presenters_namespace = 'curlybars_presenters'
-    end
-  end
-
-  example "Rendering collections" do
-    get '/categories'
-
-    expect(body).to eq(<<~HTML)
-      <html>
-      <head>
-        <title>Dummy app</title>
-      </head>
-      <body>
-
-      <h1>This are the categories</h1>
-
-      <p>One</p>
-      <p>Two</p>
-
-
-      </body>
-      </html>
-    HTML
-  end
-end

data/spec/acceptance/global_helper_spec.rb

@@ -1,25 +0,0 @@
-describe "Collection blocks", type: :request do
-  before do
-    Curlybars.configure do |config|
-      config.presenters_namespace = 'curlybars_presenters'
-      config.global_helpers_provider_classes = [GlobalHelpers]
-    end
-  end
-
-  example "Render a global helper" do
-    get '/welcome'
-
-    expect(body).to eq(<<~HTML)
-      <html>
-      <head>
-        <title>Dummy app</title>
-      </head>
-      <body>
-      Login as: Admin
-      Account: Testing
-
-      </body>
-      </html>
-    HTML
-  end
-end

data/spec/curlybars/configuration_spec.rb

@@ -1,57 +0,0 @@
-describe Curlybars::Configuration do
-  after do
-    Curlybars.reset
-  end
-
-  describe "#presenters_namespace" do
-    it "defaults to an empty string" do
-      presenters_namespace = Curlybars::Configuration.new.presenters_namespace
-
-      expect(presenters_namespace).to eq('')
-    end
-  end
-
-  describe "#presenters_namespace=" do
-    it "can set value" do
-      config = Curlybars::Configuration.new
-      config.presenters_namespace = 'foo'
-      expect(config.presenters_namespace).to eq('foo')
-    end
-  end
-
-  describe "#custom_processors" do
-    it "can set value" do
-      config = Curlybars::Configuration.new
-      config.custom_processors = ['test']
-      expect(config.custom_processors).to eq(['test'])
-    end
-  end
-
-  describe ".configure" do
-    before do
-      Curlybars.configure do |config|
-        config.presenters_namespace = 'bar'
-      end
-    end
-
-    it "returns correct value for presenters_namespace" do
-      presenters_namespace = Curlybars.configuration.presenters_namespace
-
-      expect(presenters_namespace).to eq('bar')
-    end
-  end
-
-  describe ".reset" do
-    it "resets the configuration to default value" do
-      Curlybars.configure do |config|
-        config.presenters_namespace = 'foobarbaz'
-      end
-
-      Curlybars.reset
-
-      presenters_namespace = Curlybars.configuration.presenters_namespace
-
-      expect(presenters_namespace).to eq('')
-    end
-  end
-end

data/spec/curlybars/error/base_spec.rb

@@ -1,41 +0,0 @@
-describe Curlybars::Error::Base do
-  let(:position) do
-    OpenStruct.new(line_number: 1, line_offset: 0)
-  end
-
-  it "creates an exception with the given message" do
-    message = "message"
-
-    exception = Curlybars::Error::Base.new('id', message, position)
-
-    expect(exception.message).to be message
-  end
-
-  it "doesn't change the backtrace for nil position.file_name" do
-    exception = Curlybars::Error::Base.new('id', 'message', position)
-
-    expect(exception.backtrace).to be nil
-  end
-
-  it "sets the right backtrace for non-nil position.file_name" do
-    position.file_name = 'template.hbs'
-
-    exception = Curlybars::Error::Base.new('id', 'message', position)
-
-    expect(exception.backtrace).not_to be nil
-  end
-
-  it "sets the position as an instance varaible" do
-    exception = Curlybars::Error::Base.new('id', 'message', position)
-
-    expect(exception.position).to be position
-  end
-
-  it "sets the id as an instance varaible" do
-    id = 'id'
-
-    exception = Curlybars::Error::Base.new(id, 'message', position)
-
-    expect(exception.id).to be id
-  end
-end

data/spec/curlybars/error/compile_spec.rb

@@ -1,19 +0,0 @@
-describe Curlybars::Error::Compile do
-  let(:position) do
-    OpenStruct.new(
-      line_number: 2,
-      line_offset: 3,
-      stream_offset: 14,
-      length: 3,
-      file_name: 'template.hbs'
-    )
-  end
-
-  it "prefixes the id with `compile.`" do
-    id = 'id'
-
-    exception = Curlybars::Error::Compile.new(id, 'message', position)
-
-    expect(exception.id).to eq 'compile.%s' % id
-  end
-end

data/spec/curlybars/error/lex_spec.rb

@@ -1,25 +0,0 @@
-describe Curlybars::Error::Lex do
-  let(:file_name) { 'template.hbs' }
-
-  let(:source) { "first_line\n0123456789\nthird_line" }
-
-  let(:exception) do
-    OpenStruct.new(line_number: 2, line_offset: 2)
-  end
-
-  it "creates an exception with a meaningful message" do
-    message = ".. 01 `2` 3456789 .. is not permitted symbol in this context"
-
-    lex_exception = Curlybars::Error::Lex.new(source, file_name, exception)
-
-    expect(lex_exception.message).to eq message
-  end
-
-  it "figures out a position from the exception" do
-    backtrace = "%s:%d:%d" % [file_name, exception.line_number, exception.line_offset]
-
-    lex_exception = Curlybars::Error::Lex.new(source, file_name, exception)
-
-    expect(lex_exception.backtrace).to eq [backtrace]
-  end
-end